Re: [PATCH v2] hv: hv_fcopy: drop the obsolete message on transfer failure

2014-11-26 Thread Jason Wang
- Original Message - > In the case the user-space daemon crashes, hangs or is killed, we > need to down the semaphore, otherwise, after the daemon starts next > time, the obsolete data in fcopy_transaction.message or > fcopy_transaction.fcopy_msg will be used immediately. > > Reviewed-by

RE: [PATCH] hv: hv_fcopy: drop the obsolete message on transfer failure

2014-11-26 Thread Dexuan Cui
> -Original Message- > From: Greg KH [mailto:gre...@linuxfoundation.org] > Sent: Thursday, November 27, 2014 7:54 AM > To: Dexuan Cui > Cc: linux-ker...@vger.kernel.org; driverdev-devel@linuxdriverproject.org; > o...@aepfle.de; a...@canonical.com; jasow...@redhat.com; KY Srinivasan; > Haiya

[PATCH v2] hv: hv_fcopy: drop the obsolete message on transfer failure

2014-11-26 Thread Dexuan Cui
In the case the user-space daemon crashes, hangs or is killed, we need to down the semaphore, otherwise, after the daemon starts next time, the obsolete data in fcopy_transaction.message or fcopy_transaction.fcopy_msg will be used immediately. Reviewed-by: Vitaly Kuznetsov Cc: K. Y. Srinivasan S

Re: [PATCH] staging: lustre: fix sparse warnings related to lock context imbalance

2014-11-26 Thread Loïc Pefferkorn
On Wed, Nov 26, 2014 at 12:54:43PM -0800, Greg KH wrote: > > Ugh, how horrid, please just delete these functions and push down the > spin_lock/unlock calls down into the places these are called. > > Same for these. > > Same thing here. Hello Greg, Thanks for your comments, I will write a v2. -

RE: [PATCH] Drivers: hv: vmbus: prevent cpu offlining on newer hypervisors

2014-11-26 Thread Dexuan Cui
> -Original Message- > From: devel [mailto:driverdev-devel-boun...@linuxdriverproject.org] On > Behalf Of Greg Kroah-Hartman > Sent: Thursday, November 27, 2014 11:03 AM > To: Vitaly Kuznetsov > Cc: de...@linuxdriverproject.org; Haiyang Zhang; linux- > ker...@vger.kernel.org > Subject: Re:

Re: [PATCH] Drivers: hv: vmbus: prevent cpu offlining on newer hypervisors

2014-11-26 Thread Greg Kroah-Hartman
On Wed, Nov 26, 2014 at 02:52:22PM +0100, Vitaly Kuznetsov wrote: > When an SMP Hyper-V guest is running on top of 2012R2 Server and secondary > cpus are sent offline (with echo 0 > /sys/devices/system/cpu/cpu$cpu/online) > the system freeze is observed. This happens due to the fact that on newer >

[PATCH 2/2] mmc: rtsx: add support for sdio card

2014-11-26 Thread micky_ching
From: Micky Ching Add support for sdio card by SD interface. The main change is data transfer mode, When read data, host wait data transfer while command start. When write data, host will start data transfer after command get response. The transfer mode modify can be applied both for SD/MMC card

[PATCH 1/2] mfd: rtsx: add func to split u32 into register

2014-11-26 Thread micky_ching
From: Micky Ching Add helper function to write u32 to registers, if we want to put u32 value to 4 continuous register, this can help us reduce tedious work. Signed-off-by: Micky Ching --- include/linux/mfd/rtsx_pci.h | 15 +++ 1 file changed, 15 insertions(+) diff --git a/include/

[PATCH 0/2] mmc: rtsx: add support for sdio card

2014-11-26 Thread micky_ching
From: Micky Ching This patch is used to change transfer mode for sdio card support by SD interface. Micky Ching (2): mfd: rtsx: add func to split u32 into register mmc: rtsx: add support for sdio card drivers/mmc/host/rtsx_pci_sdmmc.c | 366 ++ include/l

