Re: [PATCH v2] v4l: vsp1: Fix vsp1_regs.h license header

+/* SPDX-License-Identifier: GPL-2.0+ */ > > While you are changing this line, I believe the correct format is > to use a '//' comment. > > i.e.: > > // SPDX-License-Identifier: GPL-2.0+ Not for C header files, only for C source files. Gr{oetje,eeting}s, Geert --

[PATCH v2] [media] v4l: rcar_fdp1: Change platform dependency to ARCH_RENESAS

H_SHMOBILE on ARM and ARM64 in the near future. Signed-off-by: Geert Uytterhoeven <geert+rene...@glider.be> Acked-by: Arnd Bergmann <a...@arndb.de> Acked-by: Laurent Pinchart <laurent.pinch...@ideasonboard.com> Reviewed-by: Simon Horman <horms+rene...@verge.net.au>

[PATCH v4] media: Remove depends on HAS_DMA in case of platform dependency

by several individual drivers. Signed-off-by: Geert Uytterhoeven <ge...@linux-m68k.org> Reviewed-by: Mark Brown <broo...@kernel.org> Acked-by: Robin Murphy <robin.mur...@arm.com> --- From: kbuild test robot <l...@intel.com> tree/branch: https://git.kernel.org/pub/scm/linux/kernel

Re: [PATCH v2 2/2] ARM: dts: r8a7740: Add CEU0

Hi Jacopo, On Thu, Apr 26, 2018 at 8:24 PM, Jacopo Mondi <jacopo+rene...@jmondi.org> wrote: > Describe CEU0 peripheral for Renesas R-Mobile A1 R8A7740 Soc. > > Reported-by: Geert Uytterhoeven <ge...@glider.be> > Signed-off-by: Jacopo Mondi <jacopo+rene...@jmondi.

Re: [PATCH] media: vsp1: cleanup a false positive warning

gt;brx) { > + if (brx != pipe->brx || !pipe->brx) { > struct vsp1_entity *released_brx = NULL; > > /* Release our BRx if we have one. */ Gr{oetje,eeting}s, Geert -- Geert Uytterhoeven -- There's lots of Lin

Re: [PATCH v14 03/33] rcar-vin: add Gen3 devicetree bindings documentation

clock-lanes = <0>; > + data-lanes = <1>; > +remote-endpoint = <_txb>; > +}; > +}; > + > +

Re: [PATCH] rcar-vin: fix null pointer dereference in rvin_group_get()

21 @@ static int rvin_group_get(struct rvin_dev *vin) > > static void rvin_group_put(struct rvin_dev *vin) > { > - mutex_lock(>group->lock); > + struct rvin_group *group = vin->group; Gr{oetje,eeting}s, Geert -- Geert Uytterhoeven -- There's lots

Re: [PATCH/RFC 7/8] ARM: shmobile: Remove the ARCH_SHMOBILE Kconfig symbol

(Reducing/enhancing CC list) On Fri, Apr 20, 2018 at 3:28 PM, Geert Uytterhoeven <geert+rene...@glider.be> wrote: > All drivers for Renesas ARM SoCs have gained proper ARCH_RENESAS > platform dependencies. Hence finish the conversion from ARCH_SHMOBILE > to ARCH_RENESAS for Ren

Re: [PATCH 1/7] asm-generic, media: allow COMPILE_TEST with virt_to_bus

ate_dev_mem_ptr(phys_addr_t > phys, void *addr) > } > #endif > > -#ifdef CONFIG_VIRT_TO_BUS > +#if defined(CONFIG_VIRT_TO_BUS) || defined(CONFIG_COMPILE_TEST) > #ifndef virt_to_bus > static inline unsigned long virt_to_bus(void *address) > { Gr

Re: [PATCH 3/8] [media] v4l: rcar_fdp1: Change platform dependency to ARCH_RENESAS

Hi Laurent, On Sun, Apr 22, 2018 at 10:46 AM, Laurent Pinchart <laurent.pinch...@ideasonboard.com> wrote: > On Saturday, 21 April 2018 11:07:11 EEST Laurent Pinchart wrote: >> On Friday, 20 April 2018 16:28:29 EEST Geert Uytterhoeven wrote: >> > The Renesas Fine Display

Re: [PATCH 2/3] v4l: rcar_fdp1: Enable compilation on Gen2 platforms

(!ARM64 && !VIDEO_RENESAS_FCP) || VIDEO_RENESAS_FCP s/!ARM64/ARCH_RCAR_GEN2/? > select VIDEOBUF2_DMA_CONTIG > select V4L2_MEM2MEM_DEV Gr{oetje,eeting}s, Geert -- Geert Uytterhoeven -- There's lots of Linux beyond ia32 -- ge

Re: [PATCH 0/8] arm: renesas: Change platform dependency to ARCH_RENESAS

Hi Mark, On Fri, Apr 20, 2018 at 6:48 PM, Mark Brown <broo...@kernel.org> wrote: > On Fri, Apr 20, 2018 at 03:28:26PM +0200, Geert Uytterhoeven wrote: >> The first 6 patches can be applied independently by subsystem >> maintainers. >> The last two patches de

[PATCH/RFC 7/8] ARM: shmobile: Remove the ARCH_SHMOBILE Kconfig symbol

