[PATCH V2] staging: wilc1000: wilc_msgqueue.c : remove the goto ERRORHANDER

2015-10-13 Thread Tony Cho
From: Leo Kim This patch removes goto ERRORHANDER and the result variable in wilc_mq_send. Then, the error type is directly returned. If normal operation, freeing memory is not needed in this function. If an error occurs, returns an error after releasing the spin lock.

Re: [PATCH 54/54] staging: wilc1000: wilc_msgqueue.c : remove the goto ERRORHANDER

2015-10-13 Thread Tony Cho
On 2015년 10월 13일 23:08, Mike Rapoport wrote: On Tue, Oct 13, 2015 at 08:02:12PM +0900, Tony Cho wrote: From: Leo Kim This patch removes goto ERRORHANDER and the result variable in wilc_mq_send. Then, the error type is directly returned. If normal operation, freeing memory

Re: [RESEND] mfd: rtsx: add support for rts522A

2015-10-13 Thread Lee Jones
On Sat, 10 Oct 2015, 敬锐 wrote: > Sorry for bother you, but I still can't see this patch applied. > Is there something wrong? Sorry about that Micky, I guess this one fell through the gaps. Re-applied, should be in -next by tomorrow. > On 07/08/2015 03:38 PM, Lee Jones wrote: > > On Wed, 08 Jul

Re: [PATCH v3] mtd: nand: pass page number to ecc->write_xxx() methods

2015-10-13 Thread Boris Brezillon
Hi Brian, On Mon, 12 Oct 2015 16:35:37 -0700 Brian Norris wrote: > On Fri, Oct 02, 2015 at 07:53:25AM +0200, Boris Brezillon wrote: > > The ->read_xxx() methods are all passed the page number the NAND controller > > is supposed to read, but ->write_xxx() do not have

Re: [RFC][PATCH 1/2] WIP: Devicetree bindings for Ion

