[staging:staging-testing] BUILD SUCCESS 3fce0276f98a5e4263735f86c5c35361fafa7a59

2020-03-19 Thread kbuild test robot
malta_kvm_defconfig pariscallnoconfig parisc allyesconfig pariscgeneric-32bit_defconfig pariscgeneric-64bit_defconfig x86_64 randconfig-a001-20200319 x86_64 randconfig

Re: [PATCH v2 03/10] media: adv748x: reduce amount of code for bitwise modifications of device registers

2020-03-19 Thread Laurent Pinchart
Hi Alex, Thank you for the patch. On Thu, Mar 19, 2020 at 06:41:53PM +0100, Alex Riesen wrote: > The regmap provides a convenient utility for this. > > Signed-off-by: Alexander Riesen > --- > drivers/media/i2c/adv748x/adv748x-core.c | 6 ++ > drivers/media/i2c/adv748x/adv748x.h | 15

Re: [PATCH v2 01/10] media: adv748x: fix end-of-line terminators in diagnostic statements

2020-03-19 Thread Laurent Pinchart
Hi Alex, Thank you for the patch. On Thu, Mar 19, 2020 at 06:41:43PM +0100, Alex Riesen wrote: > Signed-off-by: Alexander Riesen Reviewed-by: Laurent Pinchart > --- > drivers/media/i2c/adv748x/adv748x-core.c | 24 > drivers/media/i2c/adv748x/adv748x-csi2.c | 2 +- >

Re: [PATCH v2 07/10] dt-bindings: adv748x: add information about serial audio interface (I2S/TDM)

2020-03-19 Thread Laurent Pinchart
Hi Alex, Thank you for the patch. On Thu, Mar 19, 2020 at 06:42:36PM +0100, Alex Riesen wrote: > As the driver has some support for the audio interface of the device, > the bindings file should mention it. While at it, how about converting the bindings to YAML ? :-) It can of course be done on

Re: [PATCH v2 02/10] media: adv748x: include everything adv748x.h needs into the file

2020-03-19 Thread Laurent Pinchart
Hi Alexander, Thank you for the patch. On Thu, Mar 19, 2020 at 06:41:48PM +0100, Alex Riesen wrote: > To follow the established practice of not depending on others to > pull everything in. > > Signed-off-by: Alexander Riesen Good idea. While at it, could you include "adv748x.h" as the very

[PATCH v2 09/10] media: adv748x: add support for log_status ioctl

2020-03-19 Thread Alex Riesen
The logged information provides insights about cable connection and the state of the HDMI decoder. It is very useful when debugging hardware problems in environments without easy access to the connectors. This change adds a device-specific wrapper for register block read, because some of the

[PATCH v2 10/10] media: adv748x: allow the HDMI sub-device to accept EDID

2020-03-19 Thread Alex Riesen
This makes it possible to load a EDID reported by the device with v4l2-ctl utility: vdev=/dev/$(grep -l '^adv748x.*hdmi$' /sys/class/video4linux/v4l-subdev*/name |cut -d/ -f5-5) v4l2-ctl -d $vdev --set-edid=pad=0,file=/etc/adv7482.edid Signed-off-by: Alexander Riesen ---

[PATCH v2 07/10] dt-bindings: adv748x: add information about serial audio interface (I2S/TDM)

2020-03-19 Thread Alex Riesen
As the driver has some support for the audio interface of the device, the bindings file should mention it. Reviewed-by: Rob Herring Signed-off-by: Alexander Riesen --- .../devicetree/bindings/media/i2c/adv748x.txt| 16 +++- 1 file changed, 15 insertions(+), 1 deletion(-) diff

[PATCH v2 06/10] media: adv748x: only activate DAI if it is described in device tree

2020-03-19 Thread Alex Riesen
To avoid setting it up even if the hardware is not actually connected to anything physically. Besides, the bindings explicitly notes that port definitions are "optional if they are not connected to anything at the hardware level". Signed-off-by: Alexander Riesen ---

[PATCH v2 05/10] media: adv748x: add support for HDMI audio

2020-03-19 Thread Alex Riesen
This adds an implemention of SoC DAI driver which provides access to the I2S port of the device. Signed-off-by: Alexander Riesen --- drivers/media/i2c/adv748x/Makefile | 3 +- drivers/media/i2c/adv748x/adv748x-core.c | 9 +- drivers/media/i2c/adv748x/adv748x-dai.c | 256

[PATCH v2 04/10] media: adv748x: add definitions for audio output related registers

2020-03-19 Thread Alex Riesen
Signed-off-by: Alexander Riesen --- drivers/media/i2c/adv748x/adv748x.h | 32 + 1 file changed, 32 insertions(+) diff --git a/drivers/media/i2c/adv748x/adv748x.h b/drivers/media/i2c/adv748x/adv748x.h index c5245464fffc..7bc1bb0b3756 100644 ---

[PATCH v2 00/10] media: adv748x: add support for HDMI audio

