Re: [PATCH v2 8/8] staging: iio: isl29018: move out of staging

2016-10-15 Thread Brian Masney
On Sat, Oct 15, 2016 at 04:15:57PM +0100, Jonathan Cameron wrote: > On 10/10/16 08:20, Brian Masney wrote: > > Move ISL29018/ISL29023/ISL29035 driver out of staging into mainline. > > > > Signed-off-by: Brian Masney > Applied to the togreg branch of iio.git and pushed out as testing > for the aut

[staging:staging-testing 275/275] /tmp/ccbrt8LJ.s:560: Error: missing immediate expression at operand 1 -- `dsb '

2016-10-15 Thread kbuild test robot
tree: https://git.kernel.org/pub/scm/linux/kernel/git/gregkh/staging.git staging-testing head: 07823993a309809c5107de94a892274bc984b92b commit: 07823993a309809c5107de94a892274bc984b92b [275/275] staging/vchi: Remove dependency on CONFIG_BROKEN. config: arm64-allmodconfig (attached as .config)

Re: [PATCH] Staging: greybus: uart: constify tty_port_operations structure

2016-10-15 Thread David Lin
On Sat, Oct 15, 2016 at 2:23 AM, Johan Hovold wrote: > > [ +CC: staging list (for real this time) ] > > On Sat, Oct 15, 2016 at 12:28:01PM +0530, Bhumika Goyal wrote: > > Declare tty_port_operations structure as const as it is only stored in > > the ops field of a tty_port structure which is a fie

Re: [PATCH v2 8/8] staging: iio: isl29018: move out of staging

2016-10-15 Thread Jonathan Cameron
On 10/10/16 08:20, Brian Masney wrote: > Move ISL29018/ISL29023/ISL29035 driver out of staging into mainline. > > Signed-off-by: Brian Masney Applied to the togreg branch of iio.git and pushed out as testing for the autobuilders to play with it. Thanks Brian for taking this one on and your hard

Re: [PATCH v2 7/8] staging: iio: isl29018: add ABI documentation for infrared suppression

2016-10-15 Thread Jonathan Cameron
On 10/10/16 21:57, Jonathan Cameron wrote: > On 10/10/16 08:20, Brian Masney wrote: >> Add ABI documentation from the ISL29018 Data Sheet (FN6619.4, Oct 8, >> 2012) about the infrared suppression that can be controlled >> with the proximity_on_chip_ambient_infrared_suppression sysfs attribute. >> >

Re: [PATCH v2] Staging:ks7010:eap_packet: fixed 80 character line limit coding style issue

2016-10-15 Thread Greg KH
On Sat, Oct 15, 2016 at 12:49:31AM +0530, Nadim Almas wrote: > So please tall me how to do? I have already done so, but you seem to be ignoring my suggestions for some reason. Also please do not post with html, the mailing lists reject it. I recommend going and learning C better and getting expe

Re: [PATCH 2/2] Staging: greybus: uart: Use gbphy_dev->dev instead of bundle->dev

2016-10-15 Thread Johan Hovold
On Wed, Oct 12, 2016 at 08:02:22AM +0530, Viresh Kumar wrote: > Some of the print messages are using the incorrect device pointer, fix > them. > > Signed-off-by: Viresh Kumar Acked-by: Johan Hovold ___ devel mailing list de...@linuxdriverproject.org h

Re: [PATCH 1/2] Staging: greybus: gpio: Use gbphy_dev->dev instead of bundle->dev

2016-10-15 Thread Johan Hovold
On Wed, Oct 12, 2016 at 08:02:21AM +0530, Viresh Kumar wrote: > Some of the print messages are using the incorrect device pointer, fix > them. > > Signed-off-by: Viresh Kumar Acked-by: Johan Hovold ___ devel mailing list de...@linuxdriverproject.org h

Re: [PATCH v2] RFC: staging: greybus: shape up greybus GPIO

2016-10-15 Thread Johan Hovold
On Mon, Oct 10, 2016 at 10:39:32AM +0200, Linus Walleij wrote: > Greybus GPIO seems to reimplement the already existing generic > gpiolib irqchip. Also use gpiochip_get_data(). Also use > devm_gpiochip_add_data(). > > Cc: Viresh Kumar > Cc: Axel Haslam > Cc: Johan Hovold > Cc: Sandeep Patil >

[PATCH] staging: unisys: visorbus: add missing include

2016-10-15 Thread Laurent Navet
this removes lots of sparse warnings : warning: symbol 'visorchannel_destroy' was not declared. Should it be static? warning: symbol 'visorchannel_get_physaddr' was not declared. Should it be static? warning: symbol 'visorchannel_get_nbytes' was not declared. Should it be static? warning: symbol '

Re: [PATCH] Staging: greybus: uart: constify tty_port_operations structure

2016-10-15 Thread Johan Hovold
[ +CC: staging list (for real this time) ] On Sat, Oct 15, 2016 at 12:28:01PM +0530, Bhumika Goyal wrote: > Declare tty_port_operations structure as const as it is only stored in > the ops field of a tty_port structure which is a field of a gb_tty > structure. The ops field is of type const struct

Re: [patch] greybus: fix a leak on error in gb_module_create()

2016-10-15 Thread Johan Hovold
On Fri, Oct 14, 2016 at 10:18:21PM +0300, Dan Carpenter wrote: > We should release ->interfaces[0] as well. Indeed. Thanks for the fix. > Fixes: b15d97d77017 ("greybus: core: add module abstraction") > Signed-off-by: Dan Carpenter Acked-by: Johan Hovold Johan _