Re: [PATCH] staging: octeon: Fix checkpatch warning

2014-11-26 Thread Greg KH
On Thu, Nov 27, 2014 at 12:35:23AM +, Luis de Bethencourt wrote: > On Wed, Nov 26, 2014 at 01:45:23PM -0800, Greg KH wrote: > > On Tue, Nov 25, 2014 at 01:26:14PM +, Luis de Bethencourt wrote: > > > This patch fixes the checkpatch.pl warnings: > > > > > > WARNING: line over 80 characters >

Re: [PATCH] hv: hv_fcopy: drop the obsolete message on transfer failure

2014-11-26 Thread Greg KH
On Tue, Nov 11, 2014 at 09:03:26PM -0800, Dexuan Cui wrote: > In the case the user-space daemon crashes, hangs or is killed, we > need to down the semaphore, otherwise, after the daemon starts next > time, the obsolete data in fcopy_transaction.message or > fcopy_transaction.fcopy_msg will be used

Re: [PATCH] Drivers:staging:comedi:drivers: Fixed checkpatch warning

2014-11-26 Thread Greg KH
On Wed, Nov 26, 2014 at 04:56:55PM +0530, Athira Lekshmi C V wrote: > Fixed the checkpatch warning: > WARNING: please, no space before tabs > > Signed-off-by: Athira Lekshmi C V Same question about the name here as before. Also, be more specific in your subject as to what you are changing. "fix

Re: [PATCH 3/3] rtl8188eu: Remove leading spaces in rtw_led.c

2014-11-26 Thread Greg Kroah-Hartman
On Fri, Nov 07, 2014 at 12:24:28AM +0100, Krzysztof Konopko wrote: > According to Linux coding convention leading spaces are not allowed. This > patch removes leading spaces in rtw_led.c > > Signed-off-by: Krzysztof Konopko > --- > drivers/staging/rtl8188eu/core/rtw_led.c | 5 +++-- > 1 file ch

Re: [PATCH] staging: media: lirc: lirc_zilog.c: fix quoted strings split across lines

2014-11-26 Thread Luis de Bethencourt
On Wed, Nov 26, 2014 at 08:05:55AM -0800, Joe Perches wrote: > On Wed, 2014-11-26 at 15:42 +, Luis de Bethencourt wrote: > > On 26 November 2014 at 01:49, Joe Perches wrote: > [] > > > There is a script I posted a while back that > > > groups various checkpatch "types" together and > > > makes

Re: [PATCH] staging: octeon: Fix checkpatch warning

2014-11-26 Thread Luis de Bethencourt
On Wed, Nov 26, 2014 at 01:45:23PM -0800, Greg KH wrote: > On Tue, Nov 25, 2014 at 01:26:14PM +, Luis de Bethencourt wrote: > > This patch fixes the checkpatch.pl warnings: > > > > WARNING: line over 80 characters > > + int cores_in_use = core_state.baseline_cores - > >

Re: [PATCH 1/2] iio: Add support for waveform output

2014-11-26 Thread Lars-Peter Clausen
On 11/26/2014 10:45 PM, George McCollister wrote: Output can be held high or low for a specified period of time. Support for waveform capture could be added in the future. That's a PWM device? ___ devel mailing list de...@linuxdriverproject.org http:

Re: [PATCH] Drivers:staging:wlan-ng: Fixed checkpatch warning