All drivers for Renesas ARM SoCs have gained proper ARCH_RENESAS platform dependencies. Hence finish the conversion from ARCH_SHMOBILE to ARCH_RENESAS for Renesas 32-bit ARM SoCs, as started by commit 9b5ba0df4ea4f940 ("ARM: shmobile: Introduce ARCH_RENESAS"). Signed-off-by: Geert Uy

[PATCH/RFC 8/8] arm64: renesas: Remove the ARCH_SHMOBILE Kconfig symbol

Signed-off-by: Geert Uytterhoeven <geert+rene...@glider.be> --- This depends on the driver patches in this series, hence the RFC. JFTR, after this, the following symbols for drivers supporting only Renesas SuperH "SH-Mobile" SoCs can no longer be selected:

[PATCH 0/8] arm: renesas: Change platform dependency to ARCH_RENESAS

. Removing the ARCH_SHMOBILE Kconfig symbols on ARM and ARM64. The first 6 patches can be applied independently by subsystem maintainers. The last two patches depend on the first 6 patches, and are thus marked RFC. Thanks for your comments! Geert Uytterhoeven (8): arm: shmobile: Change platform

[PATCH 1/8] arm: shmobile: Change platform dependency to ARCH_RENESAS

Since commit 9b5ba0df4ea4f940 ("ARM: shmobile: Introduce ARCH_RENESAS") is ARCH_RENESAS a more appropriate platform dependency than the legacy ARCH_SHMOBILE, hence use the former. This will allow to drop ARCH_SHMOBILE on ARM in the near future. Signed-off-by: Geert Uytterhoeven &

[PATCH 4/8] sh_eth: Change platform check to CONFIG_ARCH_RENESAS

ow to drop ARCH_SHMOBILE on ARM and ARM64 in the near future. Signed-off-by: Geert Uytterhoeven <geert+rene...@glider.be> --- drivers/net/ethernet/renesas/sh_eth.h | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/drivers/net/ethernet/renesas/sh_eth.h b/drivers/net/ethernet/

[PATCH 2/8] dmaengine: shdmac: Change platform check to CONFIG_ARCH_RENESAS

re support for Renesas ARM SoCs was added. Instead of blindly changing all the #ifdefs, switch the main code block in sh_dmae_probe() to IS_ENABLED(), as this allows to remove all the remaining #ifdefs. This will allow to drop ARCH_SHMOBILE on ARM in the near future. Signed-off-by: Geert Uytterhoeven &

[PATCH 5/8] staging: emxx_udc: Change platform dependency to ARCH_RENESAS

Signed-off-by: Geert Uytterhoeven <geert+rene...@glider.be> --- drivers/staging/emxx_udc/Kconfig | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/drivers/staging/emxx_udc/Kconfig b/drivers/staging/emxx_udc/Kconfig index d7577096fb25ae7a..e50e722183648c55 100644 --- a/drivers/

[PATCH 6/8] ASoC: sh: Update menu title and platform dependency

he legacy ARCH_SHMOBILE, hence use the former. Renesas SuperH SH-Mobile SoCs are still covered by the SUPERH dependency. This will allow to drop ARCH_SHMOBILE on ARM and ARM64 in the near future. Signed-off-by: Geert Uytterhoeven <geert+rene...@glider.be> --- sound/soc/sh/Kconfig | 4 ++--

[PATCH 3/8] [media] v4l: rcar_fdp1: Change platform dependency to ARCH_RENESAS

H_SHMOBILE on ARM and ARM64 in the near future. Signed-off-by: Geert Uytterhoeven <geert+rene...@glider.be> --- drivers/media/platform/Kconfig | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/drivers/media/platform/Kconfig b/drivers/media/platform/Kconfig index

[PATCH v3 18/20] spi: Remove depends on HAS_DMA in case of platform dependency

ncies keep their dependencies on HAS_DMA, to prevent compiling subsystems or drivers that cannot work anyway. This simplifies the dependencies, and allows to improve compile-testing. Signed-off-by: Geert Uytterhoeven <ge...@linux-m68k.org> Reviewed-by: Mark Brown <broo...@kernel.org>

[PATCH v3 13/20] mmc: Remove depends on HAS_DMA in case of platform dependency

ncies keep their dependencies on HAS_DMA, to prevent compiling subsystems or drivers that cannot work anyway. This simplifies the dependencies, and allows to improve compile-testing. Signed-off-by: Geert Uytterhoeven <ge...@linux-m68k.org> Reviewed-by: Mark Brown <broo...@kernel.org>

[PATCH v3 19/20] staging: vc04_services: Remove depends on HAS_DMA in case of platform dependency

ncies keep their dependencies on HAS_DMA, to prevent compiling subsystems or drivers that cannot work anyway. This simplifies the dependencies, and allows to improve compile-testing. Signed-off-by: Geert Uytterhoeven <ge...@linux-m68k.org> Reviewed-by: Mark Brown <broo...@kernel.org>

[PATCH v3 05/20] firewire: Remove depends on HAS_DMA in case of platform dependency

