RE: [GIT PULL] Samsung fixes for v4.3

Krzysztof Kozlowski wrote: > > On 30.09.2015 15:50, Kukjin Kim wrote: > > On 09/30/15 08:59, Krzysztof Kozlowski wrote: > >> Dear Kukjin, > >> > >> Below you will find fixes for current release cycle which are not > >> present yet in your tree. > >> > >> Best regards, > >> Krzysztof > >> > >> >

Re: [PATCH 3/3] mm/nommu: drop unlikely behind BUG_ON()

On Sun, 04 Oct 2015, Leon Romanovsky wrote: On Sun, Oct 4, 2015 at 9:18 AM, Geliang Tang wrote: BUG_ON() already contain an unlikely compiler flag. Drop it. It is not the case if CONFIG_BUG and HAVE_ARCH_BUG_ON are not set. Yeah, but that's like the 1% of the cases --

Re: [PATCH v4 17/26] ARM: pxa: magician: Fix wrongly enabled USB host ports

Dne 3.10.2015 v 14:45 Philipp Zabel napsal(a): > Am Montag, den 28.09.2015, 23:38 +0200 schrieb Petr Cvek: >> USB host ports on the HTC Magician are wrongly enabled. Port 1 is for >> bluetooth and port 2 is for OTG (mux in the charger connector). >> >> Signed-off-by: Petr Cvek

RE: [V4 PATCH 4/4] x86/apic: Introduce noextnmi boot option

> On Fri, Oct 02, 2015 at 12:58:02AM +, 河合英宏 / KAWAI，HIDEHIRO wrote: > > > On Thu, Oct 01, 2015 at 10:24:19AM +, 河合英宏 / KAWAI，HIDEHIRO wrote: > > > > But how do we check if the starting kernel is a dump capture kernel? > > > > > > How does that first kernel pass info to the capture kernel?

Re: [PATCH 2/2] NTB: remove unused variable

On Sat, Oct 3, 2015 at 4:09 AM, Sudip Mukherjee wrote: > These variables were not used anywhere. So remove them. > > Signed-off-by: Sudip Mukherjee Applied. Thanks, Jon > --- > drivers/ntb/ntb_transport.c | 4 > 1 file changed, 4

Re: [PATCH v4 2/4] mm, proc: account for shmem swap in /proc/pid/smaps

On Fri, 2 Oct 2015, Vlastimil Babka wrote: > Currently, /proc/pid/smaps will always show "Swap: 0 kB" for shmem-backed > mappings, even if the mapped portion does contain pages that were swapped out. > This is because unlike private anonymous mappings, shmem does not change pte > to swap entry,

[PATCH v2] arm: Adding support for atomic half word exchange

Hello Will, This is my second version of the patch which covers the byte exclusive case as pointed out by you. Please share your opinion on this. v1-->v2 : Extended the guard code to cover the byte exchange case as well following opinion of Will Deacon. Checkpatch has been run and issues

APPLY TODAY

WE OFFER LOAN AT 3% INTEREST RATE -- To unsubscribe from this list: send the line "unsubscribe linux-kernel" in the body of a message to majord...@vger.kernel.org More majordomo info at http://vger.kernel.org/majordomo-info.html Please read the FAQ at http://www.tux.org/lkml/

Re: [PATCH 8/8] bdc_pci: use PCI_VDEVICE() instead of PCI_DEVICE()

On Mon, Oct 05, 2015 at 12:08:26AM +0300, Sergei Shtylyov wrote: > Fix using the PCI_DEVICE() macro instead of less verbose PCI_VDEVICE(). Why? I hate PCI_VDEVICE(), it's impossible to grep for things and does not help with readability and is pointless. My one wish was that when I was the PCI

Re: [PATCH 2/2] drivers: staging: wilc1000: Call kfree only for error cases

On 2015년 10월 04일 19:28, Chandra Gorentla wrote: On Sun, Oct 04, 2015 at 09:44:57AM +0100, Greg KH wrote: On Sat, Oct 03, 2015 at 02:57:30PM +0530, Chandra S Gorentla wrote: - kfree is being called for the members of the queue without de-queuing them; they are just inserted within this

Re: [PATCH 1/2] drivers: staging: wilc1000: Move spin lock to the start of critical section

On 2015년 10월 04일 17:43, Greg KH wrote: On Sat, Oct 03, 2015 at 02:57:29PM +0530, Chandra S Gorentla wrote: The spin_lock_irqsave is moved to just beginning of critical section. This change moves a couple of return statements out of the lock. Signed-off-by: Chandra S Gorentla

Re: [PATCH 3/3] mm/nommu: drop unlikely behind BUG_ON()

On Mon, 05 Oct 2015, Geliang Tang wrote: On Sun, Oct 04, 2015 at 06:50:55PM -0700, Davidlohr Bueso wrote: On Sun, 04 Oct 2015, Geliang Tang wrote: >BUG_ON() already contain an unlikely compiler flag. Drop it. > >Signed-off-by: Geliang Tang Acked-by: Davidlohr Bueso

