Fix sparse warning "warning: Variable length array is used." by using
kmalloc_array to allocate the required amount of memory instead and
kfree to deallocate memory after use.
Signed-off-by: Guru Das Srinagesh
---
v4:
- Changed kmalloc_array flags from GFP_KERNEL to GFP_ATOMIC
v3:
- Fixe
On 08-05-17, 18:05, kart...@techveda.org wrote:
> From: Karthik Tummala
>
> Fixed following checkpatch.pl warning:
> * WARNING: Prefer kcalloc over kzalloc with multiply
>
> Instead of specifying no.of bytes * size as argument
> in kzalloc, prefer kcalloc.
>
> Signed-off-by: Karthik Tummala
>
On 05/08/2017 02:41 AM, Philipp Zabel wrote:
Hi Steve,
On Wed, 2017-04-12 at 17:45 -0700, Steve Longerbeam wrote:
This patch adds a call to imx_media_fill_empty_mbus_fields() in the
*_try_fmt() functions at the sink pads, to set empty field order and
colorimetry parameters.
If the field orde
This patch fixes below checkpatch.pl kind of warnings:
CHECK: Alignment should match open parenthesis
Signed-off-by: Harinath Nampally
---
drivers/staging/iio/meter/ade7753.c | 55 ++---
1 file changed, 27 insertions(+), 28 deletions(-)
diff --git a/drivers/stagi
This patch fixes below kind of warnings:
WARNING: Symbolic permissions 'S_IXXX | S_IXXX' are not preferred.
Below errors are false positives:
ade7753.c:382: ERROR: Use 4 digit octal (0777) not decimal permissions
ade7753.c:386: ERROR: Use 4 digit octal (0777) not decimal permissions
Signed-off-by
This patchset is for fixes reported by checkpatch.pl
Please find the following related patches:
[PATCH 1/3] staging: iio: meter: Add the comment for mutex definition
[PATCH 2/3] staging: iio: meter: Fix the identations for proper alignments
[PATCH 3/3] staging: iio: meter: Replace symbolic permiss
This patch fixes below checkpatch.pl warning:
CHECK: struct mutex definition without comment
Signed-off-by: Harinath Nampally
---
drivers/staging/iio/meter/ade7753.c | 3 ++-
1 file changed, 2 insertions(+), 1 deletion(-)
diff --git a/drivers/staging/iio/meter/ade7753.c
b/drivers/staging/iio/m
Use usb_put_hdc to release hdc allocated by usb_create_hcd.
This is needed to handle sub-allocations and HCD sharing correctly.
Signed-off-by: Anton Bondarenko
---
drivers/staging/octeon-usb/octeon-hcd.c | 6 +++---
1 file changed, 3 insertions(+), 3 deletions(-)
diff --git a/drivers/staging/oc
Cleared up some errors and warnings in
drivers/staging/media/atomisp/i2c/ap1302.c
Signed-off-by: Mark Railton
---
drivers/staging/media/atomisp/i2c/ap1302.c | 83 ++
1 file changed, 50 insertions(+), 33 deletions(-)
diff --git a/drivers/staging/media/atomisp/i2c/ap13
On Mon, 2017-05-08 at 14:13 +0530, Surender Polsani wrote:
> according to coding style else is not generally
> useful after a break or return
>
> Signed-off-by: Surender Polsani
> ---
> drivers/staging/rtl8188eu/hal/rtl8188e_dm.c | 3 +--
> 1 file changed, 1 insertion(+), 2 deletions(-)
>
> dif
Dmitry,
On Mon, May 8, 2017 at 1:46 PM, John Stultz wrote:
> On Mon, May 8, 2017 at 1:43 PM, Dmitry Torokhov
> wrote:
>> If binder_get_thread() fails to give us a thread data, we should avoid
>> dereferencing a NULL pointer and return POLLERR instead.
>>
>> Signed-off-by: Dmitry Torokhov
>
> Pu
On Mon, May 8, 2017 at 1:43 PM, Dmitry Torokhov
wrote:
> If binder_get_thread() fails to give us a thread data, we should avoid
> dereferencing a NULL pointer and return POLLERR instead.
>
> Signed-off-by: Dmitry Torokhov
Pulling Todd Kjos in on this too.
-john
> ---
> drivers/android/binder.c
If binder_get_thread() fails to give us a thread data, we should avoid
dereferencing a NULL pointer and return POLLERR instead.
Signed-off-by: Dmitry Torokhov
---
drivers/android/binder.c | 12
1 file changed, 8 insertions(+), 4 deletions(-)
diff --git a/drivers/android/binder.c b/
From: Christoph Fanelsa
Fixed the checkpatch warnings (task #10 of eudyptula challenge) of prefered
octal permissions over symbolic ones
Signed-off-by: Christoph Fanelsa
Cc: Michael Hennerich
Cc: Jonathan Cameron
Cc: Hartmut Knaack
Cc: Peter Meerwald-Stadler
Cc: Greg Kroah-Hartman
Cc: lin
Fixed coding style issue
The following are the renames done to avoid camecase
#Configuration-> configuration
#BeaconPeriod -> beacon_period
#ATIMWindow -> atim_window
#DSConfig -> ds_config
#FHConfig -> fh_config
#
fixed warning line over 80 characters coding style issue
Signed-off-by: Jaya Durga
---
drivers/staging/rtl8712/ieee80211.c | 4 +++-
1 file changed, 3 insertions(+), 1 deletion(-)
diff --git a/drivers/staging/rtl8712/ieee80211.c
b/drivers/staging/rtl8712/ieee80211.c
index 42cb46c..a7152df 1006
this patch will fix one code style problem (ctx:WxE), space
prohibited before that
Signed-off-by: Remco Verhoef
---
.../staging/media/atomisp/platform/intel-mid/atomisp_gmin_platform.c| 2 +-
1 file changed, 1 insertion(+), 1 deletion(-)
diff --git
a/drivers/staging/media/atomisp/platform/
Fixing following checkpatch warnnings/errors:
ERROR: code indent should use tabs where possible
WARNING: please, no spaces at the start of a line
Signed-off-by: Gideon Sheril
---
Changed from previuos version:
- Fixed alignment issues
- Verified no new warning or errors appear after patch
On 9 May 2017 at 00:42, Andy Shevchenko wrote:
> On Sun, May 7, 2017 at 1:04 PM, Ian W MORRISON wrote:
>
> What are you trying to achieve by posting this patch over and over so often?
>
> Version 5 look pretty good to me. Just stop posting it again and wait
> when Greg takes it.
>
I made a se
On Sun, May 7, 2017 at 1:04 PM, Ian W MORRISON wrote:
> A kernel cloned from
> git://git.kernel.org/pub/scm/linux/kernel/git/torvalds/linux.git and
> built with the latest RTL8723BS driver included (CONFIG_RTL8723BS=m)
> fails when booting on an Intel Atom device with the RTL8723BS wifi
> chipset
On 05/06/2017 02:49 AM, Yisheng Xie wrote:
> From: Yisheng Xie
>
> After commit 9828282e33a0 ("staging: android: ion: Remove old platform
> support"), the document about devicetree of ion is no need anymore, so
> just remove it.
>
Acked-by: Laura Abbott
> Signed-off-by: Yisheng Xie
> ---
>
A re-positioned call to kfree() in
drivers/staging/rtl8723bs/os_dep/ioctl_cfg80211.c
causes a segmentation error. This patch removed the kfree() call.
Fixes 6557ddfec348 ("staging: rtl8723bs: Fix various errors in
os_dep/ioctl_cfg80211.c")
Signed-off-by: Ian W Morrison
Reviewed-by: Hans de Goed
This should just go to the gpio people and not staging.
The address is linux-g...@vger.kernel.org CC the Device Tree people for
answers the device tree questions: devicet...@vger.kernel.org
The subject should be: [PATCH] gpio: pca9570: add pca9570 driver
The patch has a line wrapping problem so
From: Karthik Tummala
Fixed following checkpatch.pl warning:
* WARNING: Prefer kcalloc over kzalloc with multiply
Instead of specifying no.of bytes * size as argument
in kzalloc, prefer kcalloc.
Signed-off-by: Karthik Tummala
Reviewed-by: Rui Miguel Silva
---
Changes for v2:
- Changed subjec
On Mon, May 08, 2017 at 12:31:52PM +0100, Chris Dew wrote:
> First, apologies, this is my first Linux kernel patch in at least 15 years.
> While I have spent a few hours reading various pieces of documentation about
> the Linux kernel development processes, I have probably missed some critical
> fi
First, apologies, this is my first Linux kernel patch in at least 15
years. While I have spent a few hours reading various pieces of
documentation about the Linux kernel development processes, I have
probably missed some critical files entirely.
Also, there are probably a few coding issues in
Hi Karthik,
Thanks for the patch.
On Sat, May 06, 2017 at 07:44:10PM +0530, kart...@techveda.org wrote:
From: Karthik Tummala
Fixed following checkpatch.pl warning:
* WARNING: Prefer kcalloc over kzalloc with multiply
Instead of specifying no.of bytes * size as arugment
in kzalloc, prefer kca
On 05/08/2017 11:36 AM, Philipp Zabel wrote:
> Hi Hans,
>
> On Mon, 2017-05-08 at 10:27 +0200, Hans Verkuil wrote:
>> Hi Philipp,
>>
>> Sorry for the very long delay, but I finally had some time to think about
>> this.
>
> Thank you for your thoughts.
>
>> On 04/06/2017 03:55 PM, Philipp Zabel
Hi Steve,
On Wed, 2017-04-12 at 17:45 -0700, Steve Longerbeam wrote:
> This patch adds a call to imx_media_fill_empty_mbus_fields() in the
> *_try_fmt() functions at the sink pads, to set empty field order and
> colorimetry parameters.
>
> If the field order is set to ANY, choose the currently se
Hi Hans,
On Mon, 2017-05-08 at 10:27 +0200, Hans Verkuil wrote:
> Hi Philipp,
>
> Sorry for the very long delay, but I finally had some time to think about
> this.
Thank you for your thoughts.
> On 04/06/2017 03:55 PM, Philipp Zabel wrote:
> > If the the field order is set to ANY in set_fmt, c
according to coding style else is not generally
useful after a break or return
Signed-off-by: Surender Polsani
---
drivers/staging/rtl8188eu/hal/rtl8188e_dm.c | 3 +--
1 file changed, 1 insertion(+), 2 deletions(-)
diff --git a/drivers/staging/rtl8188eu/hal/rtl8188e_dm.c
b/drivers/staging/rtl8
Hi Philipp,
Sorry for the very long delay, but I finally had some time to think about this.
On 04/06/2017 03:55 PM, Philipp Zabel wrote:
> If the the field order is set to ANY in set_fmt, choose the currently
> set field order. If the colorspace is set to DEFAULT, choose the current
> colorspace.
On Sun, May 7, 2017 at 9:53 PM, Geert Uytterhoeven wrote:
> With gcc 4.1.2:
>
> drivers/staging/ccree/ssi_hash.c:1990: error: unknown field
> ‘template_ahash’ specified in initializer
> drivers/staging/ccree/ssi_hash.c:1991: error: unknown field ‘init’
> specified in initializer
> dr
33 matches
Mail list logo