2020-03-19 Thread Alex Riesen
This adds minimal support for accessing the HDMI audio provided through the I2S port available on ADV7481 and ADV7482 decoder devices by ADI. The port carries audio signal from the decoded HDMI stream. Currently, the driver only supports I2S in TDM, 8 channels a 24bit at 48kHz. Furthermore, only

[PATCH v2 01/10] media: adv748x: fix end-of-line terminators in diagnostic statements

2020-03-19 Thread Alex Riesen
Signed-off-by: Alexander Riesen --- drivers/media/i2c/adv748x/adv748x-core.c | 24 drivers/media/i2c/adv748x/adv748x-csi2.c | 2 +- 2 files changed, 13 insertions(+), 13 deletions(-) diff --git a/drivers/media/i2c/adv748x/adv748x-core.c

[PATCH v2 02/10] media: adv748x: include everything adv748x.h needs into the file

2020-03-19 Thread Alex Riesen
To follow the established practice of not depending on others to pull everything in. Signed-off-by: Alexander Riesen --- drivers/media/i2c/adv748x/adv748x-afe.c | 2 -- drivers/media/i2c/adv748x/adv748x-core.c | 2 -- drivers/media/i2c/adv748x/adv748x-csi2.c | 2 --

[PATCH v2 03/10] media: adv748x: reduce amount of code for bitwise modifications of device registers

2020-03-19 Thread Alex Riesen
The regmap provides a convenient utility for this. Signed-off-by: Alexander Riesen --- drivers/media/i2c/adv748x/adv748x-core.c | 6 ++ drivers/media/i2c/adv748x/adv748x.h | 15 --- 2 files changed, 18 insertions(+), 3 deletions(-) diff --git

[driver-core:driver-core-testing] BUILD SUCCESS 99917e37b9e78b96af304ddeb4f9b82a5948bbd9

2020-03-19 Thread kbuild test robot
_defconfig parisc allyesconfig i386 randconfig-a003-20200319 i386 randconfig-a001-20200319 x86_64 randconfig-a001-20200319 x86_64 randconfig-a002-20200319 i386 randconfig-a002-20200

Re: [PATCH RESEND] staging: vc04_services: Use scnprintf() for avoiding potential buffer overflow

2020-03-19 Thread Greg Kroah-Hartman
On Thu, Mar 19, 2020 at 05:13:00PM +0100, Takashi Iwai wrote: > Since snprintf() returns the would-be-output size instead of the > actual output size, the succeeding calls may go beyond the given > buffer limit. Fix it by replacing with scnprintf(). > > Reviewed-by: Nicolas Saenz Julienne > Cc:

[PATCH] staging: mt7621-pci: don't return if get gpio fails

2020-03-19 Thread Sergio Paracuellos
In some platforms gpio's are not used for reset but for other purposes. Because of that when we try to get them are valid gpio's but are already assigned to do other function. To avoid those kind of problems in those platforms just notice the fail in the kernel but continue doing normal boot.

[PATCH RESEND] staging: vc04_services: Use scnprintf() for avoiding potential buffer overflow

2020-03-19 Thread Takashi Iwai
Since snprintf() returns the would-be-output size instead of the actual output size, the succeeding calls may go beyond the given buffer limit. Fix it by replacing with scnprintf(). Reviewed-by: Nicolas Saenz Julienne Cc: Greg Kroah-Hartman Cc: bcm-kernel-feedback-l...@broadcom.com Cc:

Re: [kbuild-all] Re: [staging:staging-testing 222/222] drivers/ide/ide-gd.c:362:10: sparse: warning: Initializer entry defined twice

2020-03-19 Thread Philip Li
On Wed, Mar 18, 2020 at 07:53:07AM +0100, Greg Kroah-Hartman wrote: > On Wed, Mar 18, 2020 at 02:42:31PM +0800, Philip Li wrote: > > On Wed, Mar 18, 2020 at 07:17:40AM +0100, Greg Kroah-Hartman wrote: > > > On Wed, Mar 18, 2020 at 02:12:43PM +0800, Philip Li wrote: > > > > On Wed, Mar 18, 2020 at

Re: [PATCH] staging: exfat: Fix checkpatch.pl camelcase issues

2020-03-19 Thread Greg KH
On Thu, Mar 19, 2020 at 10:31:05AM -0400, Aravind Ceyardass wrote: > > > On 3/19/20 10:12 AM, Greg KH wrote: > > On Thu, Mar 19, 2020 at 10:06:47AM -0400, Aravind Ceyardass wrote: > >> Fix ffsCamelCase function names and mixed case enums > >> > >> Signed-off-by: Aravind Ceyardass > >> --- > >>

[driver-core:debugfs_remove_return_value] BUILD REGRESSION c5b434b51b2daaacdd0c69736631eaead95d9111

2020-03-19 Thread kbuild test robot
allyesconfig mips fuloong2e_defconfig pariscallnoconfig parisc allyesconfig pariscgeneric-32bit_defconfig pariscgeneric-64bit_defconfig i386 randconfig-a003-20200319

[driver-core:debugfs_cleanup] BUILD SUCCESS 75ae458d1cd8fe76666aa83dd7bf4f656424272a

