12 Jan, 2015
25 commits
-
We should be using resource_size_t and unsigned types correctly, otherwise
we sign extend the flags on a 64bit box, which is not what we want.Signed-off-by: Alan Cox
Signed-off-by: Greg Kroah-Hartman -
If uio_register_device() fails in probe(), it breaks off initialization,
deallocates all resources, but returns zero.The patch adds proper error code propagation.
Found by Linux Driver Verification project (linuxtesting.org).
Signed-off-by: Alexey Khoroshilov
Acked-by: Michael S. Tsirkin
Signed-off-by: Greg Kroah-Hartman -
The current organization of Documentation/stable_kernel_rules.txt
doesn't clearly differentiate the mutually exclusive options for
submission to the -stable review process. As I understand it, patches
are not actually required to be mailed directly to
stable@vger.kernel.org, but the instructions do not make this clear.Also, there are some established processes that are not listed --
specifically, what I call Option 2 below.This patch updates and reorganizes a bit, to make things clearer.
Signed-off-by: Brian Norris
Signed-off-by: Greg Kroah-Hartman -
Replace a misspelled function name by %s and then __func__.
This is the get function, not the set function, as was indicated by the
string.This was done using Coccinelle, including the use of Levenshtein distance,
as proposed by Rasmus Villemoes.Signed-off-by: Julia Lawall
Signed-off-by: Greg Kroah-Hartman -
This patch solves the blank line warning of checkpatch.pl
Signed-off-by: Mohammad Jamal
Signed-off-by: Greg Kroah-Hartman -
This patch removes the break statements present after return
Signed-off-by: Mohammad Jamal
Signed-off-by: Greg Kroah-Hartman -
Added a blank line after declaration to fix the warning of checkpatch.pl
Signed-off-by: Mohammad Jamal
Signed-off-by: Greg Kroah-Hartman -
Based on versioncheck.
Signed-off-by: Fabian Frederick
Signed-off-by: Greg Kroah-Hartman -
Mark driver as maintained.
Signed-off-by: Guenter Roeck
Reviewed-by: Jean Delvare
Signed-off-by: Greg Kroah-Hartman -
Fixes "Missing a blank line after declarations" reported by
checkpatch.This patch introduces no functional changes.
Signed-off-by: Rob Ward
Signed-off-by: Greg Kroah-Hartman -
Replaces the use of asm/uaccess.h with linux/uaccess.h.
Signed-off-by: Rob Ward
Signed-off-by: Greg Kroah-Hartman -
Simplify the use of CONFIG_DEVPORT by making the port_fops
so that it includes __maybe_unused.This enabled the multiple #ifdef's used for this structure
to be removed and brings it in line with the use of CONFIG_DEVMEMThis change should introduce no functional changes.
Signed-off-by: Rob Ward
Acked-by: Arnd Bergmann
Signed-off-by: Andrew Morton
Signed-off-by: Greg Kroah-Hartman -
Simplify the use of CONFIG_DEVKMEM by making the kmem_fops
so that it is __maybe_unused.This enabled the multiple #ifdef's used for this structure
to be removed and brings it in line with the use of CONFIG_DEVMEMThis change should introduce no functional changes.
Signed-off-by: Rob Ward
Acked-by: Arnd Bergmann
Signed-off-by: Andrew Morton
Signed-off-by: Greg Kroah-Hartman -
Adds Kconfig option CONFIG_DEVMEM that allows the
/dev/mem device to be disabled.Option defaults to /dev/mem enabled.
Signed-off-by: Rob Ward
Acked-by: Arnd Bergmann
Signed-off-by: Andrew Morton
Signed-off-by: Greg Kroah-Hartman -
The loop for measuring the square wave periods over some cycles is
refactored to be more easily readable. This includes avoiding a
"by-hand-implemented" for loop with a "real" one and adding some
comments.Furthermore the following compiler warning is avoided by this patch:
drivers/misc/ioc4.c: In function ‘ioc4_probe’:
drivers/misc/ioc4.c:194:16: warning: ‘start’ may be used uninitialized
in this function [-Wmaybe-uninitialized]
period = (end - start) /
^
drivers/misc/ioc4.c:148:11: note: ‘start’ was declared here
uint64_t start, end, period;
^Signed-off-by: Richard Leitner
Acked-by: Arnd Bergmann
Signed-off-by: Greg Kroah-Hartman -
We encounter situations where we got bad packet type from the
UART (probably due to platform problem or UART driver issues)
which caused us out of boundary array access,
which eventually led to kernel panic.Signed-off-by: Amir Ayun
Signed-off-by: Pavan Savoy
Signed-off-by: Leonid Iziumtsev
Signed-off-by: Gigi Joseph
Signed-off-by: Greg Kroah-Hartman -
st_kim_ref() does not take care of the fact that platform_get_drvdata() might return NULL. On AM437x EVM, this causes the platform to stop booting as soon as the module is inserted.
This patch fixes the issue by checking for NULL return value. Oops log follows.
I have not tested BT functionality after this patch. But at least the platform boots now.
[ 12.675697] Unable to handle kernel NULL pointer dereference at virtual address 0000005c
[ 12.684310] pgd = c0004000
[ 12.687157] [0000005c] *pgd=00000000
[ 12.690927] Internal error: Oops: 17 [#1] SMP ARM
[ 12.695873] Modules linked in: btwilink bluetooth ti_vpfe dwc3(+) ov2659 videobuf2_core v4l2_common videodev ti_am335x_adc 6lowpan_iphc matrix_keypad panel_dpi kfifo_buf pixcir_i2c_ts media industrialio videobuf2_dma_contig c_can_platform videobuf2_memops dwc3_omap c_can can_dev
[ 12.721969] CPU: 0 PID: 1235 Comm: kworker/u3:0 Not tainted 3.14.25-02445-g9036ac6daed6 #128
[ 12.730937] Workqueue: hci0 hci_power_on [bluetooth]
[ 12.736165] task: ebd93b40 ti: ecd7c000 task.ti: ecd7c000
[ 12.741856] PC is at st_kim_ref+0x30/0x40
[ 12.746071] LR is at st_kim_ref+0x30/0x40
[ 12.750289] pc : [] lr : [] psr: a0000013
[ 12.750289] sp : ecd7de08 ip : ecd7de08 fp : ecd7de1c
[ 12.762365] r10: bf1e710c r9 : bf1e70ec r8 : bf1e7964
[ 12.767858] r7 : ebd2fd50 r6 : bf1e7964 r5 : 00000000 r4 : ecd7de24
[ 12.774723] r3 : c0957208 r2 : 00000000 r1 : c0957208 r0 : 00000000
[ 12.781589] Flags: NzCv IRQs on FIQs on Mode SVC_32 ISA ARM Segment kernel
[ 12.789274] Control: 10c5387d Table: abde4059 DAC: 00000015
[ 12.795315] Process kworker/u3:0 (pid: 1235, stack limit = 0xecd7c248)Signed-off-by: Sekhar Nori
Signed-off-by: Gigi Joseph
Signed-off-by: Greg Kroah-Hartman -
In case the debugfs creation fails the whole init process was failing.
There is no need to do this as the shared transport can work without it.
Fix it so it just reports the failure and continue.Signed-off-by: Eyal Reizer
Signed-off-by: Gigi Joseph
Signed-off-by: Greg Kroah-Hartman -
Suspend/resume was failing if callbacks were not registered.
As it is ok not to do anything when suspending fix this
so it soen't return an error and allow the system to suspend.Signed-off-by: Eyal Reizer
Signed-off-by: Gigi Joseph
Signed-off-by: Greg Kroah-Hartman -
When using device tree, driver configuration data need to be read from
device node.
Add support for getting the platform data information from the device
tree information stored in the .dtb file in case it exists.Signed-off-by: Eyal Reizer
Signed-off-by: bvijay
Diff rendering mode:inlineside by sideSigned-off-by: Gigi Joseph
Signed-off-by: Greg Kroah-Hartman -
Add uuid, me_addr addressing also for flow control credits.
The only exception in cases for single buffer clients for which
the host address in flow credits response is always 0To in order to deal with add/remove race between fw and driver clients
addressing we need to use [uuid, me_addr] tuple to address the clientsSigned-off-by: Tomas Winkler
Signed-off-by: Greg Kroah-Hartman -
Mei bus receive and send function may return either number
of transmitted bytes or errno. It is better to use ssize_t
type for that purpose that mixing size_t with int.Signed-off-by: Tomas Winkler
Signed-off-by: Greg Kroah-Hartman -
In the case the host only injects an IPv6 address, the dhcp_enabled flag is
true (it's only for IPv4 according to Hyper-V host team), but we still need to
proceed to parse the IPv6 information.Cc: K. Y. Srinivasan
Signed-off-by: Dexuan Cui
Reviewed-By: Vitaly Kuznetsov
Signed-off-by: Greg Kroah-Hartman -
Replace calls for smp_processor_id() to get_cpu() to get the CPU ID of
the current CPU. In these instances, there is no correctness issue with
regards to preemption, we just need the current CPU ID.Signed-off-by: K. Y. Srinivasan
Signed-off-by: Greg Kroah-Hartman -
hv_fcopy_daemon is not mentioned in Makefile so it must be built
manually. Add hv_fcopy_daemon to Makefile.Signed-off-by: Matej Muzila
Signed-off-by: Greg Kroah-Hartman
29 Dec, 2014
3 commits
-
Pull KVM fixes from Paolo Bonzini:
"The important fixes are for two bugs introduced by the merge window.On top of this, add a couple of WARN_ONs and stop spamming dmesg on
pretty much every boot of a virtual machine"* tag 'for-linus' of git://git.kernel.org/pub/scm/virt/kvm/kvm:
kvm: warn on more invariant breakage
kvm: fix sorting of memslots with base_gfn == 0
kvm: x86: drop severity of "generation wraparound" message
kvm: x86: vmx: reorder some msr writing -
Pull vfs fix from Al Viro:
"An embarrassing bug in lustre patches from this cycle ;-/"* 'for-linus' of git://git.kernel.org/pub/scm/linux/kernel/git/viro/vfs:
[regression] braino in "lustre: use is_root_inode()"
28 Dec, 2014
5 commits
-
Modifying a non-existent slot is not allowed. Also check that the
first loop doesn't move a deleted slot beyond the used part of
the mslots array.Signed-off-by: Paolo Bonzini
-
Before commit 0e60b0799fed (kvm: change memslot sorting rule from size
to GFN, 2014-12-01), the memslots' sorting key was npages, meaning
that a valid memslot couldn't have its sorting key equal to zero.
On the other hand, a valid memslot can have base_gfn == 0, and invalid
memslots are identified by base_gfn == npages == 0.Because of this, commit 0e60b0799fed broke the invariant that invalid
memslots are at the end of the mslots array. When a memslot with
base_gfn == 0 was created, any invalid memslot before it were left
in place.This can be fixed by changing the insertion to use a ">=" comparison
instead of "
Reported-by: Andy Lutomirski
Tested-by: Jamie Heilman
Signed-off-by: Paolo Bonzini -
Pull sound fixes from Takashi Iwai:
"Just a couple of fixes for the new Intel Skylake HD-audio support"* tag 'sound-3.19-rc2' of git://git.kernel.org/pub/scm/linux/kernel/git/tiwai/sound:
ALSA: hda_intel: apply the Seperate stream_tag for Skylake
ALSA: hda_controller: Separate stream_tag for input and output streams. -
Since most virtual machines raise this message once, it is a bit annoying.
Make it KERN_DEBUG severity.Cc: stable@vger.kernel.org
Fixes: 7a2e8aaf0f6873b47bc2347f216ea5b0e4c258ab
Signed-off-by: Paolo Bonzini -
The commit 34a1cd60d17f, "x86: vmx: move some vmx setting from
vmx_init() to hardware_setup()", tried to refactor some codes
specific to vmx hardware setting into hardware_setup(), but some
msr writing should depend on our previous setting condition like
enable_apicv, enable_ept and so on.Reported-by: Jamie Heilman
Tested-by: Jamie Heilman
Signed-off-by: Tiejun Chen
Signed-off-by: Paolo Bonzini
27 Dec, 2014
3 commits
-
In one of the places (ll_md_blocking_ast()) we had open-coded
!is_root_inode(inode) and replaced it with is_root_inode(inode).
See the last chunk of f76c23:
- inode != inode->i_sb->s_root->d_inode)
+ is_root_inode(inode))
should've been
+ !is_root_inode(inode))
obviously...Signed-off-by: Al Viro
-
Pull parisc build fix from Helge Deller:
"This unbreaks the kernel compilation on parisc with gcc-4.9"* 'parisc-3.19-1' of git://git.kernel.org/pub/scm/linux/kernel/git/deller/parisc-linux:
parisc: fix out-of-register compiler error in ldcw inline assembler function -
The __ldcw macro has a problem when its argument needs to be reloaded from
memory. The output memory operand and the input register operand both need to
be reloaded using a register in class R1_REGS when generating 64-bit code.
This fails because there's only a single register in the class. Instead, use a
memory clobber. This also makes the __ldcw macro a compiler memory barrier.Signed-off-by: John David Anglin
Cc: [3.13+]
Signed-off-by: Helge Deller
26 Dec, 2014
4 commits
-
The total stream number of Skylake's input and output stream
exceeds 15, which will cause some streams do not work because
of the overflow on SDxCTL.STRM field if using the legacy
stream tag allocation method.This patch uses the new stream tag allocation method by add
the flag AZX_DCAPS_SEPARATE_STREAM_TAG for Skylake platform.Signed-off-by: Libin Yang
Reviewed-by: Vinod Koul
Signed-off-by: Takashi Iwai -
Implemented separate stream_tag assignment for input and output streams.
According to hda specification stream tag must be unique throughout the
input streams group, however an output stream might use a stream tag
which is already in use by an input stream. This change is necessary
to support HW which provides a total of more than 15 stream DMA engines
which with legacy implementation causes an overflow on SDxCTL.STRM
field (and the whole SDxCTL register) and as a result usage of
Reserved value 0 in the SDxCTL.STRM field which confuses HDA controller.Signed-off-by: Rafal Redzimski
Signed-off-by: Jayachandran B
Signed-off-by: Libin Yang
Reviewed-by: Vinod Koul
Signed-off-by: Takashi Iwai -
Pull drm fixes from Dave Airlie:
"Xmas fixes pull:core:
one atomic fix, revert the WARN_ON dumb buffers patch.agp:
fixup Dave J.nouveau:
fix 3.18 regression for old userspacetegra fixes:
vblank and iommu fixesamdkfd:
fix bugs shown by testing with userspace, init apertures oncemsm:
hdmi fixes and cleanupi915:
misc fixesThere is also a link ordering fix that I've asked to be cc'ed to you,
putting iommu before gpu, it fixes an issue with amdkfd when things
are all in the kernel, but I didn't like sending it via my tree
without discussion.I'll probably be a bit on/off for a few weeks with pulls now, due to
holidays and LCA, so don't be surprised if stuff gets a bit backed up,
and things end up a bit large due to lag"* 'drm-fixes' of git://people.freedesktop.org/~airlied/linux: (28 commits)
Revert "drm/gem: Warn on illegal use of the dumb buffer interface v2"
agp: Fix up email address & attributions in AGP MODULE_AUTHOR tags
nouveau: bring back legacy mmap handler
drm/msm/hdmi: rework HDMI IRQ handler
drm/msm/hdmi: enable regulators before clocks to avoid warnings
drm/msm/mdp5: update irqs on crtcencoder link change
drm/msm: block incoming update on pending updates
drm/atomic: fix potential null ptr on plane enable
drm/msm: Deletion of unnecessary checks before the function call "release_firmware"
drm/msm: Deletion of unnecessary checks before two function calls
drm/tegra: dc: Select root window for event dispatch
drm/tegra: gem: Use the proper size for GEM objects
drm/tegra: gem: Flush buffer objects upon allocation
drm/tegra: dc: Fix a potential race on page-flip completion
drm/tegra: dc: Consistently use the same pipe
drm/irq: Add drm_crtc_vblank_count()
drm/irq: Add drm_crtc_handle_vblank()
drm/irq: Add drm_crtc_send_vblank_event()
drm/i915: Disable PSMI sleep messages on all rings around context switches
drm/i915: Force the CS stall for invalidate flushes
... -
Pull ipmi driver bugfixes from Corey Minyard:
"Fix two bugs:One that lockdep turned up, I didn't go far enough with cleanup of
attributes for IPMI. This has been there a long time; my previous fix
of this didn't fix all the attributes.One fix for some arches that need an explicit linux/ctype.h for
isspace()"* tag 'for-linus-2' of git://git.code.sf.net/p/openipmi/linux-ipmi:
ipmi: Fix compile issue with isspace()
ipmi: Finish cleanup of BMC attributes