ncies keep their dependencies on HAS_DMA, to prevent compiling subsystems or drivers that cannot work anyway. This simplifies the dependencies, and allows to improve compile-testing. Signed-off-by: Geert Uytterhoeven <ge...@linux-m68k.org> Reviewed-by: Mark Brown <broo...@kernel.org>

[PATCH v3 12/20] media: Remove depends on HAS_DMA in case of platform dependency

by several individual drivers. Signed-off-by: Geert Uytterhoeven <ge...@linux-m68k.org> Reviewed-by: Mark Brown <broo...@kernel.org> Acked-by: Robin Murphy <robin.mur...@arm.com> --- v3: - Rebase to v4.17-rc1, - Handle new VIDEO_RENESAS_CEU symbol, v2: - Add Reviewed-by, Acked-by,

[PATCH v3 07/20] i2c: Remove depends on HAS_DMA in case of platform dependency

ncies keep their dependencies on HAS_DMA, to prevent compiling subsystems or drivers that cannot work anyway. This simplifies the dependencies, and allows to improve compile-testing. Signed-off-by: Geert Uytterhoeven <ge...@linux-m68k.org> Reviewed-by: Mark Brown <broo...@kernel.org>

[PATCH v3 14/20] mtd: Remove depends on HAS_DMA in case of platform dependency

ncies keep their dependencies on HAS_DMA, to prevent compiling subsystems or drivers that cannot work anyway. This simplifies the dependencies, and allows to improve compile-testing. Signed-off-by: Geert Uytterhoeven <ge...@linux-m68k.org> Reviewed-by: Mark Brown <broo...@kernel.org>

[PATCH v3 06/20] fpga: Remove depends on HAS_DMA in case of platform dependency

ncies keep their dependencies on HAS_DMA, to prevent compiling subsystems or drivers that cannot work anyway. This simplifies the dependencies, and allows to improve compile-testing. Signed-off-by: Geert Uytterhoeven <ge...@linux-m68k.org> Reviewed-by: Mark Brown <broo...@kernel.org>

[PATCH v3 16/20] remoteproc: Remove depends on HAS_DMA in case of platform dependency

ncies keep their dependencies on HAS_DMA, to prevent compiling subsystems or drivers that cannot work anyway. This simplifies the dependencies, and allows to improve compile-testing. Signed-off-by: Geert Uytterhoeven <ge...@linux-m68k.org> Reviewed-by: Mark Brown <broo...@kernel.org>

[PATCH v3 00/20] Allow compile-testing NO_DMA (drivers)

ile-tested with allmodconfig and allyesconfig for m68k/sun3, and has received attention from the kbuild test robot. Thanks for applying! Geert Uytterhoeven (20): ASoC: Remove depends on HAS_DMA in case of platform dependency ata: Remove depends on HAS_DMA in case of platform dependency cryp

[PATCH v3 02/20] ata: Remove depends on HAS_DMA in case of platform dependency

ncies keep their dependencies on HAS_DMA, to prevent compiling subsystems or drivers that cannot work anyway. This simplifies the dependencies, and allows to improve compile-testing. Signed-off-by: Geert Uytterhoeven <ge...@linux-m68k.org> Reviewed-by: Mark Brown <broo...@kernel.org>

[PATCH v3 11/20] mailbox: Remove depends on HAS_DMA in case of platform dependency

ncies keep their dependencies on HAS_DMA, to prevent compiling subsystems or drivers that cannot work anyway. This simplifies the dependencies, and allows to improve compile-testing. Signed-off-by: Geert Uytterhoeven <ge...@linux-m68k.org> Reviewed-by: Mark Brown <broo...@kernel.org>

[PATCH v3 03/20] crypto: Remove depends on HAS_DMA in case of platform dependency

ncies keep their dependencies on HAS_DMA, to prevent compiling subsystems or drivers that cannot work anyway. This simplifies the dependencies, and allows to improve compile-testing. Signed-off-by: Geert Uytterhoeven <ge...@linux-m68k.org> Reviewed-by: Mark Brown <broo...@kernel.org>

[PATCH v3 09/20] iommu: Remove depends on HAS_DMA in case of platform dependency

ncies keep their dependencies on HAS_DMA, to prevent compiling subsystems or drivers that cannot work anyway. This simplifies the dependencies, and allows to improve compile-testing. Signed-off-by: Geert Uytterhoeven <ge...@linux-m68k.org> Reviewed-by: Mark Brown <broo...@kernel.org>

[PATCH v3 01/20] ASoC: Remove depends on HAS_DMA in case of platform dependency

_SOC_STORM and/or SND_SOC_APQ8016_SBC. Signed-off-by: Geert Uytterhoeven <ge...@linux-m68k.org> Reviewed-by: Mark Brown <broo...@kernel.org> Acked-by: Robin Murphy <robin.mur...@arm.com> Acked-by: Mark Brown <broo...@kernel.org> --- v3: - Add Acked-by, - Rebase t

[PATCH v3 20/20] usb: Remove depends on HAS_DMA in case of platform dependency

ncies keep their dependencies on HAS_DMA, to prevent compiling subsystems or drivers that cannot work anyway. This simplifies the dependencies, and allows to improve compile-testing. Signed-off-by: Geert Uytterhoeven <ge...@linux-m68k.org> Reviewed-by: Mark Brown <broo...@kernel.org>

