Re: [PATCH] staging: media: zoran: fix trailing whitespaces

2019-04-07 Thread Dan Carpenter
On Mon, Apr 08, 2019 at 01:35:30AM -0300, Arthur Moraes do Lago wrote: > There were several form feeds(^L) in the file, this patch removes them > to conform with the coding style. > > Signed-off-by: Arthur Moraes do Lago > --- > drivers/staging/media/zoran/videocodec.h | 10 +- > 1 file

[PATCH] staging: media: zoran: fix trailing whitespaces

2019-04-07 Thread Arthur Moraes do Lago
There were several form feeds(^L) in the file, this patch removes them to conform with the coding style. Signed-off-by: Arthur Moraes do Lago --- Made a mistake while typing recipients on the last e-mail and it was not sent to the kernel list. Re-sending now with correct address, sorry for the ha

[PATCH] staging: media: zoran: fix trailing whitespaces

2019-04-07 Thread Arthur Moraes do Lago
There were several form feeds(^L) in the file, this patch removes them to conform with the coding style. Signed-off-by: Arthur Moraes do Lago --- drivers/staging/media/zoran/videocodec.h | 10 +- 1 file changed, 5 insertions(+), 5 deletions(-) diff --git a/drivers/staging/media/zoran/vi

Re: [PATCH 03/16] staging: m57621-mmc: delete driver from the tree.

2019-04-07 Thread NeilBrown
On Wed, Apr 03 2019, George Hilliard wrote: > On Tue, Apr 2, 2019 at 3:45 PM Christian Lütke-Stetzkamp > wrote: >> There are two other larger differences that I found during my >> work. One is that drivers/mmc/host/mtk-sd.c has much more features, >> like voltage and clock handling and some suppo

[PATCH] media: cedrus: Fix initialization order

2019-04-07 Thread Jernej Skrabec
Currently, MEDIA_IOC_G_TOPOLOGY ioctl on cedrus fails due to incorrect initialization order. Fix that by moving video_register_device() before v4l2_m2m_register_media_controller() and while at it, fix error path. Reported-by: Jonas Karlman Signed-off-by: Jernej Skrabec --- drivers/staging/media

user provided data for ion buffer

2019-04-07 Thread Skidanov, Alexey
Hi, I would like to add 64 bit user data to ion allocated buffer. This data may be assigned during buffer allocation (passed as one of the ioctl parameters) and may be queried by fd representing the buffer or pfn. Looks like may be useful in many use cases. Please, let me know what do you thi

Re: [PATCH 3/4] staging: iio: ad9832: use clock framework for clock reference

2019-04-07 Thread Jonathan Cameron
On Mon, 1 Apr 2019 11:37:45 -0300 Marcelo Schmitt wrote: > Previously external clock were set through platform_data struct. > Now device uses clk struct defined in include/linux/clk.h to handle > external clock source. > It also removes mclk from platform_data struct. > > Signed-off-by: Marcelo

Re: [PATCH 1/4] staging: iio: ad9832: organize includes

2019-04-07 Thread Jonathan Cameron
On Mon, 1 Apr 2019 11:37:10 -0300 Marcelo Schmitt wrote: > Organize includes to list them in lexicographic order. > > Signed-off-by: Marcelo Schmitt There are a few headers in there that I'd expect to be unnecessary once cleanup on the driver is finished, but good to get them in a nice order to

Re: [PATCH 2/4] staging: iio: ad9832: add SPDX identifier

2019-04-07 Thread Jonathan Cameron
On Tue, 2 Apr 2019 11:40:00 -0300 Marcelo Schmitt wrote: > On 04/01, Greg KH wrote: > > On Mon, Apr 01, 2019 at 08:36:50PM +0530, Mukesh Ojha wrote: > > > > > > On 4/1/2019 8:07 PM, Marcelo Schmitt wrote: > > > > Add SPDX identifier of GPL-2.0 for the ad9832 driver. > > > > > > > > Signed-o

Re: [PATCH 3/3] dt-bindings: iio: adc: Add AD7616 ADC documentation

2019-04-07 Thread Jonathan Cameron
On Sat, 06 Apr 2019 01:06:39 -0500 Rob Herring wrote: > On Tue, 2 Apr 2019 16:18:41 +0300, Beniamin Bia wrote: > > Document support for AD7616 Analog to Digital Converter. > > > > Signed-off-by: Beniamin Bia > > --- > > Documentation/devicetree/bindings/iio/adc/adi,ad7606.txt | 1 + > > 1 file

Re: [PATCH 2/3] iio: adc: ad7616: Add support for AD7616 ADC

2019-04-07 Thread Jonathan Cameron
On Tue, 2 Apr 2019 16:18:40 +0300 Beniamin Bia wrote: > The AD7616 is a 12-bit ADC with 16 channels. > > The AD7616 can be configured to work in hardware mode by controlling it via > gpio pins and read data via spi. No support for software mode yet, but it > is a work in progress. > > This devi

Re: [PATCH 1/3] iio: adc: ad7606: Move oversampling options in chip info and rework *_avail attributes

2019-04-07 Thread Jonathan Cameron
On Tue, 2 Apr 2019 16:18:39 +0300 Beniamin Bia wrote: > From: Stefan Popa > > Available oversampling ratios and scales can be shown by calling a > common ad7606_show_avail function which takes as parameters the array > which stores the values, together with the size of the array. > > Oversampl

Re: [PATCH] staging: iio: frequency: ad9834: Remove unnecessary parentheses

2019-04-07 Thread Jonathan Cameron
On Tue, 2 Apr 2019 17:18:54 -0300 Melissa Wen wrote: > Remove unneeded parentheses around the arguments of ||. This reduces > clutter and code behave in the same way. Change suggested by > checkpatch.pl. > > CHECK: Unnecessary parentheses around 'st->devid == ID_AD9833' > CHECK: Unnecessary pare