[PATCH] vme: tsi148: silence uninitialized variable warning

The warning is a false positive. drivers/vme/bridges/vme_tsi148.c: In function 'tsi148_master_write': drivers/vme/bridges/vme_tsi148.c:1358:31: warning: 'handler' may be used uninitialized in this function [-Wmaybe-uninitialized] vme_unregister_error_handler(handler);

Re: [PATCH] ARM: multi_v7_defconfig: Add missing QCOM APQ8064 configs

On Thu, Oct 01, 2015 at 01:57:38PM +0100, Srinivas Kandagatla wrote: > This patch adds few missing essential configs in the multi_v7_defconf, > absense of some configs like PINCTRL_APQ8064 would prevent the board from > getting access to serial. > > cc: Kevin Hilman > cc:

[PATCH v2] x86/apic: Use smaller array for __apicid_to_node[] mapping

The Intel x2APIC spec states the upper 16-bits of APIC ID is the cluster ID [1, p2-12], intended for future distributed systems. Beyond the legacy 8-bit APIC ID, Numascale NumaConnect uses 4-bits for the position of a server on each axis of a multi-dimension torus; SGI NUMAlink also structures the

Re: [RFC PATCH v8 0/10] Add external dma support for Synopsys MSHC

Hi, Shawn. On 10/02/2015 06:49 PM, Shawn Lin wrote: > On 2015/10/1 17:14, Jaehoon Chung wrote: >> Dear, All. >> >> I will apply patch 01-03 at my repository on today. >> But i don't know better how i do about other patches relevant to config file. >> > > Thanks, Jaehoon. :) > > I guess it would

[PATCH 3/4] sched/fair: Remove scale_load_down() for load_avg

Currently, load_avg = scale_load_down(load) * runnable%. This does not make much sense, because load_avg is primarily the load that takes runnable time ratio into account. We therefore remove scale_load_down() for load_avg. But we need to carefully consider the overflow risk if load has higher

[PATCH 4/4] sched/fair: Rename scale_load() and scale_load_down()

Rename scale_load() and scale_load_down() to user_to_kernel_load() and kernel_to_user_load() respectively, to allow the names to bear what they are really about. Signed-off-by: Yuyang Du --- kernel/sched/core.c | 8 kernel/sched/fair.c | 7 ---

[PATCH 0/4] sched/fair: Clean up sched metric definitions

Hi Peter and Ingo, As discussed recently, the sched metrics need a little bit cleanup. This series of patches attempt to do that: refactor, rename, remove... Thanks a lot to Ben, Morten, Dietmar, Vincent, and others who provided valuable comments. Thanks, Yuyang Yuyang Du (4): sched/fair:

[PATCH 2/4] sched/fair: Remove SCHED_LOAD_SHIFT and SCHED_LOAD_SCALE

After cleaning up the sched metrics, these two definitions that cause ambiguity are not needed any more. Use NICE_0_LOAD_SHIFT and NICE_0_LOAD instead (the names suggest clearly who they are). Suggested-by: Ben Segall Signed-off-by: Yuyang Du ---

[PATCH 1/4] sched/fair: Generalize the load/util averages resolution definition

Metric needs certain resolution to allow detail we can look into, which also determines the range of the metric. For instance, increasing the resolution of [0, 1] (two levels), one can multiply 1024 and get [0..1024] (1025 levels). In sched/fair, a few metrics depend on the resolution: weight,

Re: [PATCH 3/3] mm/nommu: drop unlikely behind BUG_ON()

On Sun, 04 Oct 2015, Geliang Tang wrote: BUG_ON() already contain an unlikely compiler flag. Drop it. Signed-off-by: Geliang Tang Acked-by: Davidlohr Bueso ... but I believe you do have some left: drivers/scsi/scsi_lib.c:

Lucky Email Winner

We are please to inform you of our yearly promotion award program held 1st October 2015,have pronounce your email-address as the lucky winner of a total sum:of €500,000.00(Five Hundred Thousand Euros Only) CONTACT: The claim office/Email:posti...@luckymail.com,with your winning

Re: [PATCH 0/2] Add RT5033 Flash LED driver

Hi, Jacek Thanks for review my patch Oops! old version patch is sent. I should have checked.. I'll send new patch, soon. Thanks Ingi Kim On 2015년 10월 02일 23:59, Jacek Anaszewski wrote: > Hi Ingi, > > Thanks for the patches. Please fix build errors > and resubmit. > > On 10/02/2015 11:41 AM,

Re: [PATCH v4 07/26] ARM: pxa: magician: Fix HTC Magician pin mux definitions

