[PATCH] staging: sm750fb: rename macro roundedDiv to rounded_div in ddk750_chip.c

2016-10-01 Thread Moshe Green
Rename CamelCased macro roundedDiv to rounded_div. This issue was found by checkpatch.pl Signed-off-by: Moshe Green --- drivers/staging/sm750fb/ddk750_chip.c | 6 +++--- 1 file changed, 3 insertions(+), 3 deletions(-) diff --git a/drivers/staging/sm750fb/ddk750_chip.c

[PATCH] [media] bcm2048: Remove FSF mailing address

2016-10-01 Thread Wayne Porter
FSF address changes, checkpatch recommends removing it Signed-off-by: Wayne Porter --- drivers/staging/media/bcm2048/radio-bcm2048.h | 5 - 1 file changed, 5 deletions(-) diff --git a/drivers/staging/media/bcm2048/radio-bcm2048.h

[PATCH] [media] v4l: omap4iss: Fix using BIT macro

2016-10-01 Thread Wayne Porter
Checks found by checkpatch Signed-off-by: Wayne Porter --- drivers/staging/media/omap4iss/iss_regs.h | 76 +++ 1 file changed, 38 insertions(+), 38 deletions(-) diff --git a/drivers/staging/media/omap4iss/iss_regs.h

[PATCH] [media] : Removing warnings caught by checkpatch.pl

2016-10-01 Thread Harman Kalra
Removing warnings caught by checkpatch.pl Signed-off-by: Harman Kalra --- drivers/staging/media/omap4iss/iss_video.c |8 1 file changed, 4 insertions(+), 4 deletions(-) diff --git a/drivers/staging/media/omap4iss/iss_video.c

Re: [PATCH v3 5/5] staging: iio: isl29018: check if the chip is in a suspended state

2016-10-01 Thread Jonathan Cameron
On 01/10/16 16:30, Brian Masney wrote: > On Sat, Oct 01, 2016 at 02:59:49PM +0100, Jonathan Cameron wrote: >> On 27/09/16 01:20, Brian Masney wrote: >>> Add a check to isl29018_write_raw() to ensure that the chip is not in a >>> suspended state. This makes the code consistent with what is present

Re: [PATCH v3 5/5] staging: iio: isl29018: check if the chip is in a suspended state

2016-10-01 Thread Brian Masney
On Sat, Oct 01, 2016 at 02:59:49PM +0100, Jonathan Cameron wrote: > On 27/09/16 01:20, Brian Masney wrote: > > Add a check to isl29018_write_raw() to ensure that the chip is not in a > > suspended state. This makes the code consistent with what is present > > in isl29018_read_raw(). > > > >

Re: [PATCH v3 5/5] staging: iio: isl29018: check if the chip is in a suspended state

2016-10-01 Thread Jonathan Cameron
On 27/09/16 01:20, Brian Masney wrote: > Add a check to isl29018_write_raw() to ensure that the chip is not in a > suspended state. This makes the code consistent with what is present > in isl29018_read_raw(). > > Signed-off-by: Brian Masney Applied to the togreg branch of

Re: [PATCH v3 4/5] staging: iio: isl29018: change isl29018_read_raw() to only have one exit point

2016-10-01 Thread Jonathan Cameron
On 27/09/16 01:20, Brian Masney wrote: > When the chip is in a suspended state, isl29018_read_raw() will return > -EBUSY. Change the function so that it only has a single exit point. > > Signed-off-by: Brian Masney Applied. Thanks, Jonathan > --- >

Re: [PATCH v3 3/5] staging: iio: isl29018: fixed race condition in in_illuminance_scale_available_show()

2016-10-01 Thread Jonathan Cameron
On 27/09/16 01:20, Brian Masney wrote: > in_illuminance_scale_available_show() references the > isl29018_chip->int_time variable in three places inside a for loop. > The value of the int_time variable can be updated by the > isl29018_set_integration_time() function, which is called by the >

Re: [PATCH v3 2/5] staging: iio: isl29018: use IIO_DEVICE_ATTR_{RO, RW} macros

2016-10-01 Thread Jonathan Cameron
On 27/09/16 01:20, Brian Masney wrote: > Use the IIO_DEVICE_ATTR_RO and IIO_DEVICE_ATTR_RW macros to > create the device attributes. > > Signed-off-by: Brian Masney Applied to the togreg branch of iio.git - initially pushed out as testing for the autobuilders to play with

Re: [PATCH v3 1/5] include: linux: iio: add IIO_ATTR_{RO, WO, RW} and IIO_DEVICE_ATTR_{RO, WO, RW} macros

2016-10-01 Thread Jonathan Cameron
On 27/09/16 01:20, Brian Masney wrote: > Add new macros: IIO_ATTR_RO, IIO_ATTR_WO, IIO_ATTR_RW, > IIO_DEVICE_ATTR_RO, IIO_DEVICE_ATTR_WO and IIO_DEVICE_ATTR_RW to reduce > the amount of boiler plate code that is needed for creating new > attributes. This mimics the *_RO, *_WO, and *_RW macros that

[PATCH] Staging: fbtft: Fix bug in fbtft-core

2016-10-01 Thread Ksenija Stanojevic
Commit 367e8560e8d7a62d96e9b1d644028a3816e04206 introduced a bug in fbtft-core where fps is always 0, this is because variable update_time is not assigned correctly. Signed-off-by: Ksenija Stanojevic --- drivers/staging/fbtft/fbtft-core.c | 4 ++-- 1 file changed,