2020-03-19 Thread kbuild test robot
randconfig-a003-20200319 i386 randconfig-a001-20200319 x86_64 randconfig-a001-20200319 x86_64 randconfig-a002-20200319 i386 randconfig-a002-20200319 x86_64 randconfig-a003-20200319 riscv

Re: [PATCH] staging: exfat: Fix checkpatch.pl camelcase issues

2020-03-19 Thread Aravind Ceyardass
On 3/19/20 10:12 AM, Greg KH wrote: > On Thu, Mar 19, 2020 at 10:06:47AM -0400, Aravind Ceyardass wrote: >> Fix ffsCamelCase function names and mixed case enums >> >> Signed-off-by: Aravind Ceyardass >> --- >> drivers/staging/exfat/TODO | 1 - >> drivers/staging/exfat/exfat.h

Re: [PATCH v4 0/6] staging: mt7621-pci: re-do reset boot process

2020-03-19 Thread Greg Ungerer
Hi Sergio, On 19/3/20 11:43 pm, Sergio Paracuellos wrote: On Thu, Mar 19, 2020 at 1:42 PM Greg Ungerer wrote: On 14/3/20 6:09 am, Sergio Paracuellos wrote: Some time ago Greg Ungerer reported some random hangs using the staging mt7621-pci driver: See: *

Re: [PATCH] staging: exfat: Fix checkpatch.pl camelcase issues

2020-03-19 Thread Matthew Wilcox
On Thu, Mar 19, 2020 at 10:06:47AM -0400, Aravind Ceyardass wrote: > Fix ffsCamelCase function names and mixed case enums This driver is now gone from staging in -next; please review the code in fs/exfat instead. ___ devel mailing list

Re: [PATCH v2] staging: rtl8192u: Corrects 'Avoid CamelCase' for variables

2020-03-19 Thread Greg KH
On Wed, Mar 18, 2020 at 09:12:05PM +, Camylla Goncalves Cantanheide wrote: > The variables of function setKey triggered a 'Avoid CamelCase' > warning from checkpatch.pl. This patch renames these > variables to correct this warning. > > Signed-off-by: Camylla Goncalves Cantanheide > --- >

Re: [PATCH] staging: exfat: Fix checkpatch.pl camelcase issues

2020-03-19 Thread Greg KH
On Thu, Mar 19, 2020 at 10:06:47AM -0400, Aravind Ceyardass wrote: > Fix ffsCamelCase function names and mixed case enums > > Signed-off-by: Aravind Ceyardass > --- > drivers/staging/exfat/TODO | 1 - > drivers/staging/exfat/exfat.h | 12 +- >

[PATCH] staging: exfat: Fix checkpatch.pl camelcase issues

2020-03-19 Thread Aravind Ceyardass
Fix ffsCamelCase function names and mixed case enums Signed-off-by: Aravind Ceyardass --- drivers/staging/exfat/TODO | 1 - drivers/staging/exfat/exfat.h | 12 +- drivers/staging/exfat/exfat_super.c | 222 ++-- 3 files changed, 117 insertions(+), 118

Re: [PATCH v4 0/6] staging: mt7621-pci: re-do reset boot process

2020-03-19 Thread Sergio Paracuellos
Sorry the mail was sent while I was still writing :) On Thu, Mar 19, 2020 at 2:43 PM Sergio Paracuellos wrote: > > Hi Greg, > > On Thu, Mar 19, 2020 at 1:42 PM Greg Ungerer wrote: > > > > Hi Sergio, > > > > On 14/3/20 6:09 am, Sergio Paracuellos wrote: > > > Some time ago Greg Ungerer reported

Re: [PATCH v4 0/6] staging: mt7621-pci: re-do reset boot process

2020-03-19 Thread Sergio Paracuellos
Hi Greg, On Thu, Mar 19, 2020 at 1:42 PM Greg Ungerer wrote: > > Hi Sergio, > > On 14/3/20 6:09 am, Sergio Paracuellos wrote: > > Some time ago Greg Ungerer reported some random hangs using > > the staging mt7621-pci driver: > > > > See: > > * > >

Re: [PATCH v4 0/6] staging: mt7621-pci: re-do reset boot process

2020-03-19 Thread Greg Ungerer
Hi Sergio, On 14/3/20 6:09 am, Sergio Paracuellos wrote: Some time ago Greg Ungerer reported some random hangs using the staging mt7621-pci driver: See: * http://driverdev.linuxdriverproject.org/pipermail/driverdev-devel/2019-June/134947.html Try to fix that is the main motivation of this

[PATCH] staging: mt7621-pci: fix register to set up virtual bridges

2020-03-19 Thread Sergio Paracuellos
Instead of being using PCI Configuration and Status Register to set up virtual bridges we are using CONFIG_ADDR Register which is wrong. Hence, set the correct value. Fixes: 9a5e71a68d20 ("staging: mt7621-pci: simplify 'mt7621_pcie_init_virtual_bridges' function") Signed-off-by: Sergio