Dne 3.10.2015 v 14:30 Philipp Zabel napsal(a): > Am Montag, den 28.09.2015, 23:15 +0200 schrieb Petr Cvek: >> This patch fixes a pin mux for the HTC Magician machine. Wrong and >> missing >> definitions caused a bad LCD operation and an unavailability of >> several >> peripherals. >> >>

Re: [PATCH 3/3] mm/nommu: drop unlikely behind BUG_ON()

On Sun, Oct 04, 2015 at 06:50:55PM -0700, Davidlohr Bueso wrote: > On Sun, 04 Oct 2015, Geliang Tang wrote: > > >BUG_ON() already contain an unlikely compiler flag. Drop it. > > > >Signed-off-by: Geliang Tang > > Acked-by: Davidlohr Bueso > > ... but I

Re: [PATCH 1/2] NTB: fix access of free-ed pointer

On Sat, Oct 3, 2015 at 4:09 AM, Sudip Mukherjee wrote: > We were accessing nt->mw_vec after freeing it. Fix the error path so > that we free nt->mw_vec after we have finished using it. > > Signed-off-by: Sudip Mukherjee Good catch. Pulled into

Re: [PATCH v3 1/3] uio: add ioctl support

On Sun, Oct 04, 2015 at 11:43:16PM +0300, Vlad Zolotarov wrote: > Signed-off-by: Vlad Zolotarov > --- > drivers/uio/uio.c | 15 +++ > include/linux/uio_driver.h | 3 +++ > 2 files changed, 18 insertions(+) You add an ioctl yet fail to justify

Re: [PATCH v7 5/6] Documentation: dt-bindings: pci: altera pcie device tree binding

On Sat, Oct 3, 2015 at 5:56 AM, Arnd Bergmann wrote: > On Friday 02 October 2015 15:53:44 Ley Foon Tan wrote: >> > Strictly speaking, if you have undocumented bindings downstream that >> > is your problem and we don't have to accept them as-is upstream. I'm >> > not going to worry

Re: [PATCH v3 2/3] uio_pci_generic: add MSI/MSI-X support

On Sun, Oct 04, 2015 at 11:43:17PM +0300, Vlad Zolotarov wrote: > Add support for MSI and MSI-X interrupt modes: >- Interrupt mode selection order is: > INT#X (for backward compatibility) -> MSI-X -> MSI. >- Add ioctl() commands: > - UIO_PCI_GENERIC_INT_MODE_GET: query the

APPLY TODAY

WE OFFER LOAN AT 3% INTEREST RATE -- To unsubscribe from this list: send the line "unsubscribe linux-kernel" in the body of a message to majord...@vger.kernel.org More majordomo info at http://vger.kernel.org/majordomo-info.html Please read the FAQ at http://www.tux.org/lkml/

Re: [PATCH 0/2] memory: Fix module autoload for OF platform driver

On 10/4/15 4:36 AM, Greg Kroah-Hartman wrote: On Tue, Sep 29, 2015 at 10:40:17AM -0700, santosh shilimkar wrote: +Greg, arm-sock folks On 9/17/2015 10:56 AM, santosh shilimkar wrote: On 9/17/2015 10:41 AM, Luis de Bethencourt wrote: Hello, These patches add the missing MODULE_DEVICE_TABLE()

Re: [RESEND PATCH 0/2] memory: Fix module autoload for OF platform driver

On 10/4/15 9:23 AM, Luis de Bethencourt wrote: Hi, Resending this patch series as requested by Greg. So he can have them in his queue for picking up. https://lkml.org/lkml/2015/10/4/76 These patches add the missing MODULE_DEVICE_TABLE() for OF to export the information so modules have the

Re: [PATCH v4 3/4] mm, shmem: Add shmem resident memory accounting

On Fri, 2 Oct 2015, Vlastimil Babka wrote: > From: Jerome Marchand > > Currently looking at /proc//status or statm, there is no way to > distinguish shmem pages from pages mapped to a regular file (shmem > pages are mapped to /dev/zero), even though their implication in >

Re: [PATCH 1/3] dt-bindings: Document the STM32 HW RNG bindings

Hi Daniel, On 10/03/2015 10:35 PM, Daniel Thompson wrote: This adds documenttaion of device tree binds for the STM32 hardware s/documenttaion/documentation/ random number generator. Signed-off-by: Daniel Thompson --- .../devicetree/bindings/hwrng/stm32-rng.txt

Re: [PATCH] staging: rtl8723au: Fix Sparse errors in rtl8723a_cmd.c

On Fri, Oct 02, 2015 at 08:36:28PM -0400, Jacob Kiefer wrote: > From: Jacob Kiefer > > This patch fixes the following sparse errors: > > > CHECK drivers/staging/rtl8723au/hal/rtl8723a_cmd.c > ... > drivers/staging/rtl8723au/hal/rtl8723a_cmd.c:118:25: warning: incorrect

[PATCH] mwifiex: fix a comment typo