[PATCH v3 10/20] lightnvm: Remove depends on HAS_DMA in case of platform dependency

ncies keep their dependencies on HAS_DMA, to prevent compiling subsystems or drivers that cannot work anyway. This simplifies the dependencies, and allows to improve compile-testing. Signed-off-by: Geert Uytterhoeven <ge...@linux-m68k.org> Reviewed-by: Mark Brown <broo...@kernel.org>

[PATCH v3 04/20] fbdev: Remove depends on HAS_DMA in case of platform dependency

ncies keep their dependencies on HAS_DMA, to prevent compiling subsystems or drivers that cannot work anyway. This simplifies the dependencies, and allows to improve compile-testing. Signed-off-by: Geert Uytterhoeven <ge...@linux-m68k.org> Reviewed-by: Mark Brown <broo...@kernel.org>

[PATCH v3 15/20] net: Remove depends on HAS_DMA in case of platform dependency

ncies keep their dependencies on HAS_DMA, to prevent compiling subsystems or drivers that cannot work anyway. This simplifies the dependencies, and allows to improve compile-testing. Signed-off-by: Geert Uytterhoeven <ge...@linux-m68k.org> Reviewed-by: Mark Brown <broo...@kernel.org>

[PATCH v3 08/20] iio: adc: Remove depends on HAS_DMA in case of platform dependency

ncies keep their dependencies on HAS_DMA, to prevent compiling subsystems or drivers that cannot work anyway. This simplifies the dependencies, and allows to improve compile-testing. Signed-off-by: Geert Uytterhoeven <ge...@linux-m68k.org> Reviewed-by: Mark Brown <broo...@kernel.org>

Re: [PATCH v7] media: platform: Renesas IMR driver

of this patch? The compatible value "renesas,r8a7796-imr-lx4" has been in use since v4.14. Thanks! Gr{oetje,eeting}s, Geert -- Geert Uytterhoeven -- There's lots of Linux beyond ia32 -- ge...@linux-m68k.org In personal conversations with technical

Re: [PATCH v13 2/2] rcar-csi2: add Renesas R-Car MIPI CSI-2 receiver driver

>> phycnt |= PHYCNT_ENABLE_0; >> break; >> >> default: >> return -EINVAL; >> } >> >> But that's really up to you. > > Wouldn't Niklas' version generate simpler code as it uses direct assignments ? Alternatively, you could check for a

[PATCH v2 04/21] fbdev: Remove depends on HAS_DMA in case of platform dependency

ncies keep their dependencies on HAS_DMA, to prevent compiling subsystems or drivers that cannot work anyway. This simplifies the dependencies, and allows to improve compile-testing. Signed-off-by: Geert Uytterhoeven <ge...@linux-m68k.org> Reviewed-by: Mark Brown <broo...@kernel.org>

[PATCH v2 01/21] ASoC: Remove depends on HAS_DMA in case of platform dependency

_SOC_STORM and/or SND_SOC_APQ8016_SBC. Signed-off-by: Geert Uytterhoeven <ge...@linux-m68k.org> Reviewed-by: Mark Brown <broo...@kernel.org> Acked-by: Robin Murphy <robin.mur...@arm.com> --- v2: - Add Reviewed-by, Acked-by, - Drop RFC state, - Drop dependency of SND_

[PATCH v2 16/21] remoteproc: Remove depends on HAS_DMA in case of platform dependency

ncies keep their dependencies on HAS_DMA, to prevent compiling subsystems or drivers that cannot work anyway. This simplifies the dependencies, and allows to improve compile-testing. Signed-off-by: Geert Uytterhoeven <ge...@linux-m68k.org> Reviewed-by: Mark Brown <broo...@kernel.org>

[PATCH v2 03/21] crypto: Remove depends on HAS_DMA in case of platform dependency

ncies keep their dependencies on HAS_DMA, to prevent compiling subsystems or drivers that cannot work anyway. This simplifies the dependencies, and allows to improve compile-testing. Signed-off-by: Geert Uytterhoeven <ge...@linux-m68k.org> Reviewed-by: Mark Brown <broo...@kernel.org>

[PATCH v2 10/21] lightnvm: Remove depends on HAS_DMA in case of platform dependency

Do we need a dummy? The use of set_dma_ops() in this driver is questionable), - SND_SOC_LPASS_IPQ806X and SND_SOC_LPASS_PLATFORM loose their dependency on HAS_DMA, as they are selected from SND_SOC_APQ8016_SBC. Signed-off-by: Geert Uytterhoeven <ge...@linux-m68k.org> Reviewed-by:

[PATCH v2 00/21] Allow compile-testing NO_DMA (drivers)

/git/geert/linux-m68k.git/log/?h=no-dma-compile-testing-v2 It has been compile-tested with allmodconfig and allyesconfig for m68k/sun3, and has received attention from the kbuild test robot. Thanks! Geert Uytterhoeven (21): ASoC: Remove depends on HAS_DMA in case of platform dependency a

[PATCH v2 08/21] iio: adc: Remove depends on HAS_DMA in case of platform dependency