2014-11-26 Thread Greg KH
On Tue, Nov 25, 2014 at 05:16:40PM +0530, Athira Lekshmi C V wrote: > The following checkpatch warning was fixed > WARNING: line over 80 characters > > Signed-off-by: Athira Lekshmi C V > --- > drivers/staging/wlan-ng/prism2fw.c | 10 +++--- > 1 file changed, 7 insertions(+), 3 deletions(-

Re: [PATCH] Drivers:Staging:rtl8188eu:hal:usb_halinit.c: Added blank lines after declarations

2014-11-26 Thread Greg KH
On Sat, Nov 22, 2014 at 11:36:40AM +0530, Anjana Sasindran wrote: > This patch fixes the five checkpatch.pl warnings: > > WARNING:Missing a blank line after declaration > > Signed-off-by: Anjana Sasindran > --- > drivers/staging/rtl8188eu/hal/usb_halinit.c | 6 ++ > 1 file changed,

Re: [PATCH] Staging: rtl8188eu: core: rtw_wlan_util: replaced __inline with inline

2014-11-26 Thread Greg KH
On Thu, Nov 20, 2014 at 04:03:23PM +0100, Abel Moyo wrote: > Changed return type of function get_my_bssid from __inline to inline > > Signed-off-by: Abel Moyo > --- > drivers/staging/rtl8188eu/core/rtw_wlan_util.c | 2 +- > 1 file changed, 1 insertion(+), 1 deletion(-) > > diff --git a/drivers/

Re: [PATCH v2 4/9] staging: panel: Use defined value or checking module params state

2014-11-26 Thread Greg Kroah-Hartman
On Wed, Nov 19, 2014 at 09:38:46PM +0100, Mariusz Gorski wrote: > Avoid magic number and use a comparison with a defined value instead > that checks whether module param has been set by the user to some > value at loading time. > > Signed-off-by: Mariusz Gorski > Acked-by: Willy Tarreau > --- >

[PATCH 1/2] iio: Add support for waveform output

2014-11-26 Thread George McCollister
Output can be held high or low for a specified period of time. Support for waveform capture could be added in the future. Signed-off-by: George McCollister --- Documentation/ABI/testing/sysfs-bus-iio | 7 +++ drivers/iio/industrialio-core.c | 3 +++ driver

Re: [PATCH] staging: octeon: Fix checkpatch warning

2014-11-26 Thread Greg KH
On Tue, Nov 25, 2014 at 01:26:14PM +, Luis de Bethencourt wrote: > This patch fixes the checkpatch.pl warnings: > > WARNING: line over 80 characters > + int cores_in_use = core_state.baseline_cores - > atomic_read(&core_state.available_cores); > > WARNING: line over 80

Re: [PATCH] Drivers:staging:octeon: Fixed checkpatch warning

2014-11-26 Thread Greg KH
On Wed, Nov 26, 2014 at 05:58:37PM +0530, Athira Lekshmi C V wrote: > Fixed the checkpatch warning: > WARNING: Missing a blank line after declarations > > Signed-off-by: Athira Lekshmi C V What is the "C V" at the end of the name here? Is that your "full name"? I need a real name, not initials

Re: [PATCH] Drivers:Staging:octeon: Fixed missing a blank line warning

2014-11-26 Thread Greg KH
On Thu, Nov 20, 2014 at 11:31:14PM +0530, Lekshmi wrote: > The following checkpatch warning was fixed: > WARNING: Missing a blank line after declarations > > Signed-off-by: Lekshmi I need a "full name" here, sorry. Please fix and resend. greg k-h ___

Re: [PATCH 2/2] staging: android: ion: One function call less in ion_buffer_create() after error detection

2014-11-26 Thread Greg Kroah-Hartman
On Sun, Nov 23, 2014 at 07:44:08PM +0100, SF Markus Elfring wrote: > From: Markus Elfring > Date: Sun, 23 Nov 2014 19:12:29 +0100 > > The jump label "err1" was used by the ion_buffer_create() function in case of > a memory allocation failure just to pass a null pointer to a vfree() function > cal

Re: [PATCH] staging: unisys: get rid of channel stub

2014-11-26 Thread Greg KH
I _strongly_ suggest unisys get their act together here. The number of patches that were sent recently that make no sense at all is horrid. I've now purged all of them from my to-apply queue, please resend _everything_ that you have, in the correct order, _AFTER_ you all review them among yoursel

Re: [PATCH 11/11] staging: unisys: small bug parser.c

2014-11-26 Thread Greg KH
On Thu, Nov 13, 2014 at 09:56:39AM -0500, Jeffrey Brown wrote: > Fixed small bug in parser.c by removing "cleanups:" in parser_init > _guts struct. Replaced it with proper error handling code > and removed the instances of rc = NULL in the code. rc = NULL is > redudant > > Signed-off-by: Jeffrey

Re: [PATCH 00/10] parser.h and parser.c patches

2014-11-26 Thread Greg KH
On Wed, Nov 12, 2014 at 11:28:15AM -0500, Jeffrey Brown wrote: > Sorry for all of the messy patches from before but here > are the improved patches for parser.h and parser.c. > The camel cases are grouped on functions and structs > that occurs in both files. You have to do a 'vN' where 'N' is a nu

Re: [PATCH] staging: unisys: Remove chanstub files

2014-11-26 Thread Greg KH
On Fri, Nov 07, 2014 at 10:17:40AM -0800, Greg KH wrote: > On Fri, Nov 07, 2014 at 12:58:24PM -0500, Ken Depro wrote: > > While fixing the CamelCase checks for the functions in the chanstub > > source and header files, I discovered they are no longer being used. > > This patch removes the two ch

Re: [PATCH] staging: lustre: fix sparse warnings related to lock context imbalance

2014-11-26 Thread Greg KH
On Wed, Nov 26, 2014 at 05:15:48PM +0100, Loic Pefferkorn wrote: > Add __acquires() and __releases() function annotations, to fix sparse > warnings related to lock context imbalance. > > This fixes the following warnings: > > drivers/staging/lustre/lustre/libcfs/linux/linux-tracefile.c:153:5: wa

Re: [PATCH 0/2] fix some sparse warnings in lustre

2014-11-26 Thread Greg KH
On Mon, Nov 24, 2014 at 07:55:40PM +0100, Zahari Doychev wrote: > The two patches fix several sparse warning in the lustre module. > > Zahari Doychev (2): > [drivers] staging/lustre: fix sparse warnings > [drivers] staging/lustre: fix sparse warnings > > drivers/staging/lustre/include/linux/

Re: [PATCH 10/10] staging: lustre: ldlm: Add a space in debug output

2014-11-26 Thread Greg Kroah-Hartman
On Sun, Nov 23, 2014 at 02:37:57PM +0100, Andreas Ruprecht wrote: > A space which was forgotten in a CDEBUG output is added by this patch. > > Signed-off-by: Andreas Ruprecht > --- > drivers/staging/lustre/lustre/ldlm/ldlm_request.c | 2 +- > 1 file changed, 1 insertion(+), 1 deletion(-) > > di

Re: [PATCH] staging:rtl8723au: core: fix checkpatch error: that open brace { should be on the previous line

2014-11-26 Thread Greg KH
On Tue, Nov 11, 2014 at 03:28:57PM +0530, Sanjeev Sharma wrote: > This is a patch to the rtw_cmd.c file that fixes following > Error. > > ERROR: that open brace { should be on the previous line This patch does much more than that :( ___ devel mailing li

atualização sua conta

2014-11-26 Thread Administrador Centro
Uma tentativa foi feita para sua conta a partir de um computador desconhecido. para a segurança da sua conta, estamos prontos para abrir uma consulta ou atualização sua conta. Por favor, preencha os dados abaixo: nome: Endereço de email: password: Re-digite a senha: Tel: Te agradece

Re: [PATCH v2 1/2] drm: imx: Move imx-drm driver out of staging

2014-11-26 Thread Philipp Zabel
Am Mittwoch, den 26.11.2014, 09:43 +1000 schrieb Dave Airlie: > On 25 November 2014 at 01:33, Philipp Zabel wrote: > > The imx-drm driver was put into staging mostly for the following reasons, > > all of which have been addressed or superseded: > > - convert the irq driver to use linear irq domai

Re: [PATCH v13 07/12] drm: bridge/dw_hdmi: add support for multi-byte register width access

2014-11-26 Thread Philipp Zabel
Am Mittwoch, den 26.11.2014, 21:32 +0800 schrieb Andy Yan: > On rockchip rk3288, only word(32-bit) accesses are > permitted for hdmi registers. Byte width accesses (writeb, > readb) generate an imprecise external abort. > > Signed-off-by: Andy Yan > > --- > > Changes in v13: None > Changes in v

Re: [PATCH v13 08/12] drm: bridge/dw_hdmi: add mode_valid support

2014-11-26 Thread Philipp Zabel
Am Mittwoch, den 26.11.2014, 21:33 +0800 schrieb Andy Yan: > some platform may not support all the display mode, > add mode_valid interface check it > > also add drm_connector_register which add a debugfs > interface for dump display modes and edid information > > Signed-off-by: Andy Yan > --- >

[PATCH] staging: lustre: fix sparse warnings related to lock context imbalance

2014-11-26 Thread Loic Pefferkorn
Add __acquires() and __releases() function annotations, to fix sparse warnings related to lock context imbalance. This fixes the following warnings: drivers/staging/lustre/lustre/libcfs/linux/linux-tracefile.c:153:5: warning: context imbalance in 'cfs_trace_lock_tcd' - wrong count at exit drive

Re: [PATCH v13 0/12] dw-hdmi: convert imx hdmi to bridge/dw_hdmi

2014-11-26 Thread Philipp Zabel
Hi Andy, I have yet to look at this in more detail, but from a quick test starting with patch 3, the HDMI display stays black on Nitrogen6X, and starting with patch 8 I get the following error. imx-drm display-subsystem: [CONNECTOR:21:HDMI-A-1] drm_connector_register failed: -12 [ cu

Re: [PATCH] staging: media: lirc: lirc_zilog.c: fix quoted strings split across lines

2014-11-26 Thread Joe Perches
On Wed, 2014-11-26 at 15:42 +, Luis de Bethencourt wrote: > On 26 November 2014 at 01:49, Joe Perches wrote: [] > > There is a script I posted a while back that > > groups various checkpatch "types" together and > > makes it a bit easier to do cleanup style > > patches. > > > > https://lkml.or

[PATCH] Drivers: hv: vmbus: prevent cpu offlining on newer hypervisors

2014-11-26 Thread Vitaly Kuznetsov
When an SMP Hyper-V guest is running on top of 2012R2 Server and secondary cpus are sent offline (with echo 0 > /sys/devices/system/cpu/cpu$cpu/online) the system freeze is observed. This happens due to the fact that on newer hypervisors (Win8, WS2012R2, ...) vmbus channel handlers are distributed

[PATCH v13 12/12] drm: bridge/dw_hdmi: add rockchip rk3288 support

2014-11-26 Thread Andy Yan
Rockchip RK3288 hdmi is compatible with dw_hdmi this patch is depend on patch by Mark Yao Add drm driver for Rockchip Socs see https://lkml.org/lkml/2014/11/19/1153 Signed-off-by: Andy Yan --- Changes in v13: None Changes in v12: - add comment for the depend on patch Changes in v11: None Cha

[PATCH v13 11/12] dt-bindings: Add documentation for rockchip dw hdmi

2014-11-26 Thread Andy Yan
Signed-off-by: Andy Yan --- Changes in v13: None Changes in v12: None Changes in v11: None Changes in v10: None Changes in v9: None Changes in v8: - Add documentation for rockchip dw hdmi Changes in v7: None Changes in v6: None Changes in v5: None Changes in v4: None Changes in v3: None .../d

[PATCH v13 10/12] drm: bridge/dw_hdmi: add function dw_hdmi_phy_enable_spare

2014-11-26 Thread Andy Yan
RK3288 HDMI will not work without the spare bit of HDMI_PHY_CONF0 enable Signed-off-by: Andy Yan --- Changes in v13: None Changes in v12: None Changes in v11: None Changes in v10: None Changes in v9: None Changes in v8: None Changes in v7: None Changes in v6: None Changes in v5: None Changes in

[PATCH v13 09/12] drm: bridge/dw_hdmi: clear i2cmphy_stat0 reg in hdmi_phy_wait_i2c_done

2014-11-26 Thread Andy Yan
HDMI_IH_I2CMPHY_STAT0 is a clear on write register, which indicates i2cm operation status(i2c transfer done or error), every hdmi phy register configuration must check this register to make sure the configuration has complete. But the indication bit should be cleared after check, otherwise the corr

[PATCH v13 08/12] drm: bridge/dw_hdmi: add mode_valid support

2014-11-26 Thread Andy Yan
some platform may not support all the display mode, add mode_valid interface check it also add drm_connector_register which add a debugfs interface for dump display modes and edid information Signed-off-by: Andy Yan --- Changes in v13: None Changes in v12: None Changes in v11: None Changes in v

[PATCH v13 07/12] drm: bridge/dw_hdmi: add support for multi-byte register width access

2014-11-26 Thread Andy Yan
On rockchip rk3288, only word(32-bit) accesses are permitted for hdmi registers. Byte width accesses (writeb, readb) generate an imprecise external abort. Signed-off-by: Andy Yan --- Changes in v13: None Changes in v12: None Changes in v11: None Changes in v10: None Changes in v9: None Changes

[PATCH v13 06/12] dt-bindings: add document for dw_hdmi

2014-11-26 Thread Andy Yan
Signed-off-by: Andy Yan --- Changes in v13: None Changes in v12: None Changes in v11: None Changes in v10: None Changes in v9: None Changes in v8: - correct some spelling mistake - modify ddc-i2c-bus and interrupt description Changes in v7: None Changes in v6: None Changes in v5: None Changes i

[PATCH v13 05/12] drm: imx: imx-hdmi: move imx-hdmi to bridge/dw_hdmi

2014-11-26 Thread Andy Yan
the original imx hdmi driver is under drm/imx/, which depends on imx-drm, so move the imx hdmi driver out to drm/bridge and rename it to dw_hdmi Signed-off-by: Andy Yan --- Changes in v13: None Changes in v12: None Changes in v11: None Changes in v10: None Changes in v9: None Changes in v8: None

[PATCH v13 04/12] drm: imx: imx-hdmi: split phy configuration to platform driver

2014-11-26 Thread Andy Yan
hdmi phy configuration is platform specific, which can be adusted according to the board to get the best SI Signed-off-by: Andy Yan --- Changes in v13: - split phy configuration from patch#4 Changes in v12: None Changes in v11: None Changes in v10: None Changes in v9: None Changes in v8: None

[PATCH v13 03/12] drm: imx: imx-hdmi: convert imx-hdmi to drm_bridge mode

2014-11-26 Thread Andy Yan
IMX6 and Rockchip RK3288 and JZ4780 (Ingenic Xburst/MIPS) use the interface compatible Designware HDMI IP, but they also have some lightly differences, such as phy pll configuration, register width, 4K support, clk useage, and the crtc mux configuration is also platform specific. To reuse the imx

[PATCH v13 02/12] drm: imx: imx-hdmi: return defer if can't get ddc i2c adapter

2014-11-26 Thread Andy Yan
drm driver may probe before the i2c bus, so the driver should defer probing until it is available Signed-off-by: Andy Yan Reviewed-by: Daniel Kurtz --- Changes in v13: None Changes in v12: - refactor of_node_put(ddc_node) Changes in v11: None Changes in v10: None Changes in v9: None Changes i

[PATCH v13 01/12] drm: imx: imx-hdmi: make checkpatch happy

2014-11-26 Thread Andy Yan
CHECK: Alignment should match open parenthesis + if ((hdmi->vic == 10) || (hdmi->vic == 11) || + (hdmi->vic == 12) || (hdmi->vic == 13) || CHECK: braces {} should be used on all arms of this statement + if (hdmi->hdmi_data.video_mode.mdvi) [...] + else { [...] Sign

[PATCH v13 0/12] dw-hdmi: convert imx hdmi to bridge/dw_hdmi

2014-11-26 Thread Andy Yan
We found Freescale imx6 and Rockchip rk3288 and Ingenic JZ4780 (Xburst/MIPS) use the interface compatible Designware HDMI IP, but they also have some lightly differences, such as phy pll configuration, register width(imx hdmi register is one byte, but rk3288 is 4 bytes width and can only be access

Re: [PATCH] staging: lustre: fix pointer declarations

2014-11-26 Thread Al Viro
On Tue, Nov 25, 2014 at 09:44:21PM +0100, Zahari Doychev wrote: > This patch fixes pointer declarations from void * to void __user * in order > to remove some sparse warnings. _Are_ those userland addresses, though? Quick grep shows that in the only caller of lnet_copy_iov2flat() we have somethin

[PATCH] Drivers:staging:octeon: Fixed checkpatch warning

2014-11-26 Thread Athira Lekshmi C V
Fixed the checkpatch warning: WARNING: Missing a blank line after declarations Signed-off-by: Athira Lekshmi C V --- drivers/staging/octeon/ethernet-rx.c |2 ++ 1 file changed, 2 insertions(+) diff --git a/drivers/staging/octeon/ethernet-rx.c b/drivers/staging/octeon/ethernet-rx.c index 44

Re: [PATCH] Drivers:staging:octeon: Fixed checkpatch warning

2014-11-26 Thread Dan Carpenter
On Wed, Nov 26, 2014 at 05:17:48PM +0530, Athira Lekshmi C V wrote: > @@ -294,8 +297,10 @@ static int cvm_oct_napi_poll(struct napi_struct *napi, > int budget) >* CPU. >*/ > union cvmx_pow_wq_int_cntx counts; > + Don't put a bl

[PATCH] Drivers:staging:octeon: Fixed checkpatch warning

2014-11-26 Thread Athira Lekshmi C V
Fixed the checkpatch warning: WARNING: Missing a blank line after declarations Signed-off-by: Athira Lekshmi C V --- drivers/staging/octeon/ethernet-rx.c |5 + 1 file changed, 5 insertions(+) diff --git a/drivers/staging/octeon/ethernet-rx.c b/drivers/staging/octeon/ethernet-rx.c index

[PATCH] Drivers:staging:comedi:drivers: Fixed checkpatch warning

2014-11-26 Thread Athira Lekshmi C V
Fixed the checkpatch warning: WARNING: please, no space before tabs Signed-off-by: Athira Lekshmi C V --- drivers/staging/comedi/drivers/ni_at_a2150.c |2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/drivers/staging/comedi/drivers/ni_at_a2150.c b/drivers/staging/comedi/dri

Re: [PATCH] Drivers:staging:speakup: Fixed checkpatch warning

2014-11-26 Thread Richard Weinberger
On Wed, Nov 26, 2014 at 12:14 PM, Athira Lekshmi C V wrote: > Fixed the checkpatch warning: > WARNING: simple_strtoul is obsolete, use kstrtoul instead > > Signed-off-by: Athira Lekshmi C V > --- > drivers/staging/speakup/varhandlers.c |2 +- > 1 file changed, 1 insertion(+), 1 deletion(-) >

[PATCH] Drivers:staging:speakup: Fixed checkpatch warning

2014-11-26 Thread Athira Lekshmi C V
Fixed the checkpatch warning: WARNING: simple_strtoul is obsolete, use kstrtoul instead Signed-off-by: Athira Lekshmi C V --- drivers/staging/speakup/varhandlers.c |2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/drivers/staging/speakup/varhandlers.c b/drivers/staging/spea

Re: [PATCH] staging: media: lirc: lirc_zilog.c: fix quoted strings split across lines

2014-11-26 Thread Joe Perches
On Tue, 2014-11-25 at 21:14 +, Luis de Bethencourt wrote: > On Tue, Nov 25, 2014 at 01:00:07PM -0800, Joe Perches wrote: > > In the future, you might consider being more > > comprehensive with your patches. > > Wasn't sure about the scope of the style fixing > patches. I've been reading Kernel