Just fix a typo in the code comment. Signed-off-by: Geliang Tang --- drivers/net/wireless/mwifiex/cfg80211.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/drivers/net/wireless/mwifiex/cfg80211.c b/drivers/net/wireless/mwifiex/cfg80211.c index

[PATCH] isci: fix two comment typos

Just fix two typos in the code comment. Signed-off-by: Geliang Tang --- drivers/scsi/isci/request.c | 4 ++-- 1 file changed, 2 insertions(+), 2 deletions(-) diff --git a/drivers/scsi/isci/request.c b/drivers/scsi/isci/request.c index cfd0084..8fe106f 100644 ---

Re: [PATCH] staging: rdma: Fix braces around if/else

On Sun, Oct 04, 2015 at 08:20:09AM +0100, Greg Kroah-Hartman wrote: On Sun, Oct 04, 2015 at 09:47:52AM +0300, Or Gerlitz wrote: On 10/3/2015 11:55 PM, Martin Kletzander wrote: >Get rid of all ELSE_AFTER_BRACE type errors reported by checkpatch.pl. Hi Greg, Is there a way to signal

Re: [PATCH 4/4] nvmem: sunxi: Check for memory allocation failure

On Wed, Sep 30, 2015 at 01:36:31PM +0100, Srinivas Kandagatla wrote: > From: Maxime Ripard > > The sunxi_sid driver doesn't check for kmalloc return status before > derefencing the returned pointer, which could lead to a NULL pointer > dereference if kmalloc

Re: [PATCH 3/3] mm/nommu: drop unlikely behind BUG_ON()

On Sun, Oct 4, 2015 at 9:18 AM, Geliang Tang wrote: > BUG_ON() already contain an unlikely compiler flag. Drop it. It is not the case if CONFIG_BUG and HAVE_ARCH_BUG_ON are not set. > > Signed-off-by: Geliang Tang > --- > mm/nommu.c | 10 +- >

Re: [PATCH 8/24] ver_linux: e2fsprogs.patch

On Sun, Oct 4, 2015 at 4:48 AM, Theodore Ts'o wrote: > On Sat, Oct 03, 2015 at 04:23:02PM +0300, Alexander Kapshuk wrote: >> 'tune2fs' is located in varying places depending on the distro. >> Current implementation output on distros where 'tune2fs' is found at >> a location that is

Re: [PATCH v2] scsi: dpt: remove i2o header in uapi

On Sun, Oct 04, 2015 at 01:22:50AM +0200, Gabriel Laskar wrote: > The i2o subsystem was removed by > commit 4a72a7af462d ("staging: remove i2o subsystem") but the header was > still used by drivers/scsi/dpt_i2o.c driver. > > This patch moves the structures used from the header into dpti_i2o.h. >

Re: [PATCH 2/24] ver_linux: path2bin_test.patch

On Sat, Oct 3, 2015 at 4:10 PM, Alexander Kapshuk wrote: > As suggested by Jim Davis , this patch checks for > binaries in the user's $PATH first by calling 'which name-of-binary'. If > the binary is not found there, an attempt is made to find it

Re: [PATCH v2 6/7] squashfs: xattr simplifications

2015-10-04 8:29 GMT+02:00 Christoph Hellwig : > IS it just me or is this handler broke before and after this patch? It > only copies out the prefix, but not the actual xattr name. Well, squashfs appends the rest of the name in squashfs_listxattr, so it's not broken, just

Re: [PATCH v3 2/4] firmware: use acpi to detect QEMU fw_cfg device for sysfs fw_cfg driver

On Sat, Oct 03, 2015 at 07:28:07PM -0400, Gabriel L. Somlo wrote: > From: Gabriel Somlo > > Instead of blindly probing fw_cfg registers at known IOport and MMIO > locations, use the ACPI subsystem to determine whether a QEMU fw_cfg > device is present, and, if found, to initialize

nfs race condition

Hello, I'm doing some " crontab -e, exit with :x" on NFS mount point, and there is a race condition whent removing temporary files : 7701 write(2, "No modification made\n", 21) = 21 7701 open("/tmp/crontab.vYPoHR", O_RDONLY|O_NONBLOCK|O_DIRECTORY|O_CLOEXEC) = 4 7701 getdents(4,

[PATCH v7 7/7] perf config: Add a option 'remove' to perf-config

A option 'remove' is to remove specific config variables. For the syntax examples, # perf config [] -r | --remove [section.name ...] Signed-off-by: Taeung Song --- tools/perf/Documentation/perf-config.txt | 6 + tools/perf/builtin-config.c | 38

[PATCH v7 6/7] perf config: normalize a value depending on default type of it

Whether or not user mis-type wrong data type to set config, normalize the value. If a config user enter isn't contained in default configs, just pass as it is. For the examples, # perf config report.queue-size=1M # perf config report.queue-size report.queue-size=1048576 Signed-off-by:

[PATCH v7 1/7] perf tools: Add 'perf-config' command