ncies keep their dependencies on HAS_DMA, to prevent compiling subsystems or drivers that cannot work anyway. This simplifies the dependencies, and allows to improve compile-testing. Signed-off-by: Geert Uytterhoeven <ge...@linux-m68k.org> Reviewed-by: Mark Brown <broo...@kernel.org>

[PATCH v2 07/21] i2c: Remove depends on HAS_DMA in case of platform dependency

ncies keep their dependencies on HAS_DMA, to prevent compiling subsystems or drivers that cannot work anyway. This simplifies the dependencies, and allows to improve compile-testing. Signed-off-by: Geert Uytterhoeven <ge...@linux-m68k.org> Reviewed-by: Mark Brown <broo...@kernel.org>

[PATCH v2 09/21] iommu: Remove depends on HAS_DMA in case of platform dependency

ncies keep their dependencies on HAS_DMA, to prevent compiling subsystems or drivers that cannot work anyway. This simplifies the dependencies, and allows to improve compile-testing. Signed-off-by: Geert Uytterhoeven <ge...@linux-m68k.org> Reviewed-by: Mark Brown <broo...@kernel.org>

[PATCH v2 05/21] firewire: Remove depends on HAS_DMA in case of platform dependency

ncies keep their dependencies on HAS_DMA, to prevent compiling subsystems or drivers that cannot work anyway. This simplifies the dependencies, and allows to improve compile-testing. Signed-off-by: Geert Uytterhoeven <ge...@linux-m68k.org> Reviewed-by: Mark Brown <broo...@kernel.org>

[PATCH v2 12/21] media: Remove depends on HAS_DMA in case of platform dependency

by several individual drivers. Signed-off-by: Geert Uytterhoeven <ge...@linux-m68k.org> Reviewed-by: Mark Brown <broo...@kernel.org> Acked-by: Robin Murphy <robin.mur...@arm.com> --- v2: - Add Reviewed-by, Acked-by, - Drop RFC state, - Drop dependency of VIDEOBUF{,2}_DMA_{CONTIG,

[PATCH v2 19/21] spi: Remove depends on HAS_DMA in case of platform dependency

ncies keep their dependencies on HAS_DMA, to prevent compiling subsystems or drivers that cannot work anyway. This simplifies the dependencies, and allows to improve compile-testing. Signed-off-by: Geert Uytterhoeven <ge...@linux-m68k.org> Reviewed-by: Mark Brown <broo...@kernel.org>

[PATCH v2 02/21] ata: Remove depends on HAS_DMA in case of platform dependency

ncies keep their dependencies on HAS_DMA, to prevent compiling subsystems or drivers that cannot work anyway. This simplifies the dependencies, and allows to improve compile-testing. Signed-off-by: Geert Uytterhoeven <ge...@linux-m68k.org> Reviewed-by: Mark Brown <broo...@kernel.org>

[PATCH v2 13/21] mmc: Remove depends on HAS_DMA in case of platform dependency

ncies keep their dependencies on HAS_DMA, to prevent compiling subsystems or drivers that cannot work anyway. This simplifies the dependencies, and allows to improve compile-testing. Signed-off-by: Geert Uytterhoeven <ge...@linux-m68k.org> Reviewed-by: Mark Brown <broo...@kernel.org>

[PATCH v2 21/21] usb: Remove depends on HAS_DMA in case of platform dependency

ncies keep their dependencies on HAS_DMA, to prevent compiling subsystems or drivers that cannot work anyway. This simplifies the dependencies, and allows to improve compile-testing. Signed-off-by: Geert Uytterhoeven <ge...@linux-m68k.org> Reviewed-by: Mark Brown <broo...@kernel.org>

[PATCH v2 20/21] staging: vc04_services: Remove depends on HAS_DMA in case of platform dependency

ncies keep their dependencies on HAS_DMA, to prevent compiling subsystems or drivers that cannot work anyway. This simplifies the dependencies, and allows to improve compile-testing. Signed-off-by: Geert Uytterhoeven <ge...@linux-m68k.org> Reviewed-by: Mark Brown <broo...@kernel.org>

[PATCH v2 18/21] serial: Remove depends on HAS_DMA in case of platform dependency

ncies keep their dependencies on HAS_DMA, to prevent compiling subsystems or drivers that cannot work anyway. This simplifies the dependencies, and allows to improve compile-testing. Signed-off-by: Geert Uytterhoeven <ge...@linux-m68k.org> Reviewed-by: Mark Brown <broo...@kernel.org>

[PATCH v2 15/21] net: Remove depends on HAS_DMA in case of platform dependency

f NO_DMA=y, and its use in this driver is questionable. Signed-off-by: Geert Uytterhoeven <ge...@linux-m68k.org> Reviewed-by: Mark Brown <broo...@kernel.org> Acked-by: Robin Murphy <robin.mur...@arm.com> --- v2: - Add Reviewed-by, Acked-by, - Drop RFC state, - Split per subsyste

[PATCH v2 14/21] mtd: Remove depends on HAS_DMA in case of platform dependency

ncies keep their dependencies on HAS_DMA, to prevent compiling subsystems or drivers that cannot work anyway. This simplifies the dependencies, and allows to improve compile-testing. Signed-off-by: Geert Uytterhoeven <ge...@linux-m68k.org> Reviewed-by: Mark Brown <broo...@kernel.org>

[PATCH v2 11/21] mailbox: Remove depends on HAS_DMA in case of platform dependency

Do we need a dummy? The use of set_dma_ops() in this driver is questionable), - SND_SOC_LPASS_IPQ806X and SND_SOC_LPASS_PLATFORM loose their dependency on HAS_DMA, as they are selected from SND_SOC_APQ8016_SBC. Signed-off-by: Geert Uytterhoeven <ge...@linux-m68k.org> Reviewed-by:

[PATCH v2 17/21] scsi: hisi_sas: Remove depends on HAS_DMA in case of platform dependency

ncies keep their dependencies on HAS_DMA, to prevent compiling subsystems or drivers that cannot work anyway. This simplifies the dependencies, and allows to improve compile-testing. Signed-off-by: Geert Uytterhoeven <ge...@linux-m68k.org> Reviewed-by: Mark Brown <broo...@kernel.org>

[PATCH v2 06/21] fpga: Remove depends on HAS_DMA in case of platform dependency

ncies keep their dependencies on HAS_DMA, to prevent compiling subsystems or drivers that cannot work anyway. This simplifies the dependencies, and allows to improve compile-testing. Signed-off-by: Geert Uytterhoeven <ge...@linux-m68k.org> Reviewed-by: Mark Brown <broo...@kernel.org>

Re: [PATCH v2 02/11] media: vsp1: Remove packed attributes from aligned structures

unaligned base. > > These structures are all correctly aligned and contain no holes, thus > the attribute is redundant and negatively impacts performance, so we > remove the attributes entirely. > > Signed-off-by: Kieran Bingham <kieran.bingham+rene...@ideasonboard.com> Revi

[PATCH] dt-bindings: media: rcar_vin: Use status "okay"

According to the Devicetree Specification, "ok" is not a valid status. Fixes: 47c71bd61b772cd7 ("[media] rcar_vin: add devicetree support") Signed-off-by: Geert Uytterhoeven <geert+rene...@glider.be> --- For the checkpatch TODO list? https://www.devicetree.org/ Documen

Re: [PATCH v2] i2c: adv748x: afe: fix sparse warning

t type in >> argument 2 (different signedness) >> >> Signed-off-by: Niklas Söderlund <niklas.soderlund+rene...@ragnatech.se> Reviewed-by: Geert Uytterhoeven <geert+rene...@glider.be> Gr{oetje,eeting}s, Geert -- Geert Uytterhoeven -- There's l

[PATCH] media: platform: Drop OF dependency of VIDEO_RENESAS_VSP1

VIDEO_RENESAS_VSP1 depends on ARCH_RENESAS && OF. As ARCH_RENESAS implies OF, the latter can be dropped. Signed-off-by: Geert Uytterhoeven <geert+rene...@glider.be> --- drivers/media/platform/Kconfig | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/drivers/

Re: [PATCH] i2c: adv748x: afe: fix sparse warning

l = V4L2_IN_ST_NO_SIGNAL; > + unsigned int signal = V4L2_IN_ST_NO_SIGNAL; u32, as adv748x_afe_status() takes an u32 signal pointer? Gr{oetje,eeting}s, Geert -- Geert Uytterhoeven -- There's lots of Linux beyond ia32 -- ge...@linux-m68k.org In personal conversations wi

Re: [PATCH 1/5] auxdisplay: charlcd: no need to call charlcd_gotoxy() if nothing changes

e: charlcd_write_char() > right now does an unconditional write_cmd() when writing a normal > character; so unless some HW requires the command for some reason even > if there is nothing changed, we can skip it. Reviewed-by: Geert Uytterhoeven <ge...@linux-m68k.org> Gr{oet

Re: [PATCH 0/4] tree-wide: fix comparison to bitshift when dealing with a mask

_MASK (0x7f < RXFC_FWM_SHIFT) drivers/usb/gadget/udc/goku_udc.h:#define INT_EPnNAK(n) (0x00100 < (n)) /* 0 < n < 4 */ Gr{oetje,eeting}s, Geert -- Geert Uytterhoeven -- There's lots of Linux beyond ia32 -- ge...@linux-m68k.org In personal co

Re: [PATCH v7 00/11] Renesas Capture Engine Unit (CEU) V4L2 driver

l resend. Good. So there's still time to fix e.g. "RZ/A1-H" to "RZ/A1H" (same for M and L). Gr{oetje,eeting}s, Geert -- Geert Uytterhoeven -- There's lots of Linux beyond ia32 -- ge...@linux-m68k.org In personal conversations with technical peopl

Re: [PATCH] sh: clk: Relax clk rate match test

Hi Jacopo, On Thu, Jan 25, 2018 at 3:14 PM, jacopo mondi <jac...@jmondi.org> wrote: > On Thu, Jan 25, 2018 at 02:53:41PM +0100, Geert Uytterhoeven wrote: >> CC linux-clk (yes I know this is about the legacy SH clock framework, but >> the public API is/should be the same)

Re: [PATCH] sh: clk: Relax clk rate match test

clk_rate_table_find() is called from sh_clk_div_set_rate()) Or are you supposed to ask for the exact clock rate? Where does the 10 MHz come from? > + > + return pos - freq_table; > + } > > return -ENOENT; > } Gr{oetje,eeting}s, Geert -- G