2015-10-13 Thread Andrew
On 2015-10-12 21:39, Mitchel Humpherys wrote: On Tue, Oct 06 2015 at 05:35:41 PM, Rob Herring wrote: On Tue, Oct 6, 2015 at 3:47 PM, Laura Abbott wrote: [...] +Example: + + ion { + compatbile = "linux,ion"; +

[PATCH v4] mtd: nand: pass page number to ecc->write_xxx() methods

2015-10-13 Thread Boris Brezillon
The ->read_xxx() methods are all passed the page number the NAND controller is supposed to read, but ->write_xxx() do not have such a parameter. This is a problem if we want to properly implement data scrambling/randomization in order to mitigate MLC sensibility to repeated pattern: to prevent

[PATCH 17/54] staging: wilc1000: rename pu8IEs of struct scan_attr

2015-10-13 Thread Tony Cho
From: Leo Kim This patch renames pu8IEs of struct scan_attr to ies to avoid CamelCase naming convention. Signed-off-by: Leo Kim Signed-off-by: Tony Cho --- drivers/staging/wilc1000/host_interface.c | 15 +++ 1 file

[PATCH 13/54] staging: wilc1000: rename u8ScanType of struct scan_attr

2015-10-13 Thread Tony Cho
From: Leo Kim This patch renames u8ScanType of struct scan_attr to type to avoid CamelCase naming convention. Signed-off-by: Leo Kim Signed-off-by: Tony Cho --- drivers/staging/wilc1000/host_interface.c | 6 +++--- 1 file changed, 3

[PATCH 26/54] staging: wilc1000: rename ssidLen of struct connect_attr

2015-10-13 Thread Tony Cho
From: Leo Kim This patch renames ssidLen of struct connect_attr to ssid_len to avoid CamelCase naming convention. Signed-off-by: Leo Kim Signed-off-by: Tony Cho --- drivers/staging/wilc1000/host_interface.c | 16 1

[PATCH 25/54] staging: wilc1000: rename pu8ssid of struct connect_attr

2015-10-13 Thread Tony Cho
From: Leo Kim This patch renames pu8ssid of struct connect_attr to ssid to avoid CamelCase naming convention. Signed-off-by: Leo Kim Signed-off-by: Tony Cho --- drivers/staging/wilc1000/host_interface.c | 32

[PATCH 30/54] staging: wilc1000: rename pfConnectResult of struct connect_attr

2015-10-13 Thread Tony Cho
From: Leo Kim This patch renames pfConnectResult of struct connect_attr to result to avoid CamelCase naming convention. Signed-off-by: Leo Kim Signed-off-by: Tony Cho --- drivers/staging/wilc1000/host_interface.c | 10 +- 1

[PATCH 22/54] staging: wilc1000: fix "ERROR: do not initialise globals to 0 or NULL"

2015-10-13 Thread Tony Cho
From: Leo Kim This patch fixes the checkpatch.pl error to host_interface.c. - ERROR: do not initialise globals to 0 or NULL Signed-off-by: Leo Kim Signed-off-by: Tony Cho --- drivers/staging/wilc1000/host_interface.c | 4 ++-- 1 file

[PATCH 32/54] staging: wilc1000: rename tenuAuth_type of struct connect_attr

2015-10-13 Thread Tony Cho
From: Leo Kim This patch renames tenuAuth_type of struct connect_attr to auth_type to avoid CamelCase naming convention. Signed-off-by: Leo Kim Signed-off-by: Tony Cho --- drivers/staging/wilc1000/host_interface.c | 6 +++--- 1 file

[PATCH 29/54] staging: wilc1000: rename u8security of struct connect_attr

2015-10-13 Thread Tony Cho
From: Leo Kim This patch renames u8security of struct connect_attr to security to avoid CamelCase naming convention. Signed-off-by: Leo Kim Signed-off-by: Tony Cho --- drivers/staging/wilc1000/host_interface.c | 6 +++--- 1 file

[PATCH 27/54] staging: wilc1000: rename pu8IEs of struct connect_attr

2015-10-13 Thread Tony Cho
From: Leo Kim This patch renames pu8IEs of struct connect_attr to ies to avoid CamelCase naming convention. Signed-off-by: Leo Kim Signed-off-by: Tony Cho --- drivers/staging/wilc1000/host_interface.c | 21 ++--- 1

[PATCH 28/54] staging: wilc1000: rename IEsLen of struct connect_attr

2015-10-13 Thread Tony Cho
From: Leo Kim This patch renames IEsLen of struct connect_attr to ies_len to avoid CamelCase naming convention. Signed-off-by: Leo Kim Signed-off-by: Tony Cho --- drivers/staging/wilc1000/host_interface.c | 16 1 file

[PATCH 23/54] staging: wilc1000: remove duplicate copy routine

2015-10-13 Thread Tony Cho
From: Leo Kim This patch remove the duplicate bssid copy routine. Already ahead bssid copy routine execute. Therefore do not necessary in this routine. Signed-off-by: Leo Kim Signed-off-by: Tony Cho ---

[PATCH 24/54] staging: wilc1000: rename pu8bssid of struct connect_attr

2015-10-13 Thread Tony Cho
From: Leo Kim This patch renames pu8bssid of struct connect_attr to bssid to avoid CamelCase naming convention. Signed-off-by: Leo Kim Signed-off-by: Tony Cho --- drivers/staging/wilc1000/host_interface.c | 33

[PATCH 37/54] staging: wilc1000: rename u8SetChan of struct channel_attr

2015-10-13 Thread Tony Cho
From: Leo Kim This patch renames u8SetChan of struct channel_attr to set_ch to avoid CamelCase naming convention. Signed-off-by: Leo Kim Signed-off-by: Tony Cho --- drivers/staging/wilc1000/host_interface.c | 6 +++--- 1 file changed,

[PATCH 45/54] staging: wilc1000: rename u8Num_AssocSta of struct del_all_sta

2015-10-13 Thread Tony Cho
From: Leo Kim This patch renames u8Num_AssocSta of struct del_all_sta to assoc_sta to avoid CamelCase naming convention. Signed-off-by: Leo Kim Signed-off-by: Tony Cho --- drivers/staging/wilc1000/host_interface.c | 10 +- 1

[PATCH 02/54] staging: wilc1000: rename enuKeyType of struct key_attr

2015-10-13 Thread Tony Cho
From: Leo Kim This patch renames enuKeyType of struct key_attr to type to avoid CamelCase naming convention. Signed-off-by: Leo Kim Signed-off-by: Tony Cho --- drivers/staging/wilc1000/host_interface.c | 18 +- 1 file

[PATCH 08/54] staging: wilc1000: rename u8seqlen of struct host_if_wpa_attr

2015-10-13 Thread Tony Cho
From: Leo Kim This patch renames u8seqlen of struct host_if_wpa_attr to seq_len to avoid CamelCase naming convention. Signed-off-by: Leo Kim Signed-off-by: Tony Cho --- drivers/staging/wilc1000/host_interface.c | 4 ++-- 1 file

[PATCH 31/54] staging: wilc1000: rename pvUserArg of struct connect_attr

2015-10-13 Thread Tony Cho
From: Leo Kim This patch renames pvUserArg of struct connect_attr to arg to avoid CamelCase naming convention. Signed-off-by: Leo Kim Signed-off-by: Tony Cho --- drivers/staging/wilc1000/host_interface.c | 8 1 file changed,

[PATCH 34/54] staging: wilc1000: rename pJoinParams of struct connect_attr

2015-10-13 Thread Tony Cho
From: Leo Kim This patch renames pJoinParams of struct connect_attr to params to avoid CamelCase naming convention. Signed-off-by: Leo Kim Signed-off-by: Tony Cho --- drivers/staging/wilc1000/host_interface.c | 6 +++--- 1 file

[PATCH 33/54] staging: wilc1000: rename u8channel of struct connect_attr

2015-10-13 Thread Tony Cho
From: Leo Kim This patch renames u8channel of struct connect_attr to ch to avoid CamelCase naming convention. Signed-off-by: Leo Kim Signed-off-by: Tony Cho --- drivers/staging/wilc1000/host_interface.c | 10 +- 1 file

[PATCH 43/54] staging: wilc1000: rename pu8Tail of struct beacon_attr

2015-10-13 Thread Tony Cho
From: Leo Kim This patch renames pu8Tail of struct beacon_attr to tail to avoid CamelCase naming convention. Signed-off-by: Leo Kim Signed-off-by: Tony Cho --- drivers/staging/wilc1000/host_interface.c | 20 ++-- 1

[PATCH 44/54] staging: wilc1000: rename au8Sta_DelAllSta of struct del_all_sta

2015-10-13 Thread Tony Cho
From: Leo Kim This patch renames au8Sta_DelAllSta of struct del_all_sta to del_all_sta to avoid CamelCase naming convention. Signed-off-by: Leo Kim Signed-off-by: Tony Cho --- drivers/staging/wilc1000/host_interface.c | 17

[PATCH 40/54] staging: wilc1000: rename u32HeadLen of struct beacon_attr

2015-10-13 Thread Tony Cho
From: Leo Kim This patch renames u32HeadLen of struct beacon_attr to head_len to avoid CamelCase naming convention. Signed-off-by: Leo Kim Signed-off-by: Tony Cho --- drivers/staging/wilc1000/host_interface.c | 18 +-

[PATCH 42/54] staging: wilc1000: rename u32TailLen of struct beacon_attr

2015-10-13 Thread Tony Cho
From: Leo Kim This patch renames u32TailLen of struct beacon_attr to tail_len to avoid CamelCase naming convention. Signed-off-by: Leo Kim Signed-off-by: Tony Cho --- drivers/staging/wilc1000/host_interface.c | 18 +-

[PATCH 41/54] staging: wilc1000: rename pu8Head of struct beacon_attr

2015-10-13 Thread Tony Cho
From: Leo Kim This patch renames pu8Head of struct beacon_attr to head to avoid CamelCase naming convention. Signed-off-by: Leo Kim Signed-off-by: Tony Cho --- drivers/staging/wilc1000/host_interface.c | 16 1 file

[PATCH 38/54] staging: wilc1000: rename u32Interval of struct beacon_attr

2015-10-13 Thread Tony Cho
From: Leo Kim This patch renames u32Interval of struct beacon_attr to interval to avoid CamelCase naming convention. Signed-off-by: Leo Kim Signed-off-by: Tony Cho --- drivers/staging/wilc1000/host_interface.c | 12 ++-- 1

[PATCH 36/54] staging: wilc1000: rename u32Length of struct rcvd_async_info

2015-10-13 Thread Tony Cho
From: Leo Kim This patch renames u32Length of struct rcvd_async_info to len to avoid CamelCase naming convention. Signed-off-by: Leo Kim Signed-off-by: Tony Cho --- drivers/staging/wilc1000/host_interface.c | 5 ++--- 1 file changed,

[PATCH 35/54] staging: wilc1000: rename pu8Buffer of struct rcvd_async_info

2015-10-13 Thread Tony Cho
From: Leo Kim This patch renames pu8Buffer of struct rcvd_async_info to buffer to avoid CamelCase naming convention. Signed-off-by: Leo Kim Signed-off-by: Tony Cho --- drivers/staging/wilc1000/host_interface.c | 33

[PATCH 39/54] staging: wilc1000: rename u32DTIMPeriod of struct beacon_attr

2015-10-13 Thread Tony Cho
From: Leo Kim This patch renames u32DTIMPeriod of struct beacon_attr to dtim_period to avoid CamelCase naming convention. Signed-off-by: Leo Kim Signed-off-by: Tony Cho --- drivers/staging/wilc1000/host_interface.c | 12 ++--

[PATCH 54/54] staging: wilc1000: wilc_msgqueue.c : remove the goto ERRORHANDER

2015-10-13 Thread Tony Cho
From: Leo Kim This patch removes goto ERRORHANDER and the result variable in wilc_mq_send. Then, the error type is directly returned. If normal operation, freeing memory is not needed in this function. Signed-off-by: Leo Kim Signed-off-by: Tony Cho

[PATCH 52/54] staging: wilc1000: rename pu8Buffer of struct rcvd_net_info

2015-10-13 Thread Tony Cho
From: Leo Kim This patch renames pu8Buffer of struct rcvd_net_info to buffer to avoid CamelCase naming convention. Signed-off-by: Leo Kim Signed-off-by: Tony Cho --- drivers/staging/wilc1000/host_interface.c | 13 ++---

Re: [PATCH 02/10] Drivers: hv: utils: run polling callback always in interrupt context

2015-10-13 Thread Olaf Hering
On Fri, Oct 09, Olaf Hering wrote: > On Thu, Oct 08, KY Srinivasan wrote: > > > > yes, but after doing fcopy_respond_to_host(). I'd suggest we leave the > > > check in place, better safe than sorry. > > > > Agreed; Olaf, if it is ok with you, I can fix it up and send. > > I will retest with

[PATCH 04/54] staging: wilc1000: rename uniHostIFkeyAttr of struct key_attr

2015-10-13 Thread Tony Cho
From: Leo Kim This patch renames uniHostIFkeyAttr of struct key_attr to attr to avoid CamelCase naming convention. Signed-off-by: Leo Kim Signed-off-by: Tony Cho --- drivers/staging/wilc1000/host_interface.c | 217

[PATCH 05/54] staging: wilc1000: rename pu8key of struct host_if_wpa_attr

2015-10-13 Thread Tony Cho
From: Leo Kim This patch renames pu8key of struct host_if_wpa_attr to key to avoid CamelCase naming convention. Signed-off-by: Leo Kim Signed-off-by: Tony Cho --- drivers/staging/wilc1000/host_interface.c | 36

[PATCH 12/54] staging: wilc1000: rename u8ScanSource of struct scan_attr

2015-10-13 Thread Tony Cho
From: Leo Kim This patch renames u8ScanSource of struct scan_attr to src to avoid CamelCase naming convention. Signed-off-by: Leo Kim Signed-off-by: Tony Cho --- drivers/staging/wilc1000/host_interface.c | 6 +++--- 1 file changed, 3

[PATCH 06/54] staging: wilc1000: rename pu8macaddr of struct host_if_wpa_attr

2015-10-13 Thread Tony Cho
From: Leo Kim This patch renames pu8macaddr of struct host_if_wpa_attr to mac_addr to avoid CamelCase naming convention. Signed-off-by: Leo Kim Signed-off-by: Tony Cho --- drivers/staging/wilc1000/host_interface.c | 8 1 file

[PATCH 11/54] staging: wilc1000: rename u8Ciphermode of struct host_if_wpa_attr

2015-10-13 Thread Tony Cho
From: Leo Kim This patch renames u8Ciphermode of struct host_if_wpa_attr to mode to avoid CamelCase naming convention. Signed-off-by: Leo Kim Signed-off-by: Tony Cho --- drivers/staging/wilc1000/host_interface.c | 10 +- 1

[PATCH 01/54] staging: wilc1000: host_interface.c : remove over-commenting

2015-10-13 Thread Tony Cho
From: Leo Kim There are over-commenting in the host_interface.c file and most of them are not helpful to explain what the code does and generate 80 ending line over warnings. So, all of comments are removed in this patch and the comments will later be added if necessary with

[PATCH 10/54] staging: wilc1000: rename u8Keylen of struct host_if_wpa_attr

2015-10-13 Thread Tony Cho
From: Leo Kim This patch renames u8Keylen of struct host_if_wpa_attr to key_len to avoid CamelCase naming convention. Signed-off-by: Leo Kim Signed-off-by: Tony Cho --- drivers/staging/wilc1000/host_interface.c | 22

[PATCH 19/54] staging: wilc1000: rename pfScanResult of struct scan_attr

2015-10-13 Thread Tony Cho
From: Leo Kim This patch renames pfScanResult of struct scan_attr to result to avoid CamelCase naming convention. Signed-off-by: Leo Kim Signed-off-by: Tony Cho --- drivers/staging/wilc1000/host_interface.c | 6 +++--- 1 file changed,

[PATCH 20/54] staging: wilc1000: rename pvUserArg of struct scan_attr

2015-10-13 Thread Tony Cho
From: Leo Kim This patch renames pvUserArg of struct scan_attr to arg to avoid CamelCase naming convention. Signed-off-by: Leo Kim Signed-off-by: Tony Cho --- drivers/staging/wilc1000/host_interface.c | 6 +++--- 1 file changed, 3

[PATCH 15/54] staging: wilc1000: rename pu8ChnlFreqList of struct scan_attr

2015-10-13 Thread Tony Cho
From: Leo Kim This patch renames pu8ChnlFreqList of struct scan_attr to ch_freq_list to avoid CamelCase naming convention. Signed-off-by: Leo Kim Signed-off-by: Tony Cho --- drivers/staging/wilc1000/host_interface.c | 21

[PATCH 14/54] staging: wilc1000: remove duplicate null check routine

2015-10-13 Thread Tony Cho
From: Leo Kim This patch removes the duplicate null check routine. The same null check routine is twice executed. Therefore the duplicated routine is not necessary. Signed-off-by: Leo Kim Signed-off-by: Tony Cho ---

[PATCH 16/54] staging: wilc1000: rename u8ChnlListLen of struct scan_attr

2015-10-13 Thread Tony Cho
From: Leo Kim This patch renames u8ChnlListLen of struct scan_attr to ch_list_len to avoid CamelCase naming convention. Signed-off-by: Leo Kim Signed-off-by: Tony Cho --- drivers/staging/wilc1000/host_interface.c | 10 +- 1

[PATCH 21/54] staging: wilc1000: rename strHiddenNetwork of struct scan_attr

2015-10-13 Thread Tony Cho
From: Leo Kim This patch renames strHiddenNetwork of struct scan_attr to hidden_network to avoid CamelCase naming convention. Signed-off-by: Leo Kim Signed-off-by: Tony Cho --- drivers/staging/wilc1000/host_interface.c | 28

[PATCH 18/54] staging: wilc1000: rename IEsLen of struct scan_attr

2015-10-13 Thread Tony Cho
From: Leo Kim This patch renames IEsLen of struct scan_attr to ies_len to avoid CamelCase naming convention. Signed-off-by: Leo Kim Signed-off-by: Tony Cho --- drivers/staging/wilc1000/host_interface.c | 6 +++--- 1 file changed, 3

[PATCH 09/54] staging: wilc1000: rename u8keyidx of struct host_if_wpa_attr

2015-10-13 Thread Tony Cho
From: Leo Kim This patch renames u8keyidx of struct host_if_wpa_attr to index to avoid CamelCase naming convention. Signed-off-by: Leo Kim Signed-off-by: Tony Cho --- drivers/staging/wilc1000/host_interface.c | 12 ++-- 1 file

[PATCH 03/54] staging: wilc1000: rename u8KeyAction of struct key_attr

2015-10-13 Thread Tony Cho
From: Leo Kim This patch renames u8KeyAction of struct key_attr to action to avoid CamelCase naming convention. Signed-off-by: Leo Kim Signed-off-by: Tony Cho --- drivers/staging/wilc1000/host_interface.c | 34

[PATCH 07/54] staging: wilc1000: rename pu8seq of struct host_if_wpa_attr

2015-10-13 Thread Tony Cho
From: Leo Kim This patch renames pu8seq of struct host_if_wpa_attr to seq to avoid CamelCase naming convention. Signed-off-by: Leo Kim Signed-off-by: Tony Cho --- drivers/staging/wilc1000/host_interface.c | 15 +++ 1 file

[PATCH 51/54] staging: wilc1000: rename au8IPAddr of struct set_ip_addr

2015-10-13 Thread Tony Cho
From: Leo Kim This patch renames au8IPAddr of struct set_ip_addr to ip_addr to avoid CamelCase naming convention. Signed-off-by: Leo Kim Signed-off-by: Tony Cho --- drivers/staging/wilc1000/host_interface.c | 10 +- 1 file

[PATCH 53/54] staging: wilc1000: rename u32Length of struct rcvd_net_info

2015-10-13 Thread Tony Cho
From: Leo Kim This patch renames u32Length of struct rcvd_net_info to len to avoid CamelCase naming convention. Signed-off-by: Leo Kim Signed-off-by: Tony Cho --- drivers/staging/wilc1000/host_interface.c | 2 +-

[PATCH 49/54] staging: wilc1000: rename bIsEnabled of struct power_mgmt_param

2015-10-13 Thread Tony Cho
From: Leo Kim This patch renames bIsEnabled of struct power_mgmt_param to enabled to avoid CamelCase naming convention. Signed-off-by: Leo Kim Signed-off-by: Tony Cho --- drivers/staging/wilc1000/host_interface.c | 7 +++ 1 file

[PATCH 48/54] staging: wilc1000: rename au8MacAddr of struct del_sta

2015-10-13 Thread Tony Cho
From: Leo Kim This patch renames au8MacAddr of struct del_sta to mac_addr to avoid CamelCase naming convention. Signed-off-by: Leo Kim Signed-off-by: Tony Cho --- drivers/staging/wilc1000/host_interface.c | 8 1 file changed,

[PATCH 46/54] staging: wilc1000: rename bIsEnabled of struct set_multicast

2015-10-13 Thread Tony Cho
From: Leo Kim This patch renames bIsEnabled of struct set_multicast to enabled to avoid CamelCase naming convention. Signed-off-by: Leo Kim Signed-off-by: Tony Cho --- drivers/staging/wilc1000/host_interface.c | 12 ++-- 1

[PATCH 50/54] staging: wilc1000: rename u32Timeout of struct power_mgmt_param

2015-10-13 Thread Tony Cho
From: Leo Kim This patch renames u32Timeout of struct power_mgmt_param to timeout to avoid CamelCase naming convention. Signed-off-by: Leo Kim Signed-off-by: Tony Cho --- drivers/staging/wilc1000/host_interface.c | 5 ++--- 1 file

[PATCH 47/54] staging: wilc1000: rename u32count of struct set_multicast

2015-10-13 Thread Tony Cho
From: Leo Kim This patch renames u32count of struct set_multicast to cnt to avoid CamelCase naming convention. Signed-off-by: Leo Kim Signed-off-by: Tony Cho --- drivers/staging/wilc1000/host_interface.c | 18 +- 1

[staging:staging-testing 1385/1471] ERROR: "ion_device_add_heap" [drivers/staging/android/ion/tegra/tegra_ion.ko] undefined!

2015-10-13 Thread kbuild test robot
tree: https://git.kernel.org/pub/scm/linux/kernel/git/gregkh/staging.git staging-testing head: fa7ac9c0fc5418154a6356583f880164cdcaa7f1 commit: 35890c28eec9535d910380b598712bd4b6d7717d [1385/1471] drivers/staging: make android tegra_ion.c properly tristate config: arm-allmodconfig (attached

lustre: TODO updated?

2015-10-13 Thread Xose Vazquez Perez
Hi, Is drivers/staging/lustre/TODO file updated? -thanks- ___ devel mailing list de...@linuxdriverproject.org http://driverdev.linuxdriverproject.org/mailman/listinfo/driverdev-devel

Re: [PATCH v2 0/5] staging: comedi: adl_pci9111: fix checkpatch.pl issues

2015-10-13 Thread Ian Abbott
On 12/10/15 18:42, H Hartley Sweeten wrote: Fix the checkpatch.pl issues in this driver. v2: split the ai range macro out of patch 1 as requested by Ian Abbott. H Hartley Sweeten (5): staging: comedi: adl_pci9111: prefer using the BIT macro staging: comedi: adl_pci9111: define a macro

Re: [PATCH 00/10] staging: comedi: cb_das16_cs: cleanup driver

2015-10-13 Thread Ian Abbott
On 13/10/15 00:07, H Hartley Sweeten wrote: Fix the checkpatch.pl issues in this driver and tidy it up a bit. H Hartley Sweeten (10): staging: comedi: cb_das16_cs: tidy up multi-line comments staging: comedi: cb_das16_cs: rename register map defines staging: comedi: cb_das16_cs: fix ai

[PATCH 2/3] staging: lustre: remove multiple blank lines

2015-10-13 Thread Mike Rapoport
Fixes checkpatch.pl CHECK:LINE_SPACING: Please don't use multiple blank lines. The patch is generated using checkpatch.pl --fix-inplace: for f in $(find drivers/staging/lustre/ -type f) ; do ./scripts/checkpatch.pl --types "LINE_SPACING" --test-only=multiple \ --fix-inplace -f $f done

[PATCH 1/3] staging: lustre: add missing blank line after declarations

2015-10-13 Thread Mike Rapoport
Fixes checkpatch.pl WARNING:LINE_SPACING: Missing a blank line after declarations. The patch is generated using checkpatch.pl --fix-inplace: for f in $(find drivers/staging/lustre/ -type f) ; do ./scripts/checkpatch.pl --types "LINE_SPACING" --test-only=Missing \ --fix-inplace -f $f done

[PATCH 0/3] staging: lustre: fix LINE_SPACING checkpatch warnings

2015-10-13 Thread Mike Rapoport
Hi, These patches add/remove blank lines to fix several types of LINE_SPACING warnings reported by the checkpatch.pl Mike Rapoport (3): staging: lustre: add missing blank line after declarations staging: lustre: remove multiple blank lines staging: lustre: add a blank line after

Re: [PATCH v2 0/6] staging: comedi: adv_pci1710: fix checkpatch.pl issues

2015-10-13 Thread Ian Abbott
On 12/10/15 19:01, H Hartley Sweeten wrote: Fix the checkpaatch.pl issues in this driver. v2: reword the comments in patch 6 to clarify the 'ai_et_MuxVal'. H Hartley Sweeten (6): staging: comedi: adv_pci1710: tidy up multi-line comments staging: comedi: adv_pci1710: tidy up status

[PATCH 3/3] staging: lustre: add a blank line after function/struct/union/enum declarations

2015-10-13 Thread Mike Rapoport
Fixes checkpatch.pl CHECK:LINE_SPACING: Please use a blank line after function/struct/union/enum declarations The patch is generated using checkpatch.pl --fix-inplace: for f in $(find drivers/staging/lustre/ -type f) ; do ./scripts/checkpatch.pl --types "LINE_SPACING" --fix-inplace -f $f

Re: [PATCH 00/10] staging: comedi: aio_aio12_8: cleanup driver

2015-10-13 Thread Ian Abbott
On 12/10/15 20:16, H Hartley Sweeten wrote: Fix the checkpatch.pl issues in this driver and tidy it up a bit. H Hartley Sweeten (10): staging: comedi: aio_aio12_8: prefer using the BIT macro staging: comedi: aio_aio12_8: tidy up multi-line comments staging: comedi: aio_aio12_8: update

[staging:staging-testing 1385/1471] ERROR: "ion_device_add_heap" undefined!

2015-10-13 Thread kbuild test robot
tree: https://git.kernel.org/pub/scm/linux/kernel/git/gregkh/staging.git staging-testing head: fa7ac9c0fc5418154a6356583f880164cdcaa7f1 commit: 35890c28eec9535d910380b598712bd4b6d7717d [1385/1471] drivers/staging: make android tegra_ion.c properly tristate config: arm64-allmodconfig

Re: [RESEND PATCH v2] staging/wlan-ng: Fix potential buffer overflow in firmware handling

2015-10-13 Thread Tillmann Heidsieck
On Mon, Oct 12, 2015 at 05:11:08PM +0530, Sudip Mukherjee wrote: > On Mon, Oct 12, 2015 at 01:35:54PM +0200, Tillmann Heidsieck wrote: > > > > I must have rediscovered a problem I already fixed :-( > > Which tree are you using? You should be using staging-testing branch of > the staging tree.

Re: [PATCH 2/3] staging: rtl8188eu: bugfix: correct channels range is set in Hal_ReadTxPowerInfo88E

2015-10-13 Thread Ivan Safonov
On 10/13/2015 10:47 AM, Greg Kroah-Hartman wrote: On Sat, Oct 10, 2015 at 01:27:00PM +0700, Ivan Safonov wrote: Correct channels range is 1..14 (numbering from 1) but not 0..13. Have you tested this? I have not tested it. Why? From condition if (chnl <= 14) { bIn24G = true;

Re: [PATCH 9/9] staging: unisys: vmcallinterface.h: convert pragma to __packed

2015-10-13 Thread Ben Romer
On 10/12/2015 11:51 PM, Greg KH wrote: On Mon, Oct 12, 2015 at 03:19:47PM -0400, Benjamin Romer wrote: From: David Kershner Convert from pragma to __packed Signed-off-by: David Kershner Signed-off-by: Benjamin Romer

Re: [PATCH 54/54] staging: wilc1000: wilc_msgqueue.c : remove the goto ERRORHANDER

2015-10-13 Thread Mike Rapoport
On Tue, Oct 13, 2015 at 08:02:12PM +0900, Tony Cho wrote: > From: Leo Kim > > This patch removes goto ERRORHANDER and the result variable in wilc_mq_send. > Then, the error type is directly returned. If normal operation, freeing memory > is not needed in this function. > >

Re: [PATCH 1/3] staging: rtl8188eu: 5 GHz channel groups removed from /hal/rtl8188e_hal_init.c

2015-10-13 Thread Ivan Safonov
On 10/13/2015 10:47 AM, Greg Kroah-Hartman wrote: On Sat, Oct 10, 2015 at 01:23:13PM +0700, Ivan Safonov wrote: Unnecessary channel groups for 5 GHz removed from Hal_GetChnlGroup88E and it transformed to pretty get_channel_group(const u8 channel). Also removed code for 5 GHz frequency in

[PATCH 9/9 v2] staging: unisys: vmcallinterface.h: convert pragma to __packed

2015-10-13 Thread Benjamin Romer
From: David Kershner Convert from pragma to __packed Signed-off-by: David Kershner Signed-off-by: Benjamin Romer --- v2: remove comments about pragma as well ---

Re: [PATCH 1/3] staging: rtl8188eu: 5 GHz channel groups removed from /hal/rtl8188e_hal_init.c

2015-10-13 Thread Greg Kroah-Hartman
On Tue, Oct 13, 2015 at 10:00:41PM +0700, Ivan Safonov wrote: > On 10/13/2015 10:47 AM, Greg Kroah-Hartman wrote: > >On Sat, Oct 10, 2015 at 01:23:13PM +0700, Ivan Safonov wrote: > >>Unnecessary channel groups for 5 GHz removed from Hal_GetChnlGroup88E > >>and it transformed to pretty

Re: [PATCH 1/3] staging: rtl8188eu: 5 GHz channel groups removed from /hal/rtl8188e_hal_init.c

2015-10-13 Thread Ivan Safonov
On 10/13/2015 10:40 PM, Greg Kroah-Hartman wrote: On Tue, Oct 13, 2015 at 10:00:41PM +0700, Ivan Safonov wrote: On 10/13/2015 10:47 AM, Greg Kroah-Hartman wrote: On Sat, Oct 10, 2015 at 01:23:13PM +0700, Ivan Safonov wrote: Unnecessary channel groups for 5 GHz removed from Hal_GetChnlGroup88E

randconfig build error with next-20151013, in drivers/staging/unisys/visorinput

2015-10-13 Thread Jim Davis
Building with the attached random configuration file, ERROR: "input_free_device" [drivers/staging/unisys/visorinput/visorinput.ko] undefined! ERROR: "input_set_capability" [drivers/staging/unisys/visorinput/visorinput.ko] undefined! ERROR: "input_set_abs_params"

RE: randconfig build error with next-20151013, in drivers/staging/unisys/visorinput

2015-10-13 Thread Sell, Timothy C
day, October 13, 2015 1:34 PM > To: Stephen Rothwell; linux-next; linux-kernel; Romer, Benjamin M; > Kershner, David A; Greg Kroah-Hartman; *S-Par-Maintainer; devel > Subject: randconfig build error with next-20151013, in > drivers/staging/unisys/visorinput > > Building with the

Re: [staging:staging-testing 1385/1471] ERROR: "ion_device_add_heap" undefined!

2015-10-13 Thread Paul Gortmaker
[[staging:staging-testing 1385/1471] ERROR: "ion_device_add_heap" undefined!] On 13/10/2015 (Tue 20:14) kbuild test robot wrote: > tree: https://git.kernel.org/pub/scm/linux/kernel/git/gregkh/staging.git > staging-testing > head: fa7ac9c0fc5418154a6356583f880164cdcaa7f1 > commit:

Re: [PATCH v4] mtd: nand: pass page number to ecc->write_xxx() methods

2015-10-13 Thread Brian Norris
On Tue, Oct 13, 2015 at 11:22:18AM +0200, Boris Brezillon wrote: > The ->read_xxx() methods are all passed the page number the NAND controller > is supposed to read, but ->write_xxx() do not have such a parameter. > > This is a problem if we want to properly implement data >

[PATCH] Drivers: hv: vss: run only on supported host versions

2015-10-13 Thread Olaf Hering
The Backup integration service on WS2012 has appearently trouble to negotiate with a guest which does not support the provided util version. Currently the VSS driver supports only version 5/0. A WS2012 offers only version 1/x and 3/x, and vmbus_prep_negotiate_resp correctly returns an empty

[PATCH] "drivers/staging: mark android/ion fcns with EXPORT_SYMBOL for tristate

2015-10-13 Thread Paul Gortmaker
In a recent change, we made a bool into a tristate in: "drivers/staging: make android tegra_ion.c properly tristate", since it was self evident that was the original intention. However on the final link phase we'll see an allmodconfig fail with: ERROR: "ion_device_add_heap"

RE: [PATCH 02/10] Drivers: hv: utils: run polling callback always in interrupt context

2015-10-13 Thread KY Srinivasan
> -Original Message- > From: Olaf Hering [mailto:o...@aepfle.de] > Sent: Tuesday, October 13, 2015 2:47 AM > To: KY Srinivasan > Cc: Vitaly Kuznetsov ; gre...@linuxfoundation.org; > linux-ker...@vger.kernel.org; de...@linuxdriverproject.org; >

[PATCH 01/45] staging: comedi: cb_pcidas: tidy up multi-line comments

2015-10-13 Thread H Hartley Sweeten
Reformat the multi-line comments in the kernel CodingStyle. Signed-off-by: H Hartley Sweeten Cc: Ian Abbott Cc: Greg Kroah-Hartman --- drivers/staging/comedi/drivers/cb_pcidas.c | 115 ++--- 1

[PATCH 00/45] staging: comedi: cb_pcidas: cleanup driver

2015-10-13 Thread H Hartley Sweeten
Following is the big cleanup of the comedi cb_pcidas driver. H Hartley Sweeten (45): staging: comedi: cb_pcidas: tidy up multi-line comments staging: comedi: cb_pcidas: remove NUM_CHANNELS_* defines staging: comedi: cb_pcidas: refactor 'ai_bits' boardinfo staging: comedi: cb_pcidas:

[PATCH 5/7] staging: lustre: osc_io.c: replace container_of0 by container_of

2015-10-13 Thread Aya Mahfouz
Replaces container_of0 by container_of. The only difference between the two implementations is that container_of0 tries to evade type casting if the pointer is erroneous or null. The use of container_of is encouraged to bring lustre one step closer to community standards. Signed-off-by: Aya

[PATCH 4/7] staging: lustre: echo_client.c: replace container_of0 by container_of

2015-10-13 Thread Aya Mahfouz
Replaces container_of0 by container_of. The only difference between the two implementations is that container_of0 tries to evade type casting if the pointer is erroneous or null. The use of container_of is encouraged to bring lustre one step closer to community standards. Signed-off-by: Aya

Re: [PATCH 0/7] staging: lustre: remove uses and definition of container_of0

2015-10-13 Thread Dilger, Andreas
On 2015/10/13, 16:19, "Aya Mahfouz" wrote: >container_of0 can be replaced by the Linux kernel macro container_of. >The only difference is that container_of0 tries to evade type casting >when the pointer is erroneous or null. All uses of container_of0 have >been

[PATCH 05/12] staging_fsl-mc: Changed types of flags, portal size in

2015-10-13 Thread J. German Rivera
Changed these two fields from 32-bit integers to 16-bit integers in struct fsl_mc_io, as 32 bits is too much for these fields. This change does not affect other components since fsl_mc_io is an opaque type. --- drivers/staging/fsl-mc/include/mc-sys.h | 4 ++-- 1 file changed, 2 insertions(+), 2

[PATCH 07/12] staging: fsl-mc: Fixed alignment of copyright comment

2015-10-13 Thread J. German Rivera
Whitespace cleanup-- add missing spaces in column 1 of copyright --- drivers/staging/fsl-mc/include/dpcon-cmd.h | 60 +++--- 1 file changed, 30 insertions(+), 30 deletions(-) diff --git a/drivers/staging/fsl-mc/include/dpcon-cmd.h

[PATCH 02/12] staging: fsl-mc: fsl_mc_io object refactoring

2015-10-13 Thread J. German Rivera
Each fsl_mc_io object is associated with an fsl_mc_device object of type "dpmcp" representing the MC portal associated with the fsl_mc_io object. Before, we were representing this association with an fsl_mc_resource pointer. To enhance code clarity, it is more straight forward to use an

[PATCH 12/12] staging: fsl-mc: Added missing initializer in fsl_mc_bus_driver

2015-10-13 Thread J. German Rivera
owner needs to be initialized as THIS_MOUDLE. --- drivers/staging/fsl-mc/bus/mc-bus.c | 1 + 1 file changed, 1 insertion(+) diff --git a/drivers/staging/fsl-mc/bus/mc-bus.c b/drivers/staging/fsl-mc/bus/mc-bus.c index 4ac3d07..84db55b 100644 --- a/drivers/staging/fsl-mc/bus/mc-bus.c +++

[PATCH 04/12] staging: fsl-mc: Changed dev_info() calls to dev_dbg()

2015-10-13 Thread J. German Rivera
Changed dev_info() calls to dev_dbg() in fsl_mc_allocator_probe/fsl_mc_allocator_remove, as they are useful only for debugging. --- drivers/staging/fsl-mc/bus/mc-allocator.c | 8 1 file changed, 4 insertions(+), 4 deletions(-) diff --git a/drivers/staging/fsl-mc/bus/mc-allocator.c

[PATCH 11/12] staging: fsl-mc: fixed bug in uninitialized root dprc irq count

2015-10-13 Thread J. German Rivera
When initializing the object attributes for the root dprc, the irq_count was uninitialized. Initialize it to 1. --- drivers/staging/fsl-mc/bus/mc-bus.c | 1 + 1 file changed, 1 insertion(+) diff --git a/drivers/staging/fsl-mc/bus/mc-bus.c b/drivers/staging/fsl-mc/bus/mc-bus.c index

[PATCH 08/12] staging: fsl-mc: Fixed bug in fsl_mc_allocator_remove

2015-10-13 Thread J. German Rivera
Call fsl_mc_resource_pool_remove_device() only if mc_dev->resource is not NULL. --- drivers/staging/fsl-mc/bus/mc-allocator.c | 8 +--- 1 file changed, 5 insertions(+), 3 deletions(-) diff --git a/drivers/staging/fsl-mc/bus/mc-allocator.c b/drivers/staging/fsl-mc/bus/mc-allocator.c index

[PATCH 09/12] staging: fsl-mc: refactored error exit in allocator probe/remove

2015-10-13 Thread J. German Rivera
Replaced error gotos with direct returns in fsl_mc_allocator_probe() and fsl_mc_allocator_remove(), since the only error handling done in those functions is to exit. Signed-off-by: J. German Rivera --- drivers/staging/fsl-mc/bus/mc-allocator.c | 23

  1   2   >