[PATCH v2 0/2] staging: unisys: visorbus: Don't check for more than PAGE_SIZE

2017-01-03 Thread David Kershner
This series is just a resubmission of the last 2 patches from our previous series. It just removes extraneous PAGE_SIZE checks. Changes since v1: -- Remove references to procfs since we are only dealing with sysfs David Binder (2): staging: unisys: visorbus: visorchipset.c: Don't check

[PATCH v2 2/2] staging: unisys: visorbus: Don't check for more than PAGE_SIZE length in visorbus

2017-01-03 Thread David Kershner
From: David Binder Since a sysfs entry is allocated 1 page of memory (4096 bytes) by default, there is no need to enforce this limit in the driver. This patch corrects visorbus/visorbus_main.c. Reported-by: Greg Kroah-Hartman Signed-off-by: David Binder Signed-off-by: David Kershner --- driv

Re: [PATCH v2 00/19] i.MX Media Driver

2017-01-03 Thread Fabio Estevam
On Tue, Jan 3, 2017 at 6:57 PM, Steve Longerbeam wrote: > In version 2 (no functional changes): > > - removed patch "gpio: pca953x: Add optional reset gpio control", it > has been submitted separately. > - fixed some whitespace errors. > - added a few missing Signed-off-by's. Tested the series

Re: [PATCH 00/20] i.MX Media Driver

2017-01-03 Thread Steve Longerbeam
Hi Fabio, On 01/02/2017 01:09 PM, Fabio Estevam wrote: Hi Steve, On Thu, Dec 29, 2016 at 8:27 PM, Steve Longerbeam wrote: This is a media driver for video capture on i.MX. Refer to Documentation/media/v4l-drivers/imx.rst for example capture pipelines on SabreSD, SabreAuto, and SabreLite ref

Re: [PATCH 11/20] media: Add i.MX media core driver

2017-01-03 Thread Rob Herring
On Thu, Dec 29, 2016 at 02:27:26PM -0800, Steve Longerbeam wrote: > Add the core media driver for i.MX SOC. > > Signed-off-by: Steve Longerbeam > --- > Documentation/devicetree/bindings/media/imx.txt | 205 + > Documentation/media/v4l-drivers/imx.rst | 429 ++ > drivers/s

Re: [PATCH 2/2] isdn: i4l: move active-isdn drivers to staging

2017-01-03 Thread Arnd Bergmann
On Tuesday, January 3, 2017 4:24:36 PM CET Greg Kroah-Hartman wrote: > On Wed, Mar 02, 2016 at 08:06:46PM +0100, Arnd Bergmann wrote: > > The icn, act2000 and pcbit drivers are all for very old hardware, > > and it is highly unlikely that anyone is actually still using them > > on modern kernels, i

RE: [PATCH 7/8] staging: fsl-dpaa2/eth: Add TODO file

2017-01-03 Thread Stuart Yoder
> -Original Message- > From: Greg KH [mailto:gre...@linuxfoundation.org] > Sent: Tuesday, January 03, 2017 10:48 AM > To: Stuart Yoder > Cc: de...@driverdev.osuosl.org; a...@arndb.de; Roy Pledge > ; Alexandru Marginean > ; linux-ker...@vger.kernel.org; ag...@suse.de; > Bogdan Hamciuc >

Re: [PATCH 2/2] isdn: i4l: move active-isdn drivers to staging

2017-01-03 Thread Paul Bolle
On Tue, 2017-01-03 at 22:19 +0100, Arnd Bergmann wrote: > Sounds good to me. My original series contained four more patches that > I did not post again after there was some concern[1] that we did not > come to a conclusion on: > > isdn: gigaset: remove i4l code Let me repeat that I'm fine with a

Re: [PATCH 2/2] isdn: i4l: move active-isdn drivers to staging