Re: [PATCH] v4l2-dev.h: fix symbol collision in media_entity_to_video_device()

t media_entity *entity = link->sink->entity; > vdev = media_entity_to_video_device(entity); > > Fix the collision by renaming the macro argument to 'media_entity'. Thanks! Given there also exists a "struct media_entity", using "_media_entity" seems safe to me.

Re: [PATCH -next] media: rcar_drif: fix error return code in rcar_drif_alloc_dmachannels()

in the for loop. > > Signed-off-by: Wei Yongjun <weiyongj...@huawei.com> Reviewed-by: Geert Uytterhoeven <geert+rene...@glider.be> Gr{oetje,eeting}s, Geert -- Geert Uytterhoeven -- There's lots of Linux beyond ia32 -- ge...@linux-m68k.org In personal conve

Re: [PATCH v2] v4l: async: Protect against double notifier registrations

if (WARN_ON(n == notifier)) { > + ret = -EEXIST; > + goto err_unlock; > + } > + } > + > INIT_LIST_HEAD(>waiting); > INIT_LIST_HEAD(>done); > > - mutex_lock(_lock); > - >

Re: [PATCH v4 3/9] v4l: platform: Add Renesas CEU driver

ambiguous GPL-2.0. Could you please > update all patches in this series ? IMHO it's a bit premature to do that. As long as Documentation/process/license-rules.rst isn't updated, I wouldn't follow this change. See also https://www.spinics.net/lists/linux-xfs/msg14536.html Gr{oetje,eeting}s,

Re: [PATCH v3 1/9] dt-bindings: media: Add Renesas CEU bindings

en be > replaced by "renesas,rz-ceu" ? If they're not compatible, it indeed doesn't make much sense to have a generic "renesas,ceu". Note that anything with "rz-" is a bad idea, as after RZ/A1, Renesas introduced RZ/G1, RZ/N1, and RZ/T1, which are completely

Re: [PATCH v2 4/9] ARM: dts: r7s72100: Add Capture Engine Unit (CEU)

Hi Jacopo, On Thu, Dec 28, 2017 at 3:01 PM, Jacopo Mondi <jacopo+rene...@jmondi.org> wrote: > Add Capture Engine Unit (CEU) node to device tree. Thanks for your patch! > Signed-off-by: Jacopo Mondi <jacopo+rene...@jmondi.org> With the issue below fixed: Reviewed-by: Geert U

Re: [PATCH v2 1/9] dt-bindings: media: Add Renesas CEU bindings

