On 16 September 2016 at 07:40, Laura Abbott wrote:
> On 09/15/2016 08:57 AM, Arnd Bergmann wrote:
>>
>> The newly added Hi6220 Ion code fails to build when the ION_OF helpers
>> are not present:
>>
>> drivers/staging/android/ion/hisilicon/hi6220_ion.o: In function
>> `hi6220_ion_remove':
>> hi6220
On 16 September 2016 at 07:40, Laura Abbott wrote:
> On 09/14/2016 07:20 PM, Wei Yongjun wrote:
>>
>> From: Wei Yongjun
>>
>> In case of error, the function ion_device_create() returns ERR_PTR() and
>> never returns NULL. The NULL test in the return value check should be
>> replaced with IS_ERR()
On 09/15/2016 08:57 AM, Arnd Bergmann wrote:
The newly added Hi6220 Ion code fails to build when the ION_OF helpers
are not present:
drivers/staging/android/ion/hisilicon/hi6220_ion.o: In function
`hi6220_ion_remove':
hi6220_ion.c:(.text.hi6220_ion_remove+0x4c): undefined reference to
`ion_des
On 09/14/2016 07:20 PM, Wei Yongjun wrote:
From: Wei Yongjun
In case of error, the function ion_device_create() returns ERR_PTR() and
never returns NULL. The NULL test in the return value check should be
replaced with IS_ERR().
Acked-by: Laura Abbott
Signed-off-by: Wei Yongjun
---
drive
Fix the following warning types:
- line length
- block comment line * prefix
- trailing */ on a separate line
found by the checkpatch.pl tool in multiple block comments.
Fix a single spelling error in a comment.
Signed-off-by: Moshe Green
---
drivers/staging/sm750fb/ddk750_chip.c | 49 ++
Fix multiple line length warnings found by the checkpatch.pl tool
in ddk750_chip.c.
Signed-off-by: Moshe Green
---
drivers/staging/sm750fb/ddk750_chip.c | 23 +++
1 file changed, 15 insertions(+), 8 deletions(-)
diff --git a/drivers/staging/sm750fb/ddk750_chip.c
b/drivers/s
On Thu, 2016-09-15 at 19:22 +0200, Sergio Paracuellos wrote:
> That's true. It seems that checkpatch script is getting a false
> positive. Just trying to do checkpatch happy. Sorry for inconvenience.
Keeping the style consistent instead of taking the simplest path
to quiet the checkpatch messages
Signed-off-by: Jon Eyolfson
---
drivers/staging/wlan-ng/cfg80211.c | 8
1 file changed, 4 insertions(+), 4 deletions(-)
diff --git a/drivers/staging/wlan-ng/cfg80211.c
b/drivers/staging/wlan-ng/cfg80211.c
index a36e40d..457f176 100644
--- a/drivers/staging/wlan-ng/cfg80211.c
+++ b/dri
On Mon, Sep 12, 2016 at 01:17:25PM +0200, Greg KH wrote:
> On Sun, Sep 04, 2016 at 09:04:10PM +0300, Moshe Green wrote:
> > Fix the following warning types:
> > - line length
> > - block comment line * prefix
> > - trailing */ on a separate line
> > found by the checkpatch.pl tool in multiple bl
On Mon, Sep 12, 2016 at 01:16:35PM +0200, Greg KH wrote:
> On Sun, Sep 04, 2016 at 09:03:27PM +0300, Moshe Green wrote:
> > Fix multiple line length warnings found by the checkpatch.pl tool
> > in ddk750_chip.c.
> >
> > Signed-off-by: Moshe Green
> > ---
> > drivers/staging/sm750fb/ddk750_chip.c
This patch fixes the checkpatch.pl warning:
WARNING: Comparisons should place the constant on the right side of the test
Signed-off-by: Sergio Paracuellos
---
drivers/staging/rts5208/rtsx_card.c | 18 +-
1 file changed, 9 insertions(+), 9 deletions(-)
diff --git a/drivers/stagi
On Wednesday 14 September 2016 10:44 AM, Dilger, Andreas wrote:
On Sep 12, 2016, at 04:27, Greg KH wrote:
On Fri, Sep 09, 2016 at 08:50:35PM +0530, Nayeemahmed Badebade wrote:
Added __acquires / __releases sparse locking annotations
to lock_res_and_lock and unlock_res_and_lock functions in
On Thu, Sep 15, 2016 at 12:18:56PM +0200, Richard Groux wrote:
> Minor warnings spotted by checkpatch.pl in lustre
> Remove unnecessary space before function pointer arguments.
>
> Signed-off-by: Richard Groux
> ---
> drivers/staging/lustre/lustre/include/lustre_net.h | 46
> +++
That's true. It seems that checkpatch script is getting a false
positive. Just trying to do checkpatch happy. Sorry for inconvenience.
I'll be careful with Changelog stuff from now. Thanks for advice.
Cheers,
Sergio Paracuellos
El 2016年09月15日 a las 16:51, Greg KH escribió:
On Thu, Sep
The newly added Hi6220 Ion code fails to build when the ION_OF helpers
are not present:
drivers/staging/android/ion/hisilicon/hi6220_ion.o: In function
`hi6220_ion_remove':
hi6220_ion.c:(.text.hi6220_ion_remove+0x4c): undefined reference to
`ion_destroy_platform_data'
drivers/staging/android/ion
We get 2 warnings when building kernel with W=1:
drivers/staging/lustre/lustre/obdclass/linux/linux-module.c:413:5: warning: no
previous prototype for 'class_procfs_init' [-Wmissing-prototypes]
drivers/staging/lustre/lustre/obdclass/linux/linux-module.c:449:5: warning: no
previous prototype for
> -Original Message-
> From: Wei Yongjun [mailto:weiyj...@gmail.com]
> Sent: Wednesday, September 14, 2016 9:27 PM
> To: J . German Rivera ; Stuart Yoder
> ; Greg Kroah-
> Hartman ; German Rivera ;
> Itai Katz
>
> Cc: Wei Yongjun ; linux-ker...@vger.kernel.org;
> de...@driverdev.osuos
> -Original Message-
> From: Wei Yongjun [mailto:weiyj...@gmail.com]
> Sent: Wednesday, September 14, 2016 9:27 PM
> To: J . German Rivera ; Stuart Yoder
> ; Greg Kroah-
> Hartman ; German Rivera ;
> Itai Katz
>
> Cc: Wei Yongjun ; linux-ker...@vger.kernel.org;
> de...@driverdev.osuos
On Thu, Sep 15, 2016 at 09:51:27PM +0800, Baoyou Xie wrote:
> We get 2 warnings when building kernel with W=1:
Don't do that! :)
>
> drivers/staging/lustre/lustre/obdclass/linux/linux-module.c:413:5: warning:
> no previous prototype for 'class_procfs_init' [-Wmissing-prototypes]
> drivers/stag
On Thu, Sep 15, 2016 at 04:29:54PM +0200, Sergio Paracuellos wrote:
> Signed-off-by: Sergio Paracuellos
> ---
> drivers/staging/rts5208/rtsx_card.c | 4 ++--
> 1 file changed, 2 insertions(+), 2 deletions(-)
>
> diff --git a/drivers/staging/rts5208/rtsx_card.c
> b/drivers/staging/rts5208/rtsx_c
From: Andrey Shvetsov
This patch implements a workaround for a DIM2 issue where the device
macro, in case the asynchronous channel sends data (to MOST), sporadically
duplicates the synchronous blocks with a size of half the synchronous DBR
buffer.
The patch monitors the size of the unused asynch
From: Andrey Shvetsov
The DIM2 HDM checks the parameters passed to the function
configure_channel() and adapts the buffer sizes of the configuration
according to the hardware limitation. This patch is needed to init
the HAL layer with the correct values.
Signed-off-by: Andrey Shvetsov
Signed-of
From: Andrey Shvetsov
This patch removes the macro ENABLE_HDM_TEST, which was needed while
doing DIM2-HAL simulations. But these are not conducted anymore.
Signed-off-by: Andrey Shvetsov
Signed-off-by: Christian Gromm
---
drivers/staging/most/hdm-dim2/dim2_hdm.c | 35 -
From: Andrey Shvetsov
This patch renames the function dim_service_irq. It subsitutes the name
with the more suitable identifier dim_service_ahb_int_irq.
Signed-off-by: Andrey Shvetsov
Signed-off-by: Christian Gromm
---
drivers/staging/most/hdm-dim2/dim2_hal.c | 2 +-
drivers/staging/most/hdm-
From: Andrey Shvetsov
This patch fixes the DIM2 IP interrupt names.
Signed-off-by: Andrey Shvetsov
Signed-off-by: Christian Gromm
---
drivers/staging/most/hdm-dim2/dim2_hdm.c | 6 +++---
1 file changed, 3 insertions(+), 3 deletions(-)
diff --git a/drivers/staging/most/hdm-dim2/dim2_hdm.c
b/
From: Andrey Shvetsov
As the DBR memory is allocated in units of DBR_BLOCK_SIZE, it is more
efficient to configure the DIM2 IP to use whole blocks of the DBR memory.
This patch ceils the DBR memory size used by the DIM2 IP for async/ctrl
channels.
Signed-off-by: Andrey Shvetsov
Signed-off-by: C
From: Andrey Shvetsov
This patch increases the size of the asynchronous and control DBR buffers
in the Tx path to twice the max. message size. This patch is needed to
increase the throughput for big messages.
Signed-off-by: Andrey Shvetsov
Signed-off-by: Christian Gromm
---
drivers/staging/mo
This patch set is needed to fix the issues of the DIM2 module of the
MOST driver.
Andrey Shvetsov (11):
staging: most: hdm-dim2: fix channel configuration
staging: most: hdm-dim2: remove macro ENABLE_HDM_TEST
staging: most: hdm-dim2: relocate variable declarations
staging: most: hdm-dim2:
From: Andrey Shvetsov
This patch eliminates nested variable declarations by putting them at
the beginning of the function in order to flatten the code.
Signed-off-by: Andrey Shvetsov
Signed-off-by: Christian Gromm
---
drivers/staging/most/hdm-dim2/dim2_hal.c | 19 ---
1 file c
From: Andrey Shvetsov
This patch removes unnecessary clearing of a pending mlb_int from the
service routine for ahb interrupt.
Signed-off-by: Andrey Shvetsov
Signed-off-by: Christian Gromm
---
drivers/staging/most/hdm-dim2/dim2_hal.c | 4
1 file changed, 4 deletions(-)
diff --git a/driv
From: Andrey Shvetsov
This patch removes the member irq_ahb0 of the structure dim2_hdm, since
its job is suitably done by a simple auto variable.
Signed-off-by: Andrey Shvetsov
Signed-off-by: Christian Gromm
---
drivers/staging/most/hdm-dim2/dim2_hdm.c | 12 +---
1 file changed, 5 ins
From: Andrey Shvetsov
This patch removes the logging of the returned error code of function
dim2_probe as it is done by the calling function.
Signed-off-by: Andrey Shvetsov
Signed-off-by: Christian Gromm
---
drivers/staging/most/hdm-dim2/dim2_hdm.c | 3 +--
1 file changed, 1 insertion(+), 2 d
We get 2 warnings when building kernel with W=1:
drivers/staging/lustre/lustre/obdclass/linux/linux-module.c:413:5: warning: no
previous prototype for 'class_procfs_init' [-Wmissing-prototypes]
drivers/staging/lustre/lustre/obdclass/linux/linux-module.c:449:5: warning: no
previous prototype for
Minor warnings spotted by checkpatch.pl in lustre
Remove unnecessary space before function pointer arguments.
Signed-off-by: Richard Groux
---
drivers/staging/lustre/lustre/include/lustre_net.h | 46 +++---
1 file changed, 23 insertions(+), 23 deletions(-)
diff --git a/drivers/s
From: Wei Yongjun
Remove including that don't need it.
Signed-off-by: Wei Yongjun
---
drivers/staging/ks7010/ks_wlan_net.c | 1 -
1 file changed, 1 deletion(-)
diff --git a/drivers/staging/ks7010/ks_wlan_net.c
b/drivers/staging/ks7010/ks_wlan_net.c
index a35325e..c17c45d 100644
--- a/driver
From: Wei Yongjun
Fix a double lock bug in SetPSModeWorkItemCallback().
Fixes: 5c2ba8b85e35 ("rtl8712: pwrctrl_priv: Replace semaphore lock
with mutex")
Signed-off-by: Wei Yongjun
---
drivers/staging/rtl8712/rtl871x_pwrctrl.c | 2 +-
1 file changed, 1 insertion(+), 1 deletion(-)
diff --git a/
36 matches
Mail list logo