2017-01-03 Thread Arnd Bergmann
On Tuesday, January 3, 2017 10:54:19 PM CET Paul Bolle wrote: > On Tue, 2017-01-03 at 22:19 +0100, Arnd Bergmann wrote: > > isdn: move isdnhdlc out of i4l > > isdn: i4l: move hisax driver to staging > > isdn: move i4l to staging > > > > I can post those as well, at least I think the first two are

Re: [PATCH 2/2] isdn: i4l: move active-isdn drivers to staging

2017-01-03 Thread Paul Bolle
On Tue, 2017-01-03 at 23:25 +0100, Arnd Bergmann wrote: > As far as I'm concerned, we are totally fine as long as there exists a > longterm supported kernel that has i4l in drivers/staging. Or in drivers/isdn, right? Paul Bolle ___ devel mailing list d

Re: [PATCH 2/2] isdn: i4l: move active-isdn drivers to staging

2017-01-03 Thread Arnd Bergmann
On Tuesday, January 3, 2017 11:57:36 PM CET Paul Bolle wrote: > On Tue, 2017-01-03 at 23:25 +0100, Arnd Bergmann wrote: > > As far as I'm concerned, we are totally fine as long as there exists a > > longterm supported kernel that has i4l in drivers/staging. > > Or in drivers/isdn, right? Right, I

Re: Designated initializers, struct randomization and addressing?

2017-01-03 Thread Kees Cook
On Tue, Dec 20, 2016 at 9:29 AM, Joe Perches wrote: > On Fri, 2016-12-16 at 17:00 -0800, Kees Cook wrote: >> Prepare to mark sensitive kernel structures for randomization by making > sure they're using designated initializers. > > About the designated initializer patches, > which by themselves are

Re: Designated initializers, struct randomization and addressing?

2017-01-03 Thread Bruce Korb
As a tangential party, I am a bit curious: does the randomization plugin result in a compact structure? I ask because I know many/most programmers don't bother with it and so doing so ought to make the data more compact. On Tue, Jan 3, 2017 at 3:47 PM, Kees Cook wrote: >> how is the code to be v

[PATCH v3 3/4] staging: greybus: loopback: convert loopback to use generic async operations

2017-01-03 Thread Bryan O'Donoghue
Loopback has its own internal method for tracking and timing out asynchronous operations however previous patches make it possible to use functionality provided by operation.c to do this instead. Using the code in operation.c means we can completely subtract the timer, the work-queue, the kref and

[PATCH v3 2/4] staging: greybus: operation: add private data with get/set accessors

2017-01-03 Thread Bryan O'Donoghue
Asynchronous operation completion handler's lives are made easier if there is a generic pointer that can store private data associated with the operation. This patch adds a pointer field to operation.h and get/set methods to access that pointer. Signed-off-by: Bryan O'Donoghue --- drivers/stagin

[PATCH v3 1/4] staging: greybus: operation: add generic asynchronous timeout operation support

2017-01-03 Thread Bryan O'Donoghue
This patch adds a generic mechanism for handling timeouts of asynchronous operations to operation.c. After doing a gb_operation_request_send() it schedules a delayed worker. When the delayed worker's timer expires the worker runs and does a gb_operation_cancel(). A gb_operation_cancel() operation w

[PATCH v3 0/4] drivers/staging/greybus: add async operations

2017-01-03 Thread Bryan O'Donoghue
V3: Picard: "Let's make sure history never forgets the name ... Enterprise." Drop gb_operation_cancel_async. The sync version works just as well - Johan V2: Scotty : "What are you standing around for - do you not know a jailbreak when you see one" ? Making sure to run format-patch against the rig

[PATCH v3 4/4] staging: greybus: loopback: convert to use msecs internally

2017-01-03 Thread Bryan O'Donoghue
The API presented by operation.h expects milliseconds to be passed in. This patch drops the conversion from user-input microseconds-to-jiffies and from jiffies-to-milliseconds and instead converts directly from microseconds-to-milliseconds. The new minimum timeout will be one millisecond as opposed

