On Wed, Oct 14, 2015 at 07:33:00PM +0530, Sakshi Bansal wrote:
> Fixed block comments usage of * on subsequent lines
>
> Signed-off-by: Sakshi Bansal
> ---
> drivers/staging/comedi/comedi.h| 181
> +++--
> drivers/staging/comedi/comedi_pcmcia.h | 3 +-
> dr
On Tue, Oct 13, 2015 at 05:48:03PM -0700, H Hartley Sweeten wrote:
> Clarify the interrupt handler by splitting the analog input handling
> into a new function.
>
> Signed-off-by: H Hartley Sweeten
> Reviewed-by: Ian Abbott
> ---
> drivers/staging/comedi/drivers/cb_pcidas.c | 80
>
On Wed, Oct 14, 2015 at 03:11:41PM -0500, J. German Rivera wrote:
> This patch series depends on the patch series posted
> at http://www.mail-archive.com/linux-kernel@vger.kernel.org/msg996780.html
I didn't take that series for obvious reasons :(
___
de
On Wed, Oct 14, 2015 at 02:51:40PM -0500, J. German Rivera wrote:
> mc_adev is a local variable for the allocated dpmcp object.
> Renamed mc_adev as dpmcp_dev for clarity.
> ---
> CHANGE HISTORY
>
> Changes in v2: none
None of the patches in this series has a signed-off-by: line, as
required by D
On Fri, Oct 16, 2015 at 10:06:48AM -0400, Benjamin Romer wrote:
> From: Tim Sell
>
> This is NOT technically required for the code as it stands now, but will
> be needed for subsequent patches.
>
> Signed-off-by: Tim Sell
> Signed-off-by: Benjamin Romer
> ---
> drivers/staging/unisys/visorinp
On Sat, Oct 17, 2015 at 12:05:44AM +0200, Aya Mahfouz wrote:
> Replaces IS_PO2 by is_power_of_2. IS_PO2 is used with several debug
> macros. In this case, it is LASSERT. Note that the replacement changes
> the types involved, because the parameter of IS_PO2 is of type long
> long and the return typ
On Sat, Oct 17, 2015 at 12:06:59AM +0200, Aya Mahfouz wrote:
> Replaces IS_PO2 by is_power_of_2. IS_PO2 is used with several debug
> macros. In this case, it is CDEBUG. Note that the replacement changes
> the types involved, because the parameter of IS_PO2 is of type long long
> and the return type
On Sat, Oct 17, 2015 at 12:06:28AM +0200, Aya Mahfouz wrote:
> Replaces IS_PO2 by is_power_of_2. IS_PO2 is used with several debug
> macros. In this case, it is CDEBUG. Note that the replacement changes
> the types involved, because the parameter of IS_PO2 is of type long long
> and the return type
On Sat, Oct 17, 2015 at 12:07:28AM +0200, Aya Mahfouz wrote:
> Removes IS_PO2 and __is_po2 since the uses of IS_PO2 have
> been replaced by is_power_of_2
>
> Signed-off-by: Aya Mahfouz
> ---
> drivers/staging/lustre/include/linux/libcfs/libcfs.h | 7 ---
> 1 file changed, 7 deletions(-)
You
On Wed, Oct 14, 2015 at 11:55:55PM +0530, Punit Vara wrote:
> Put constant on the right side of a test.Problem found using checkpatch:
>
> Warning:Comparisons should place the constant on the right side of the test
>
> Signed-off-by: Punit Vara
> ---
> drivers/staging/rtl8712/rtl871x_ioctl_lin
On Tue, Oct 13, 2015 at 11:27:54AM +0530, Sudip Mukherjee wrote:
> On Mon, Oct 12, 2015 at 08:30:41PM -0700, Greg Kroah-Hartman wrote:
> > On Fri, Oct 09, 2015 at 12:32:32PM +0530, Sudip Mukherjee wrote:
> > > On Thu, Oct 08, 2015 at 09:11:07PM +0100, Mike Rapoport wrote:
> > > > Hi Sudip,
> > > >
On Fri, Oct 16, 2015 at 06:22:57PM +0900, Tony Cho wrote:
> From: Glen Lee
>
> This patch adds struct wilc to struct perInterface_wlan_t which is netdev
> private data to access wilc via netdev_priv function and assigns wilc to
> nic->wilc. The global variable wl will be replaced with netdev priv
On Fri, Oct 16, 2015 at 01:37:31PM +0900, Tony Cho wrote:
> From: Leo Kim
>
> This patch renames ssidLen of struct join_bss_param to ssid_len
> to avoid CamelCase naming convention.
>
> Signed-off-by: Leo Kim
> Signed-off-by: Tony Cho
> ---
> drivers/staging/wilc1000/host_interface.c | 4 ++--
On Thu, Oct 15, 2015 at 01:25:16PM +0900, Tony Cho wrote:
> From: Leo Kim
>
> This patch renames variable s32Error to result
> to avoid CamelCase naming convention.
>
> Signed-off-by: Leo Kim
> Signed-off-by: Tony Cho
> ---
> drivers/staging/wilc1000/host_interface.c | 784
> ++--
Hi Aya,
[auto build test ERROR on staging/staging-next -- if it's inappropriate base,
please suggest rules for selecting the more suitable base]
url:
https://github.com/0day-ci/linux/commits/Aya-Mahfouz/Remove-uses-and-definition-of-IS_PO2/20151017-060923
config: s390-allmodconfig (attached
On Wed, Oct 14, 2015 at 08:46:33PM +0900, Tony Cho wrote:
> This patch changes the type of wilc1000_initialized in the struct wilc from
> int to bool and renames it to the initialized. In addition, the unncessary
> wilc1000_initialized codes are removed in this patch.
because I didn't take the ren
On Wed, Oct 14, 2015 at 08:46:31PM +0900, Tony Cho wrote:
> This patch renames g_linux_wlan to wl.
Why? What's wrong with g_linux_wlan? You will need to delete this
entire variable soon anyway, might as well leave it alone until then.
thanks,
greg k-h
__
On Wed, Oct 14, 2015 at 05:52:55PM +0900, Tony Cho wrote:
>
>
> On 2015년 10월 14일 17:32, Mike Rapoport wrote:
> >On Wed, Oct 14, 2015 at 02:55:43PM +0900, Tony Cho wrote:
> >>From: Leo Kim
> >>
> >>This patch removes goto ERRORHANDER and the result variable in wilc_mq_send.
> >>Then, the error ty
> -Original Message-
> From: Dan Carpenter [mailto:dan.carpen...@oracle.com]
> Sent: Friday, October 16, 2015 9:03 PM
> To: Romer, Benjamin M
> Cc: gre...@linuxfoundation.org; *S-Par-Maintainer; driverdev-
> de...@linuxdriverproject.org; Sell, Timothy C
> Subject: Re: [PATCH 3/9] staging: u
Hi Aya,
[auto build test ERROR on staging/staging-next -- if it's inappropriate base,
please suggest rules for selecting the more suitable base]
url:
https://github.com/0day-ci/linux/commits/Aya-Mahfouz/Remove-uses-and-definition-of-IS_PO2/20151017-060923
config: i386-allyesconfig (attached
On Fri, Oct 16, 2015 at 10:06:48AM -0400, Benjamin Romer wrote:
> @@ -526,7 +549,8 @@ visorinput_channel_interrupt(struct visor_device *dev)
> int xmotion, ymotion, zmotion, button;
> int i;
>
> - struct visorinput_devdata *devdata = dev_get_drvdata(&dev->device);
> + struct v
Hi Aya,
[auto build test ERROR on staging/staging-next -- if it's inappropriate base,
please suggest rules for selecting the more suitable base]
url:
https://github.com/0day-ci/linux/commits/Aya-Mahfouz/Remove-uses-and-definition-of-IS_PO2/20151017-060923
config: i386-allmodconfig (attached
On Tue, Oct 13, 2015 at 07:49:48PM +0900, Tony Cho wrote:
> 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
> ---
This is a beha
On Fri, Oct 16, 2015 at 02:26:31PM +0100, Luis de Bethencourt wrote:
> diff --git a/drivers/staging/rtl8192u/ieee80211/ieee80211_softmac.c
> b/drivers/staging/rtl8192u/ieee80211/ieee80211_softmac.c
> index c443e2e..f85e5ff 100644
> --- a/drivers/staging/rtl8192u/ieee80211/ieee80211_softmac.c
> +++
The iio dummy code was recently changed to use irq_work_queue, but
that code is compiled into the kernel only if IRQ_WORK is set, so
we can get a link error here:
drivers/built-in.o: In function `iio_evgen_poke':
(.text+0x208a04): undefined reference to `irq_work_queue'
This changes the Kconfig f
Removes IS_PO2 and __is_po2 since the uses of IS_PO2 have
been replaced by is_power_of_2
Signed-off-by: Aya Mahfouz
---
drivers/staging/lustre/include/linux/libcfs/libcfs.h | 7 ---
1 file changed, 7 deletions(-)
diff --git a/drivers/staging/lustre/include/linux/libcfs/libcfs.h
b/drivers/s
Replaces IS_PO2 by is_power_of_2. IS_PO2 is used with several debug
macros. In this case, it is CDEBUG. Note that the replacement changes
the types involved, because the parameter of IS_PO2 is of type long long
and the return type is int, while the parameter of is_power_of_2 is of
type long and the
Replaces IS_PO2 by is_power_of_2. IS_PO2 is used with several debug
macros. In this case, it is CDEBUG. Note that the replacement changes
the types involved, because the parameter of IS_PO2 is of type long long
and the return type is int, while the parameter of is_power_of_2 is of
type long and the
Replaces IS_PO2 by is_power_of_2. IS_PO2 is used with several debug
macros. In this case, it is LASSERT. Note that the replacement changes
the types involved, because the parameter of IS_PO2 is of type long
long and the return type is int, while the parameter of is_power_of_2
is of type long and th
Concerned with the removal of IS_PO2 by replacing its uses
with is_power_of_2 and then removing the definition
Aya Mahfouz (4):
staging: lustre: ldlm_extent.c: replace IS_PO2 by is_power_of_2
staging: lustre: hash.c: replace IS_PO2 by is_power_of_2
staging: lustre: workitem.c: replace IS_PO2
My first attempt to fix the Kconfig logic for wilc1000 was incomplete,
as it missed the case where SPI is built-in while SDIO is modular
and wilc1000 is configured as built-in in SPI mode (or vice versa),
which would still lead to a link failure.
This works around the problem by adding an intermed
A recent change to wilc1000 accidentally deleted an important
variable, so we now get a build error when the SPI mode is
selected:
ERROR: "wilc_spi_dev" [drivers/staging/wilc1000/wilc1000.ko] undefined!
This partially reverts the broken commit to put the variable back.
Signed-off-by: Arnd Bergma
Boolean tests do not need explicit comparison to true or false.
Signed-off-by: Luis de Bethencourt
---
drivers/staging/wilc1000/wilc_wfi_cfgoperations.c | 8
1 file changed, 4 insertions(+), 4 deletions(-)
diff --git a/drivers/staging/wilc1000/wilc_wfi_cfgoperations.c
b/drivers/stagin
From: Tim Sell
Previously, we used a hack to determine the max x,y resolution of the
visor virtual mouse: we just looked at the resolution of the
first-registered framebuffer device, using the currently-valid assumption
that in a Unisys s-Par guest environment the video will be provided by an
efi
From: Tim Sell
Fix the multi-line comment formatting in visorinput.c.
Signed-off-by: Tim Sell
Signed-off-by: Benjamin Romer
---
drivers/staging/unisys/visorinput/visorinput.c | 3 ++-
1 file changed, 2 insertions(+), 1 deletion(-)
diff --git a/drivers/staging/unisys/visorinput/visorinput.c
This patch series fixes visorinput to remove the dependency on FB and
add dependency to INPUT, cleans up some formatting issues found with
checkpatch.pl, and adds the capability to change screen resolutions
without breaking mouse functionality.
Tim Sell (9):
staging: unisys: visorinput: address
From: Tim Sell
This is NOT technically required for the code as it stands now, but will
be needed for subsequent patches.
Signed-off-by: Tim Sell
Signed-off-by: Benjamin Romer
---
drivers/staging/unisys/visorinput/visorinput.c | 45 --
1 file changed, 35 insertions(+),
From: Tim Sell
Whenever the absolute x,y resolution of the video (and mouse) changes, the
back-end sends an inputaction_set_max_xy message indicating the new
resolution. This commit adds the infrastructure to detect and correctly
respond to that message.
There were several reasons this wasn't a
From: Tim Sell
Ensure we properly lock between visorinput_channel_interrupt() and
devdata_create(). We now guarantee that interrupts will be disabled and
we will be holding lock_visor_dev at the time input_register_device() is
called (from register_client_keyboard() or register_client_mouse()),
From: Tim Sell
The dev_info() messages at init time are particularly useful for mapping
visor devices to input devices.
Signed-off-by: Tim Sell
Signed-off-by: Benjamin Romer
---
drivers/staging/unisys/visorinput/visorinput.c | 11 +++
1 file changed, 11 insertions(+)
diff --git a/dri
From: Tim Sell
Fix the alignment of function parameters to the parenthesis above.
Signed-off-by: Tim Sell
Signed-off-by: Benjamin Romer
---
drivers/staging/unisys/visorinput/visorinput.c | 4 ++--
1 file changed, 2 insertions(+), 2 deletions(-)
diff --git a/drivers/staging/unisys/visorinput/
From: Tim Sell
This commit sanity checks so that if a change resolution request is ever
received for a non-mouse device, that an error message will be logged and
the message will be ignored.
Signed-off-by: Tim Sell
Signed-off-by: Benjamin Romer
---
drivers/staging/unisys/visorinput/visorinput
From: Tim Sell
This was an obvious omission, as visorinput is an input-class driver.
Signed-off-by: Tim Sell
Signed-off-by: Benjamin Romer
---
drivers/staging/unisys/visorinput/Kconfig | 2 +-
1 file changed, 1 insertion(+), 1 deletion(-)
diff --git a/drivers/staging/unisys/visorinput/Kconfi
The code can be much cleaner and readable by simplifying the conditional
statement.
Only need to check if (ieee->state > IEEE80211_LINKED) and not >= because
(ieee->state == IEEE80211_LINKED) is already checked a few lines above.
Signed-off-by: Luis de Bethencourt
---
Sorry for the typo in the
From: Glen Lee
This patch changes function argument type wilc with net_device and use
netdev private data member wilc instead of p_nic.
Signed-off-by: Glen Lee
Signed-off-by: Tony Cho
---
drivers/staging/wilc1000/linux_wlan.c | 18 +++---
1 file changed, 11 insertions(+), 7 deleti
From: Glen Lee
Use netdev private data member wilc instead of global variable wl.
Signed-off-by: Glen Lee
Signed-off-by: Tony Cho
---
drivers/staging/wilc1000/wilc_wfi_cfgoperations.c | 4
1 file changed, 4 insertions(+)
diff --git a/drivers/staging/wilc1000/wilc_wfi_cfgoperations.c
b/
From: Glen Lee
Use netdev private data member wilc instead of global variable wl.
Signed-off-by: Glen Lee
Signed-off-by: Tony Cho
---
drivers/staging/wilc1000/wilc_wfi_cfgoperations.c | 4
1 file changed, 4 insertions(+)
diff --git a/drivers/staging/wilc1000/wilc_wfi_cfgoperations.c
b/
From: Glen Lee
Use netdev private data member wilc instead of global variable wl
Signed-off-by: Glen Lee
Signed-off-by: Tony Cho
---
drivers/staging/wilc1000/wilc_wfi_cfgoperations.c | 4
1 file changed, 4 insertions(+)
diff --git a/drivers/staging/wilc1000/wilc_wfi_cfgoperations.c
b/d
From: Glen Lee
Use netdev private data member wilc instead of global variable wl.
Signed-off-by: Glen Lee
Signed-off-by: Tony Cho
---
drivers/staging/wilc1000/wilc_wfi_cfgoperations.c | 4
1 file changed, 4 insertions(+)
diff --git a/drivers/staging/wilc1000/wilc_wfi_cfgoperations.c
b/
From: Glen Lee
Use netdev private data member wilc instead of global variable wl.
Signed-off-by: Glen Lee
Signed-off-by: Tony Cho
---
drivers/staging/wilc1000/wilc_wfi_cfgoperations.c | 4 ++--
1 file changed, 2 insertions(+), 2 deletions(-)
diff --git a/drivers/staging/wilc1000/wilc_wfi_cfg
From: Glen Lee
This patch changes function argument wilc with net_device and use nic->wilc
instead of global variable wl. The null check codes should be placed before it
is used so move it.
Signed-off-by: Glen Lee
Signed-off-by: Tony Cho
---
drivers/staging/wilc1000/linux_wlan.c |
From: Glen Lee
Use netdev private data member wilc instead of global variable wl.
Signed-off-by: Glen Lee
Signed-off-by: Tony Cho
---
drivers/staging/wilc1000/linux_wlan.c | 2 ++
1 file changed, 2 insertions(+)
diff --git a/drivers/staging/wilc1000/linux_wlan.c
b/drivers/staging/wilc1000/l
From: Glen Lee
Use netdev private data member wilc instead of global variable wl.
Signed-off-by: Glen Lee
Signed-off-by: Tony Cho
---
drivers/staging/wilc1000/linux_wlan.c | 3 ++-
1 file changed, 2 insertions(+), 1 deletion(-)
diff --git a/drivers/staging/wilc1000/linux_wlan.c
b/drivers/st
From: Glen Lee
Use netdev private data member wilc instead of global variable wl.
Signed-off-by: Glen Lee
Signed-off-by: Tony Cho
---
drivers/staging/wilc1000/wilc_wfi_cfgoperations.c | 2 ++
1 file changed, 2 insertions(+)
diff --git a/drivers/staging/wilc1000/wilc_wfi_cfgoperations.c
b/dr
From: Glen Lee
This patch uses netdev private data member wilc instead of global variabl wl.
Signed-off-by: Glen Lee
Signed-off-by: Tony Cho
---
drivers/staging/wilc1000/linux_wlan.c | 1 +
1 file changed, 1 insertion(+)
diff --git a/drivers/staging/wilc1000/linux_wlan.c
b/drivers/staging/w
From: Glen Lee
This patch changes function parameter type wilc with net_device and use netdev
private data member wilc instead of global variable wl.
Signed-off-by: Glen Lee
Signed-off-by: Tony Cho
---
drivers/staging/wilc1000/linux_wlan.c | 9 +++--
1 file changed, 7 insertions(+), 2 del
From: Glen Lee
This patch adds struct wilc to struct perInterface_wlan_t which is netdev
private data to access wilc via netdev_priv function and assigns wilc to
nic->wilc. The global variable wl will be replaced with netdev private data
member wilc step by step.
Signed-off-by: Glen Lee
Signed-
This patch replaces dma_alloc_coherent with the corresponding
managed interface.
Signed-off-by: Ronit Halder
---
drivers/staging/rts5208/rtsx.c | 6 ++
1 file changed, 2 insertions(+), 4 deletions(-)
diff --git a/drivers/staging/rts5208/rtsx.c b/drivers/staging/rts5208/rtsx.c
index b4e4543.
58 matches
Mail list logo