The perf configuration file contains many variables which can make the perf command's action more effective. But looking through state of configuration is difficult and there's no knowing what kind of other variables except variables in perfconfig.example exist. So This patch adds 'perf-config'

[PATCH v7 0/7] perf tools: Add 'perf-config' command

So far, it is difficult that the state of perf configs is looked through and there's no knowing what kind of other variables except variables in perfconfig.example. Also perf configs can't be changed without manually modifying $HOME/.perfconfig or $(sysconfdir)/perfconfig file. So I suggest this

[PATCH] btrfs: fix a comment typo

Just fix a typo in the code comment. Signed-off-by: Geliang Tang --- fs/btrfs/free-space-cache.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/fs/btrfs/free-space-cache.c b/fs/btrfs/free-space-cache.c index abe3a66..7332995 100644 ---

Re: [PATCH 2/2] drivers: staging: wilc1000: Call kfree only for error cases

On Sun, Oct 04, 2015 at 09:44:57AM +0100, Greg KH wrote: > Aren't you now leaking memory as you aren't freeing pstrMessage and the > buffer on the "normal" return path? It's supposed to. It's a bug fix. I explained to him in the first version that his changelog sucks. regards, dan carpenter

Re: [PATCH 1/2] drivers: staging: wilc1000: Move spin lock to the start of critical section

On Sun, Oct 04, 2015 at 09:43:35AM +0100, Greg KH wrote: > On Sat, Oct 03, 2015 at 02:57:29PM +0530, Chandra S Gorentla wrote: > > The spin_lock_irqsave is moved to just beginning of critical section. > > This change moves a couple of return statements out of the lock. > > > > Signed-off-by:

Re: [PATCH 2/3] hwrng: stm32 - add support for STM32 HW RNG

On Sat, Oct 3, 2015 at 10:35 PM, Daniel Thompson wrote: > Add support for STMicroelectronics STM32 random number generator. > > The config value defaults to N, reflecting the fact that STM32 is a > very low resource microcontroller platform and unlikely to be targeted

Re: [PATCH][RESEND] drm: panel-simple: add URT UMSH-8596MD-xT panel support

Hi Maciej, On 2 October 2015 at 22:40, Maciej S. Szmigiero wrote: > Anybody here? > > I've already submitted this patch two times but received no response... > Seems that the maintainer (Thierry) isn't Cc'ed. You might want to split the DT binding and vendor prefix to

Re: Problems with printk logs and my driver

On 30 September 2015 at 13:07, Austin S Hemmelgarn wrote: > On 2015-09-29 18:11, Eric Curtin wrote: >> >> On 25 September 2015 at 16:45, Austin S Hemmelgarn >> wrote: >>> >>> On 2015-09-25 08:02, Jiri Kosina wrote: On Fri, 25 Sep 2015,

Re: [PATCH 1/2] block: generic request_queue reference counting

On Wed, Sep 30, 2015 at 8:41 AM, Dan Williams wrote: > Allow pmem, and other synchronous/bio-based block drivers, to fallback Just a bit curious, why not extend it for all(both synchronous and asynchrounous) bio-based drivers? As you mentioned in introductory message,

[PATCH v7 2/7] perf config: Add '--system' and '--user' options to select which config file is used

Which config file is used is decided in only perf_config(). And a perf-config command depend on perf_config() to list config variables with values. So add '--system' and '--user' options to select which config file to be used without perf_config(). The file-options '--system' means

[PATCH v7 4/7] perf config: Add 'get' functionality

This patch consists of functions which can get specific config variables. For the syntax examples, perf config [] [section.name ...] display key-value pairs of specific config variables # perf config report.queue-size report.children Signed-off-by: Taeung Song

[PATCH v7 5/7] perf config: Add 'set' feature

This patch consists of functions which can set specific config variables. For the syntax examples, perf config [] [options] [section.name[=value] ...] set specific config variables # perf config report.queue-size=100M report.children=true Signed-off-by: Taeung Song

[PATCH v7 3/7] perf config: Add a option 'list-all' to perf-config

A option 'list-all' is to display both current config variables and all possible config variables with default values. The syntax examples are like below perf config [] [options] display all perf config with default values. # perf config -a | --list-all Signed-off-by: Taeung Song

[PATCH] IB/iser: fix a comment typo

Just fix a typo in the code comment. Signed-off-by: Geliang Tang --- drivers/infiniband/ulp/iser/iscsi_iser.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/drivers/infiniband/ulp/iser/iscsi_iser.c b/drivers/infiniband/ulp/iser/iscsi_iser.c index

Re: [PATCH V5 1/2] ACPI / EC: Fix broken 64bit big-endian users of 'global_lock'

On Sun, Sep 27, 2015 at 03:48:24PM +0200, Rafael J. Wysocki wrote: > On Sun, Sep 27, 2015 at 12:04 AM, Viresh Kumar > wrote: > > global_lock is defined as an unsigned long and accessing only its lower > > 32 bits from sysfs is incorrect, as we need to consider other 32