+ pinctrl-0 = <_pins>; > + > + status = "okay"; > + > + port { > + ceu_in: endpoint { > + remote-endpoint = <_out>; > + > + hsync-active = <1>; > + vsync-active = &

Re: [GIT PULL for v4.15-rc3] media fixes

Hi Mauro, On Wed, Dec 13, 2017 at 12:53 PM, Mauro Carvalho Chehab <mche...@osg.samsung.com> wrote: > Em Wed, 13 Dec 2017 10:03:56 +0100 > Geert Uytterhoeven <ge...@linux-m68k.org> escreveu: >> On Mon, Dec 11, 2017 at 12:12 PM, Mauro Carvalho Chehab >> <mche...@osg

Re: [GIT PULL for v4.15-rc3] media fixes

the media tree. $ linux-log-diff build.log{.old,} (from https://github.com/geertu/linux-scripts) Gr{oetje,eeting}s, Geert -- Geert Uytterhoeven -- There's lots of Linux beyond ia32 -- ge...@linux-m68k.org In personal conversations with technical people, I call myself a h

Re: [PATCH v4 0/9] vsp1: TLB optimisation and DL caching

ay's release. Gr{oetje,eeting}s, Geert -- Geert Uytterhoeven -- There's lots of Linux beyond ia32 -- ge...@linux-m68k.org In personal conversations with technical people, I call myself a hacker. But when I'm talking to journalists I just say "programmer" or something like that. -- Linus Torvalds

[PATCH] media: i2c: adv748x: Restore full DT paths in kernel messages

o-receiver@70/port@10/endpoint on port 10 adv748x 4-0070: Endpoint /soc/i2c@e66d8000/video-receiver@70/port@11/endpoint on port 11 Fixes: a7e4cfb0a7ca4773 ("of/fdt: only store the device node basename in full_name") Signed-off-by: Geert Uytterhoeven <geert+rene...@glider.be&

Re: [PATCH v1 04/10] ARM: dts: r7s72100: Add Capture Engine Unit (CEU)

+ reg = <0xe821 0x209c>; > + compatible = "renesas,renesas-ceu"; > + interrupts = ; > + power-domains = <_clocks>; if you describe the device to be part of the CPG clock domain, you should provide a clocks pr

Re: [PATCH v2 1/4] dt-bindings: media: rcar_vin: Reverse SoC part number list

Reviewed-by: Biju Das <biju@bp.renesas.com> Reviewed-by: Geert Uytterhoeven <geert+rene...@glider.be> Gr{oetje,eeting}s, Geert -- Geert Uytterhoeven -- There's lots of Linux beyond ia32 -- ge...@linux-m68k.org In personal conversations with technical peopl

Re: [PATCH v2 2/4] dt-bindings: media: rcar_vin: add device tree support for r8a774[35]

.renesas.com> > Reviewed-by: Biju Das <biju....@bp.renesas.com> Reviewed-by: Geert Uytterhoeven <geert+rene...@glider.be> Gr{oetje,eeting}s, Geert -- Geert Uytterhoeven -- There's lots of Linux beyond ia32 -- ge...@linux-m68k.org In personal conversations with t

[PATCH] [media] c8sectpfe: DVB_C8SECTPFE should depend on HAS_DMA

If NO_DMA=y: ERROR: "bad_dma_ops" [drivers/media/platform/sti/c8sectpfe/c8sectpfe.ko] undefined! Add a dependency on HAS_DMA to fix this. Signed-off-by: Geert Uytterhoeven <ge...@linux-m68k.org> --- drivers/media/platform/sti/c8sectpfe/Kconfig | 2 +- 1 file changed, 1

Re: [PATCH 1/2] dt-bindings: media: rcar_vin: add device tree support for r8a774[35]

>> > + - "renesas,vin-r8a7745" for the R8A7745 device >> > + - "renesas,vin-r8a7743" for the R8A7743 device >> >> Please keep the list sorted by SoC part number. > > It is sorted, just in descending order. Do you want me to re-order the ful

Re: [PATCH 1/2] dt-bindings: media: rcar_vin: add device tree support for r8a774[35]

sas,rcar-gen3-vin" for a generic R-Car Gen3 compatible device. > > When compatible with the generic version nodes must list the Gr{oetje,eeting}s, Geert -- Geert Uytterhoeven -- There's lots of Linux beyond ia32 -- ge...@linux-m68k.org In personal conversati

Re: [PATCH 1/2] dt-bindings: media: rcar_vin: add device tree support for r8a774[35]

t;fabrizio.cas...@bp.renesas.com> > Reviewed-by: Biju Das <biju@bp.renesas.com> Reviewed-by: Geert Uytterhoeven <geert+rene...@glider.be> Gr{oetje,eeting}s, Geert -- Geert Uytterhoeven -- There's lots of Linux beyond ia32 -- ge...@linux-m68k.org

[PATCH] media: dvb_frontend: Fix uninitialized error in dvb_frontend_handle_ioctl()

is not implemented. Preinitialize err to -EOPNOTSUPP like before to fix this. Fixes: d73dcf0cdb95a47f ("media: dvb_frontend: cleanup ioctl handling logic") Signed-off-by: Geert Uytterhoeven <ge...@linux-m68k.org> --- drivers/media/dvb-core/dvb_frontend.c | 2 +- 1 file changed, 1

Re: [PATCH v1 01/10] dt-bindings: media: Add Renesas CEU bindings

Hi Jacopo, On Wed, Nov 15, 2017 at 7:15 PM, jacopo mondi <jac...@jmondi.org> wrote: > On Wed, Nov 15, 2017 at 02:07:31PM +0100, Geert Uytterhoeven wrote: >> On Wed, Nov 15, 2017 at 11:55 AM, Jacopo Mondi >> <jacopo+rene...@jmondi.org> wrote: >> > --- /dev/null

Re: [PATCH v1 06/10] sh: sh7722: Rename CEU clock

ON_ID("vpu0", _clks[HWBLK_VPU]), > CLKDEV_DEV_ID("sh_mobile_lcdc_fb.0", _clks[HWBLK_LCDC]), Shouldn't this be merged with "[PATCH v1 05/10] arch: sh: migor: Use new renesas-ceu camera driver", to avoid breaking bisection? Gr{oetje,eeting}s,

Re: [PATCH v1 01/10] dt-bindings: media: Add Renesas CEU bindings

clock-names = "xclk"; > + > + xclk: fixed_clk { > + compatible = "fixed-clock"; > + #clock-cells = <0>; > + clock-frequency = <2400>; > + }; > + &g

Re: [PATCH v7 25/25] rcar-vin: enable support for r8a7796

CC DT On Sat, Nov 11, 2017 at 1:38 AM, Niklas Söderlund wrote: > Add the SoC specific information for Renesas r8a7796. > > Signed-off-by: Niklas Söderlund > Reviewed-by: Hans Verkuil > --- >

Re: [PATCH v9 2/2] media: rcar-csi2: add Renesas R-Car MIPI CSI-2 receiver driver

gt; + unsigned int i; > + int ret; > + > + priv = devm_kzalloc(>dev, sizeof(*priv), GFP_KERNEL); > + if (!priv) > + return -ENOMEM; > + > + priv->info = of_device_get_match_data(>dev); > + > + /* r8a7795 ES1.x behaves diff

Re: [PATCH v9 1/2] media: rcar-csi2: add Renesas R-Car MIPI CSI-2 receiver documentation

interrupts = <0 184 IRQ_TYPE_LEVEL_HIGH>; > + clocks = < CPG_MOD 714>; > + power-domains = < R8A7796_PD_ALWAYS_ON>; resets? I know this is just an example, but your prototype patches to add the csi nodes to r8a7795.dtsi also

  1   2   3   4   >