[PATCH net-next] net/hyperv: remove use of VLAN_TAG_PRESENT

2017-01-03 Thread Michał Mirosław
Signed-off-by: Michał Mirosław --- drivers/net/hyperv/hyperv_net.h | 2 +- drivers/net/hyperv/netvsc_drv.c | 13 ++--- drivers/net/hyperv/rndis_filter.c | 4 ++-- 3 files changed, 9 insertions(+), 10 deletions(-) diff --git a/drivers/net/hyperv/hyperv_net.h b/drivers/net/hyperv/hyp

Re: Designated initializers, struct randomization and addressing?

2017-01-03 Thread Kees Cook
On Tue, Jan 3, 2017 at 3:55 PM, Bruce Korb wrote: > On Tue, Jan 3, 2017 at 3:47 PM, Kees Cook wrote: >>> how is the code to be verified so that >>> any use of things like offsetof and any >>> address/indexing is not impacted? > > As a tangential party, I am a bit curious: does the randomization >

[PATCH] Staging: fbtft: hx8340bn: Style fix, octal not decimal permissions

2017-01-03 Thread Derek Robson
Fixed coding style issue, found with Checkpatch.pl changed permissions to octal style. --- drivers/staging/fbtft/fb_hx8340bn.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/drivers/staging/fbtft/fb_hx8340bn.c b/drivers/staging/fbtft/fb_hx8340bn.c index 9970ed74bb38..c24331cc

[PATCH] staging:iio:addac Fixes Alignment should match open parenthesis

2017-01-03 Thread Scott Matheina
Fixes style issue where Alignment doesn't match open parenthesis Signed-off-by: Scott Matheina --- drivers/staging/iio/addac/adt7316-i2c.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/drivers/staging/iio/addac/adt7316-i2c.c b/drivers/staging/iio/addac/adt7316-i2c.c index

Re: Designated initializers, struct randomization and addressing?

2017-01-03 Thread Julia Lawall
On Tue, 3 Jan 2017, Kees Cook wrote: > On Tue, Dec 20, 2016 at 9:29 AM, Joe Perches wrote: > > On Fri, 2016-12-16 at 17:00 -0800, Kees Cook wrote: > >> Prepare to mark sensitive kernel structures for randomization by making > > sure they're using designated initializers. > > > > About the desig

Re: Designated initializers, struct randomization and addressing?

2017-01-03 Thread Kees Cook
On Tue, Jan 3, 2017 at 10:27 PM, Julia Lawall wrote: > > > On Tue, 3 Jan 2017, Kees Cook wrote: > >> On Tue, Dec 20, 2016 at 9:29 AM, Joe Perches wrote: >> > On Fri, 2016-12-16 at 17:00 -0800, Kees Cook wrote: >> >> Prepare to mark sensitive kernel structures for randomization by making >> > sure

Re: [PATCH] staging: gdm724x: add forced casts in endian converters to fix sparse warnings

2017-01-03 Thread Eric S. Stone
On Tue, Jan 03, 2017 at 04:30:58PM +0100, Greg KH wrote: > On Wed, Dec 28, 2016 at 10:08:43PM -0800, Eric S. Stone wrote: > > The modified functions do explicit endian checking and conversion. The > > added forced casts fix these sparse warnings: > > > > CHECK drivers/staging/gdm724x/gdm_endian.

Re: [PATCH] staging: greybus: arche-apb-ctrl: fix unused warnings on resume/suspend

2017-01-03 Thread Vaibhav Hiremath
On Sat, Dec 17, 2016 at 6:03 AM, Jérémy Lefaure wrote: > When CONFIG_PM_SLEEP is disabled, SIMPLE_DEV_PM_OPS does not use > arche_apb_ctrl_resume and arche_apb_ctrl_suspend functions: > > drivers/staging/greybus/arche-apb-ctrl.c:478:12: warning: > ‘arche_apb_ctrl_resume’ defined but not used [-Wun

<    1   2