Re: [PATCH] staging: rdma: Fix braces around if/else

On Sun, Oct 04, 2015 at 11:05:50AM +0200, Martin Kletzander wrote: > On Sun, Oct 04, 2015 at 08:20:09AM +0100, Greg Kroah-Hartman wrote: > >On Sun, Oct 04, 2015 at 09:47:52AM +0300, Or Gerlitz wrote: > >>On 10/3/2015 11:55 PM, Martin Kletzander wrote: > >>>Get rid of all ELSE_AFTER_BRACE type

Re: [PATCH 2/2] drivers: staging: wilc1000: Call kfree only for error cases

On Sun, Oct 04, 2015 at 12:16:31PM +0300, Dan Carpenter wrote: > On Sun, Oct 04, 2015 at 09:44:57AM +0100, Greg KH wrote: > > Aren't you now leaking memory as you aren't freeing pstrMessage and the > > buffer on the "normal" return path? > > It's supposed to. It's a bug fix. I explained to him

Re: [PATCH] coccinelle: misc: remove "complex return code" warnings

On Fri, Oct 02, 2015 at 11:33:46PM +0200, Julia Lawall wrote: > Do you consider that this function would be better off in two lines? > > static int mxt_acquire_irq(struct mxt_data *data) > { > int error; > > enable_irq(data->irq); > > error =

Re: [PATCH] coccinelle: misc: remove "complex return code" warnings

On Sat, Oct 03, 2015 at 06:24:27PM +0200, Julia Lawall wrote: > Acked-by: Julia Lawall > > Perhaps there is a more restricted version that can be acceptable, but I'm > OK with dropping the current version. Great, thanks! Johan -- To unsubscribe from this list: send the

Re: [char-misc 1/2 4.3 V2] mei: Fix debugfs filename in error output

On Mon, Sep 21, 2015 at 10:58:15AM +0300, Tomas Winkler wrote: > From: Alexander Kuleshov > > Signed-off-by: Alexander Kuleshov > Signed-off-by: Tomas Winkler > --- > V2: fixed author address > >

Re: [PATCH 0/2] memory: Fix module autoload for OF platform driver

On Tue, Sep 29, 2015 at 10:40:17AM -0700, santosh shilimkar wrote: > +Greg, arm-sock folks > > On 9/17/2015 10:56 AM, santosh shilimkar wrote: > >On 9/17/2015 10:41 AM, Luis de Bethencourt wrote: > >>Hello, > >> > >>These patches add the missing MODULE_DEVICE_TABLE() for OF to export > >>the

Re: [PATCH] Staging: iio: meter: Use devm functions

On 10/04/2015 06:34 AM, Shraddha Barke wrote: > [...] Hi, Thanks for the patch. > - st->trig = iio_trigger_alloc("%s-dev%d", > - spi_get_device_id(st->us)->name, > - indio_dev->id); > + st->trig =

Re: [PATCH] extcon: Modify the id and name of external connector

On Sun, Oct 04, 2015 at 12:45:36PM +0900, Chanwoo Choi wrote: > This patch modifies the id and name of external connector with the additional > prefix to clarify both attribute and meaning of external connector as > following: > - EXTCON_CHG_* mean the charger connector. > - EXTCON_JACK_* mean

Re: [PATCH 02/11] staging: dgnc: remove multiple blank lines

On Sat, Oct 03, 2015 at 08:52:40PM +0530, Sudip Mukherjee wrote: > checkpatch warns us about multiple blank lines which are not needed. > Remove them. > > Signed-off-by: Sudip Mukherjee > --- > drivers/staging/dgnc/dgnc_driver.c | 10 >

Re: [PATCH 2/2] drivers: staging: wilc1000: Call kfree only for error cases

On Sat, Oct 03, 2015 at 02:57:30PM +0530, Chandra S Gorentla wrote: > - kfree is being called for the members of the queue without >de-queuing them; they are just inserted within this function; >they are supposed to be de-queued and freed in a function >for receiving the queue items >

Re: [PATCH 1/2] drivers: staging: wilc1000: Move spin lock to the start of critical section

On Sat, Oct 03, 2015 at 02:57:29PM +0530, Chandra S Gorentla wrote: > The spin_lock_irqsave is moved to just beginning of critical section. > This change moves a couple of return statements out of the lock. > > Signed-off-by: Chandra S Gorentla > --- >

Re: [PATCH 3/3] Staging: rtl8192u: remove ieee80211_ccmp_null()

On Sat, Oct 03, 2015 at 01:09:56PM -0800, mike dupuis wrote: > This is a patch to remove the function ieee80211_ccmp_null(). > This function does nothing and can therefore be safely removed. > > Signed-off-by: Mike Dupuis > --- >

Re: [PATCH 15/15] dma: remove external references to dma_supported

On Sat, Oct 03, 2015 at 05:19:39PM +0200, Christoph Hellwig wrote: > Signed-off-by: Christoph Hellwig > --- > Documentation/DMA-API.txt | 13 - > drivers/usb/host/ehci-hcd.c | 2 +- > drivers/usb/host/fotg210-hcd.c | 2 +- > drivers/usb/host/fusbh200-hcd.c |

Re: [PATCH 06/14] RFC: usb/host/fotg210: replace msleep by usleep_range

On Fri, Oct 2, 2015 at 7:52 PM, Alan Stern wrote: > On Fri, 2 Oct 2015, Felipe Balbi wrote: > >> On Mon, Sep 21, 2015 at 05:01:10PM +0200, Peter Senna Tschudin wrote: >> > msleep under 20ms can result in sleeping up to 20ms, which may not be >> > intended. Replace

Re: [PATCH 1/2] drivers: staging: wilc1000: Move spin lock to the start of critical section

On Sun, Oct 04, 2015 at 03:37:13PM +0530, Chandra Gorentla wrote: > On Sun, Oct 04, 2015 at 09:43:35AM +0100, Greg KH wrote: > > On Sat, Oct 03, 2015 at 02:57:29PM +0530, Chandra S Gorentla wrote: > > > The spin_lock_irqsave is moved to just beginning of critical section. > > > This change moves a

Re: [PATCH 2/2] drivers: staging: wilc1000: Call kfree only for error cases

On Sun, Oct 04, 2015 at 09:44:57AM +0100, Greg KH wrote: > On Sat, Oct 03, 2015 at 02:57:30PM +0530, Chandra S Gorentla wrote: > > - kfree is being called for the members of the queue without > >de-queuing them; they are just inserted within this function; > >they are supposed to be

For Your Kind Notice

You have been selected to receive ($950,000.00 USD) as Charity Donations/Aid from (Qatar Foundation) provide the below information ASAP. 1:Full Names: 2:Complete Address: 3:Direct Telephone numbers Sincerely, Dr.Allan Faroze Mohammed. Email: poesq0...@foxmail.com Tel:+44 70457 96790 -- To

Re: [PATCH] misc: mic: replace kfree with put_device

On Wed, Sep 30, 2015 at 12:58:22AM +0800, Geliang Tang wrote: > Handle a failed device_register(), replace kfree() with put_device(), > which will call mbus_release_dev() or scif_release_dev(). > > Signed-off-by: Geliang Tang > --- > drivers/misc/mic/bus/mic_bus.c | 2 +- >

Re: [PATCH] staging: iio: adc: fix comment block coding style issue

On 10/03/2015 08:32 PM, Hugo Camboulive wrote: > This patch to ad7746.c makes the comment block end with a */ > on a separate line. > > Signed-off-by: Hugo Camboulive > --- > drivers/staging/iio/cdc/ad7746.c | 2 +- > 1 file changed, 1 insertion(+), 1 deletion(-) > >

Re: [PATCH] misc: mic: use kstrdup() in mic_sysfs

On Fri, Oct 02, 2015 at 10:43:34AM +0800, Geliang Tang wrote: > Use kstrdup instead of kmalloc and strncpy. > > Signed-off-by: Geliang Tang > --- > drivers/misc/mic/host/mic_sysfs.c | 23 --- > 1 file changed, 4 insertions(+), 19 deletions(-) This file

Re: [PATCH 2/2] drivers/base/node.c: skip non-present sections in register_mem_sect_under_node

On Tue, Sep 22, 2015 at 04:27:01AM -0400, Jan Stancek wrote: > > > > > - Original Message - > > From: "Jan Stancek" > > To: gre...@linuxfoundation.org > > Cc: linux-kernel@vger.kernel.org, linux...@kvack.org, "Dave Young" > > > > Sent:

Re: [PATCH] Staging: iio: meter: Use devm functions

On 04/10/15 05:34, Shraddha Barke wrote: > Introduce use of managed resource function devm_iio_trigger_alloc > instead of iio_trigger_alloc and devm_request_irq instead of request_irq > Remove corresponding calls to iio_trigger_free and free_irq in the probe > and remove functions. > The now

Re: [PATCH] Staging: iio: meter: Use devm functions

On Sun, 4 Oct 2015, Jonathan Cameron wrote: On 04/10/15 05:34, Shraddha Barke wrote: Introduce use of managed resource function devm_iio_trigger_alloc instead of iio_trigger_alloc and devm_request_irq instead of request_irq Remove corresponding calls to iio_trigger_free and free_irq in the

Re: [PATCH 01/18] mpt2sas: Use mpi headers from mpt3sas

Wasn't this originally my patch? Not that I really care for this trivial move, but the commit log just sounded familar. -- To unsubscribe from this list: send the line "unsubscribe linux-kernel" in the body of a message to majord...@vger.kernel.org More majordomo info at

Re: RFC: reduce CONFIG_SCSI_CONSTANTS impact by 4k

On Sun, Oct 04, 2015 at 12:09:58AM +0200, Rasmus Villemoes wrote: > On Sat, Oct 03 2015, Christoph Hellwig wrote: > > > Hi Rasmus, > > > > I like this idea. But maybe it's also time to just move the constants > > to a plain text file and auto-generate C headers from them?

Re: [PATCH 2/3] hwrng: stm32 - add support for STM32 HW RNG

Hi Daniel, On 10/03/2015 10:35 PM, Daniel Thompson wrote: Add support for STMicroelectronics STM32 random number generator. The config value defaults to N, reflecting the fact that STM32 is a very low resource microcontroller platform and unlikely to be targeted by any "grown up" defconfigs.

Re: [PATCH v8 0/3] usb: xhci-platform: Configure 64-bit DMA mask if the platform is capable

On Wed, Sep 30, 2015 at 02:24:30PM -0700, Duc Dang wrote: > On Thu, Sep 17, 2015 at 11:19 AM, Duc Dang wrote: > > The xhci platform driver does not work with system that only supports > > 64-bit DMA as it requests 32-bit DMA mask during driver initialization. > > This patch set

Re: [PATCH 3/3] ARM: dts: stm32f429: Adopt STM32 RNG driver

Hi Daniel, On 10/03/2015 10:35 PM, Daniel Thompson wrote: New bindings and driver have been created for STM32 series parts. This patch integrates this changes. Signed-off-by: Daniel Thompson --- arch/arm/boot/dts/stm32f429.dtsi | 7 +++ 1 file changed, 7

Re: [PATCH v2 3/3] mfd: add CSR SiRFSoC on-chip power management module driver

2015-09-29 16:55 GMT+08:00 Lee Jones : > On Tue, 29 Sep 2015, Barry Song wrote: > >> 2015-09-29 15:16 GMT+08:00 Lee Jones : >> > On Tue, 29 Sep 2015, Barry Song wrote: >> >> >> >> +static int sirfsoc_pwrc_probe(struct platform_device *pdev) >> >> >> >>

[PATCH v2] video: fbdev: add Marvell PXA framebuffer binding

Add documentation for the PXA frambuffer devicetree binding. Signed-off-by: Robert Jarzmik --- Since v1: Philipp's review on the whole binding --- .../devicetree/bindings/video/marvell,pxafb.txt| 80 ++ 1 file changed, 80 insertions(+) create

Re: [patch 1/2] x86/process: Add proper bound checks in 64bit get_wchan()

On Sat, Oct 3, 2015 at 1:31 PM, Andrey Ryabinin wrote: > 2015-10-03 13:54 GMT+03:00 Thomas Gleixner : >> On Fri, 2 Oct 2015, Sasha Levin wrote: >>> I'm seeing a different issue with this patch: >>> >>> [ 5228.736320] BUG: KASAN: out-of-bounds in

[PATCH 2/3] sched/core: drop unlikely behind BUG_ON()

BUG_ON() already contain an unlikely compiler flag. Drop it. Signed-off-by: Geliang Tang --- kernel/sched/core.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/kernel/sched/core.c b/kernel/sched/core.c index fe81929..a3518789 100644 ---

[PATCH 1/3] scsi: drop unlikely behind BUG_ON()

BUG_ON() already contain an unlikely compiler flag. Drop it. Signed-off-by: Geliang Tang --- drivers/scsi/scsi_lib.c | 4 ++-- 1 file changed, 2 insertions(+), 2 deletions(-) diff --git a/drivers/scsi/scsi_lib.c b/drivers/scsi/scsi_lib.c index f570b48..3b5faab 100644 ---

[PATCH 3/3] mm/nommu: drop unlikely behind BUG_ON()

BUG_ON() already contain an unlikely compiler flag. Drop it. Signed-off-by: Geliang Tang --- mm/nommu.c | 10 +- 1 file changed, 5 insertions(+), 5 deletions(-) diff --git a/mm/nommu.c b/mm/nommu.c index 1e0f168..92be862 100644 --- a/mm/nommu.c +++ b/mm/nommu.c @@

Re: [PATCH v2 6/7] squashfs: xattr simplifications

On Tue, Sep 22, 2015 at 02:26:51PM +0200, Andreas Gruenbacher wrote: > Now that the xattr handler is passed to the xattr handler operations, we > have access to the attribute name prefix, so simplify the squashfs xattr > handlers a bit. > > Signed-off-by: Andreas Gruenbacher

Re: [PATCH] staging: rdma: Fix braces around if/else

On 10/3/2015 11:55 PM, Martin Kletzander wrote: Get rid of all ELSE_AFTER_BRACE type errors reported by checkpatch.pl. Hi Greg, Is there a way to signal people/tools that a certain driver parks in staging on their way **out** of the kernel and not the other way around? I guess you (nor Doug)

  1   2   3   4   5   6   >