Re: [PATCH 00/51] R-Car VSP improvements for v4.6

2016-03-25 Thread Geert Uytterhoeven
.org/pinchartl/media.git drm/du/vsp1-kms/boards I assume this is the branch to be included by renesas-drivers? Thanks! Gr{oetje,eeting}s, Geert -- Geert Uytterhoeven -- There's lots of Linux beyond ia32 -- ge...@linux-m68k.org In personal conversations with technical pe

Re: [PATCH v3 1/2] media: soc_camera: rcar_vin: add R-Car Gen 2 and 3 fallback compatibility strings

2016-03-11 Thread Geert Uytterhoeven
back compatibility string scheme being adopted for > drivers for Renesas SoCs. > > Signed-off-by: Yoshihiro Kaneko <ykaneko0...@gmail.com> > Signed-off-by: Simon Horman <horms+rene...@verge.net.au> Acked-by: Geert Uytterhoeven <geert+rene...@glider.be> > --- a/driv

Re: [PATCH v2] media: sh_mobile_ceu_camera: Remove dependency on SUPERH

2016-03-08 Thread Geert Uytterhoeven
e > appropriate name than SHMOBILE for the majority of Renesas ARM based SoCs. > > Thanks to Geert Uytterhoeven for analysis and portions of the > change log text. > > Cc: Geert Uytterhoeven <ge...@linux-m68k.org> > Signed-off-by: Simon Horman <horms+rene...@verge.n

Re: [PATCH v2] media: rcar_vin: Use ARCH_RENESAS

2016-03-08 Thread Geert Uytterhoeven
more > appropriate name than SHMOBILE for the majority of Renesas ARM based SoCs. > > Signed-off-by: Simon Horman <horms+rene...@verge.net.au> Acked-by: Geert Uytterhoeven <geert+rene...@glider.be> Gr{oetje,eeting}s, Geert -- Geert Uytterhoeven --

Re: Build regressions/improvements in v4.5-rc7

2016-03-07 Thread Geert Uytterhoeven
On Mon, Mar 7, 2016 at 9:55 AM, Geert Uytterhoeven <ge...@linux-m68k.org> wrote: > JFYI, when comparing v4.5-rc7[1] to v4.5-rc6[3], the summaries are: > - build errors: +8/-7 + error: debugfs.c: undefined reference to `clk_round_rate': => .text+0x11b9e0) arm-randconfig

Re: [PATCH] media: sh_mobile_ceu_camera, rcar_vin: Use ARCH_RENESAS

2016-03-06 Thread Geert Uytterhoeven
Hi Simon, Oops, seems I dropped all CCs in my earlier reply. Fixing up... On Mon, Mar 7, 2016 at 2:28 AM, Simon Horman <ho...@verge.net.au> wrote: > On Thu, Mar 03, 2016 at 09:40:07AM +0100, Geert Uytterhoeven wrote: >> On Thu, Mar 3, 2016 at 2:52 AM, Simon Horman <horms+re

Re: [RFC 09/15] media: platform: Add missing MFD_SYSCON dependency on HAS_IOMEM

2016-03-03 Thread Geert Uytterhoeven
es HAS_IOMEM afaik. No systems around with HV-based DMA? 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. B

Re: [RFC 08/15] rtc: at91sam9: Add missing MFD_SYSCON dependency on HAS_IOMEM

2016-03-03 Thread Geert Uytterhoeven
did, all other RTC drivers would also fail. So UML has no RTC. Should/can it use RTC_DRV_GENERIC? Gr{oetje,eeting}s, Geert -- Geert Uytterhoeven -- There's lots of Linux beyond ia32 -- ge...@linux-m68k.org In personal conversations with te

Re: [PATCH/RFC 1/9] clk: shmobile: r8a7795: Add FCP clocks

2016-03-03 Thread Geert Uytterhoeven
PG is capable of turning the supply of clock signals to individual modules on or off and of resetting individual modules." So it is a clock signal, or better (or worse): clock signals (plural). Hence MSTP gates one or more clocks. Sigh... Gr{oetje,eeting}s, Geert -- Geert Uytterhoeven --

Re: [PATCH/RFC 26/48] videodev2.h: Add request field to v4l2_pix_format_mplane

2015-12-18 Thread Geert Uytterhoeven
ld a __u16. > >> I would prefer a __u16 here. Also put the request field *before* the >> reserved array, not after. > > The reserved array isn't aligned to a 32 bit (or even 16 bit) boundary. I can > put the request field before it, with a 8 bit hole before the field. There's no alignm

Re: [PATCH v2 00/32] VSP: Add R-Car Gen3 support

2015-12-16 Thread Geert Uytterhoeven
Hi Laurent, On Sat, Dec 5, 2015 at 11:54 PM, Laurent Pinchart <laurent.pinch...@ideasonboard.com> wrote: > On Saturday 05 December 2015 11:57:49 Geert Uytterhoeven wrote: >> As http://git.linuxtv.org/pinchartl/media.git/tag/?id=vsp1-kms-20151112 is >> getting old, and

vsp1 BUG_ON() and crash (Re: [PATCH v9 03/12] media: Entities, pads and links)

2015-12-14 Thread Geert Uytterhoeven
fter which the BUG_ON() takes over: commit eb9e998983338811c3328294ed2f093367161df1 Author: Javier Martinez Canillas <jav...@osg.samsung.com> Date: Thu Sep 3 12:19:25 2015 -0300 [media] v4l: vsp1: separate links creation from entities init Gr{oetje,eeting}s,

Re: [PATCH v2 00/32] VSP: Add R-Car Gen3 support

2015-12-05 Thread Geert Uytterhoeven
ternatively, I can just import the series you posted, but having the broken-out integration part would be nice. Thanks again! 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 2/2] [media] include/media: move platform driver headers to a separate dir

2015-11-13 Thread Geert Uytterhoeven
kdir include/media/platform_data > (cd include/media/; git mv $(grep -l platform_data *.h|grep -v v4l2) I think include/linux/platform_data/media/, like Arnd suggested, would be better. Then we can make it a common goal to empty include/linux/platform_data/ ;-) Gr{oetje,eeting}s,

Re: [PATCH] media: adv7180: increase delay after reset to 5ms

2015-11-12 Thread Geert Uytterhoeven
uld you like to take this in your tree with other Analog Devices > patches, or should I take it ? Which tree is this? Should I include it in renesas-drivers? Does it contain more fixes? During the s2ram suspend phase, the lockdep_assert_held() in adv7180_write() is triggered on r8a7791/koelsch. Gr{oetje,eet

Re: [PATCH v2 2/2] [media] rcar_vin: Remove obsolete platform data support

2015-10-07 Thread Geert Uytterhoeven
/arm/mach-shmobile/board-bockw.c:24:45: fatal error: >>> linux/platform_data/camera-rcar.h: No such file or directory > #include > ^ Please ignore. Legacy (non-DT) bockw support has been removed in arm-soc/for-next. Gr{oetje,eeting}s,

[PATCH v2 1/2] [media] rcar_vin: Remove obsolete r8a779x-vin platform_device_id entries

2015-10-07 Thread Geert Uytterhoeven
y. Signed-off-by: Geert Uytterhoeven <geert+rene...@glider.be> Acked-by: Simon Horman <horms+rene...@verge.net.au> --- v2: - Add Acked-by. --- drivers/media/platform/soc_camera/rcar_vin.c | 2 -- 1 file changed, 2 deletions(-) diff --git a/drivers/media/platform/soc_camera/rcar_

[PATCH v2 0/2] [media] rcar_vin: Remove obsolete platform data support

2015-10-07 Thread Geert Uytterhoeven
figuration. Changes compared to v1: - Added patch 2. Geert Uytterhoeven (2): [media] rcar_vin: Remove obsolete r8a779x-vin platform_device_id entries [media] rcar_vin: Remove obsolete platform data support drivers/media/platform/soc_camera/rcar_vin.c | 77 +++

[PATCH v2 2/2] [media] rcar_vin: Remove obsolete platform data support

2015-10-07 Thread Geert Uytterhoeven
Since commit 3d7608e4c169af03 ("ARM: shmobile: bockw: remove legacy board file and config"), Renesas R-Car SoCs are only supported in generic DT-only ARM multi-platform builds. The driver doesn't need to use platform data anymore, hence remove platform data configuration. Signed-off

[PATCH] [media] atmel-isi: Protect PM-only functions to kill warning

2015-09-06 Thread Geert Uytterhoeven
to fix this. Signed-off-by: Geert Uytterhoeven <ge...@linux-m68k.org> --- Resend with correct suject --- drivers/media/platform/soc_camera/atmel-isi.c | 2 ++ 1 file changed, 2 insertions(+) diff --git a/drivers/media/platform/soc_camera/atmel-isi.c b/drivers/media/platform/soc_camera/atmel

[PATCH] [media] atmel-isi:

2015-09-06 Thread Geert Uytterhoeven
to fix this. Signed-off-by: Geert Uytterhoeven <ge...@linux-m68k.org> --- drivers/media/platform/soc_camera/atmel-isi.c | 2 ++ 1 file changed, 2 insertions(+) diff --git a/drivers/media/platform/soc_camera/atmel-isi.c b/drivers/media/platform/soc_camera/atmel-isi.c index 90701726a0

[PATCH] [media] VIDEO_RENESAS_JPU should depend on HAS_DMA

2015-09-06 Thread Geert Uytterhoeven
ion of function ‘dma_get_sgtable’ VIDEO_RENESAS_JPU selects VIDEOBUF2_DMA_CONTIG, which bypasses its dependency on HAS_DMA. Make VIDEO_RENESAS_JPU depend on HAS_DMA to fix this. Signed-off-by: Geert Uytterhoeven <ge...@linux-m68k.org> --- drivers/media/platform/Kconfig | 2 +- 1 file change

[PATCH 1/3] [media] dvb-frontends: Make all DVB Frontends visible if COMPILE_TEST=y

2015-06-29 Thread Geert Uytterhoeven
Make the DVB Frontends menu visible when compile-testing, to allow selecting additional drivers on top of the drivers that are already automatically selected if MEDIA_SUBDRV_AUTOSELECT is enabled. Without this, many drivers stay disabled during e.g. allmodconfig. Signed-off-by: Geert

[PATCH 3/3] [media] tuners: Make all TV tuners visible if COMPILE_TEST=y

2015-06-29 Thread Geert Uytterhoeven
Make the TV tuners menu visible when compile-testing, to allow selecting additional drivers on top of the drivers that are already automatically selected if MEDIA_SUBDRV_AUTOSELECT is enabled. Without this, many drivers stay disabled during e.g. allmodconfig. Signed-off-by: Geert Uytterhoeven ge

[PATCH] [media] adv7604/cobalt: Allow compile test if !GPIOLIB

2015-06-29 Thread Geert Uytterhoeven
the former) on GPIOLIB if COMPILE_TEST is enabled. Signed-off-by: Geert Uytterhoeven ge...@linux-m68k.org --- The VIDEO_COBALT part was untested, but I assume it's OK given the dependency was added because of the select, cfr. commit 29fba6a84bc73b92 ([media] adv7604/cobalt: missing GPIOLIB dependency

[PATCH 2/3] [media] i2c: Make all i2c devices visible if COMPILE_TEST=y

2015-06-29 Thread Geert Uytterhoeven
Make the i2c devices menu visible when compile-testing, to allow selecting additional drivers on top of the drivers that are already automatically selected if MEDIA_SUBDRV_AUTOSELECT is enabled. Without this, many drivers stay disabled during e.g. allmodconfig. Signed-off-by: Geert Uytterhoeven

[PATCH] [media] mt9v032: Add missing initialization of pdata in mt9v032_get_pdata()

2015-06-29 Thread Geert Uytterhoeven
drivers/media/i2c/mt9v032.c: In function ‘mt9v032_get_pdata’: drivers/media/i2c/mt9v032.c:885: warning: ‘pdata’ may be used uninitialized in this function If parsing the endpoint node properties fails, mt9v032_get_pdata() will return an uninitialized pointer value. Signed-off-by: Geert

[PATCH] [media] rcar_vin: Remove obsolete r8a779x-vin platform_device_id entries

2015-06-23 Thread Geert Uytterhoeven
-off-by: Geert Uytterhoeven geert+rene...@glider.be --- drivers/media/platform/soc_camera/rcar_vin.c | 2 -- 1 file changed, 2 deletions(-) diff --git a/drivers/media/platform/soc_camera/rcar_vin.c b/drivers/media/platform/soc_camera/rcar_vin.c index db7700b0af7cd569..ef784d311253b142 100644

Re: Build regressions/improvements in v4.1

2015-06-23 Thread Geert Uytterhoeven
Hi Russell, On Mon, Jun 22, 2015 at 11:18 PM, Russell King - ARM Linux li...@arm.linux.org.uk wrote: On Mon, Jun 22, 2015 at 10:52:13PM +0200, Geert Uytterhoeven wrote: On Mon, Jun 22, 2015 at 10:48 PM, Geert Uytterhoeven ge...@linux-m68k.org wrote: JFYI, when comparing v4.1[1] to v4.1-rc8[3

Re: Build regressions/improvements in v4.1

2015-06-22 Thread Geert Uytterhoeven
On Mon, Jun 22, 2015 at 10:48 PM, Geert Uytterhoeven ge...@linux-m68k.org wrote: JFYI, when comparing v4.1[1] to v4.1-rc8[3], the summaries are: - build errors: +44/-7 + /home/kisskb/slave/src/arch/arm/mm/dump.c: error: 'L_PTE_MT_BUFFERABLE' undeclared here (not in a function): = 81:10

Re: [PATCH 01/10] sh-vou: hook up the clock correctly

2015-06-06 Thread Geert Uytterhoeven
= -1, }; so unless I'm mistaken, the platform device's name will be sh-vou, not sh-vou.0. Does it work if you just correct the name in the CLKDEV_DEV_ID() line? Thanks! Gr{oetje,eeting}s, Geert -- Geert Uytterhoeven -- There's lots of Linux beyond ia32

Re: Build regressions/improvements in v4.1-rc6

2015-06-01 Thread Geert Uytterhoeven
On Mon, Jun 1, 2015 at 12:09 PM, Geert Uytterhoeven ge...@linux-m68k.org wrote: JFYI, when comparing v4.1-rc6[1] to v4.1-rc5[3], the summaries are: - build errors: +8/-52 + /home/kisskb/slave/src/drivers/media/i2c/ov2659.c: error: 'struct v4l2_subdev_fh' has no member named 'pad': = 1264

Re: [PATCH 02/13] dmaengine: Introduce dma_request_slave_channel_compat_reason()

2015-05-29 Thread Geert Uytterhoeven
dma_request_slave_channel_compat() Then all callers of dma_request_slave_channel_compat() have to be modified to handle ERR_PTR first. The same is true for (the existing) dma_request_slave_channel_reason() vs. dma_request_slave_channel(). Gr{oetje,eeting}s, Geert -- Geert Uytterhoeven -- There's

[PATCH 10/27] [media] wl128x: Allow compile test of GPIO consumers if !GPIOLIB

2015-05-05 Thread Geert Uytterhoeven
. Signed-off-by: Geert Uytterhoeven ge...@linux-m68k.org Cc: Mauro Carvalho Chehab mche...@osg.samsung.com Cc: linux-media@vger.kernel.org --- drivers/media/radio/wl128x/Kconfig | 4 ++-- 1 file changed, 2 insertions(+), 2 deletions(-) diff --git a/drivers/media/radio/wl128x/Kconfig b/drivers/media

Re: [PATCH/RFC] v4l: vsp1: Align crop rectangle to even boundary for YUV formats

2015-04-30 Thread Geert Uytterhoeven
On Thu, Apr 30, 2015 at 1:22 PM, Sergei Shtylyov sergei.shtyl...@cogentembedded.com wrote: Since there is no distintion between 12bit and 16bit YUV formats in Вistinсtion. Distinction? Gr{oetje,eeting}s, Geert -- Geert Uytterhoeven -- There's lots of Linux beyond

[PATCH 3/3] Input: TOUCHSCREEN_SUR40 should depend on HAS_DMA

2015-04-23 Thread Geert Uytterhoeven
this. Signed-off-by: Geert Uytterhoeven ge...@linux-m68k.org --- drivers/input/touchscreen/Kconfig | 3 +-- 1 file changed, 1 insertion(+), 2 deletions(-) diff --git a/drivers/input/touchscreen/Kconfig b/drivers/input/touchscreen/Kconfig index b0a94cdd96add5c9..4591389108f0057b 100644 --- a/drivers

[PATCH 1/3] [media] v4l: xilinx: VIDEO_XILINX should depend on HAS_DMA

2015-04-23 Thread Geert Uytterhoeven
bypasses its dependency on HAS_DMA. Make VIDEO_XILINX depend on HAS_DMA to fix this. Signed-off-by: Geert Uytterhoeven ge...@linux-m68k.org --- drivers/media/platform/xilinx/Kconfig | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/drivers/media/platform/xilinx/Kconfig b/drivers

[PATCH] [media] vb2: Remove unused variable fileio in vb2_thread_stop()

2015-04-07 Thread Geert Uytterhoeven
drivers/media/v4l2-core/videobuf2-core.c: In function 'vb2_thread_stop': drivers/media/v4l2-core/videobuf2-core.c:3228:26: warning: unused variable 'fileio' [-Wunused-variable] Fixes: 0e661006370b7e7f ([media] vb2: fix 'UNBALANCED' warnings when calling vb2_thread_stop()) Signed-off-by: Geert

Re: [PATCH 00/14] Fix fallout from per-user struct clk patches

2015-04-05 Thread Geert Uytterhoeven
a number of places too... Thanks for your patches! Can you please tell which are critical fixes for regressions, and which are cleanups? It's not so obvious to me from the patch descriptions. Gr{oetje,eeting}s, Geert -- Geert Uytterhoeven -- There's lots of Linux beyond

Re: [PATCH/RFC] v4l: vsp1: Change VSP1 LIF linebuffer FIFO

2015-03-16 Thread Geert Uytterhoeven
the significance of changing obth? 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

Re: [PATCH 05/10] clkdev: add clkdev_create() helper

2015-03-02 Thread Geert Uytterhoeven
On Mon, Mar 2, 2015 at 6:46 PM, Russell King - ARM Linux li...@arm.linux.org.uk wrote: On Mon, Mar 02, 2015 at 06:22:31PM +0100, Geert Uytterhoeven wrote: On Mon, Mar 2, 2015 at 6:06 PM, Russell King rmk+ker...@arm.linux.org.uk wrote: --- a/include/linux/clkdev.h +++ b/include/linux

Re: [PATCH 02/10] SH: use clkdev_add_table()

2015-03-02 Thread Geert Uytterhoeven
the correct interface. Signed-off-by: Russell King rmk+ker...@arm.linux.org.uk Thanks, looks good. Acked-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

Re: [PATCH 05/10] clkdev: add clkdev_create() helper

2015-03-02 Thread Geert Uytterhoeven
, 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

Re: [PATCH] [media] soc-camera: Remove bogus devm_kfree() in soc_of_bind()

2015-02-19 Thread Geert Uytterhoeven
Hi Guennadi, On Wed, Feb 18, 2015 at 10:16 PM, Guennadi Liakhovetski g.liakhovet...@gmx.de wrote: On Wed, 18 Feb 2015, Sergei Shtylyov wrote: On 02/18/2015 07:32 PM, Geert Uytterhoeven wrote: Unlike scan_async_group(), soc_of_bind() doesn't allocate its soc_camera_async_client structure

[PATCH v2] [media] soc-camera: Fix devm_kfree() in soc_of_bind()

2015-02-19 Thread Geert Uytterhoeven
---[ end trace e3a984cc0335c8a0 ]--- rcar_vin e6ef1000.video: group probe failed: -6 Fixes: 1ddc6a6caa94e1e1 ([media] soc_camera: add support for dt binding soc_camera drivers) Cc: sta...@vger.kernel.org Signed-off-by: Geert Uytterhoeven geert+rene...@glider.be --- Triggered with shmobile

[PATCH] [media] soc-camera: Remove bogus devm_kfree() in soc_of_bind()

2015-02-18 Thread Geert Uytterhoeven
probe failed: -6 Fixes: 1ddc6a6caa94e1e1 ([media] soc_camera: add support for dt binding soc_camera drivers) Cc: sta...@vger.kernel.org Signed-off-by: Geert Uytterhoeven geert+rene...@glider.be --- Triggered with shmobile-defconfig on r8a7791/koelsch. --- drivers/media/platform/soc_camera

[PATCH 1/2] [media] am437x: VIDEO_AM437X_VPFE should depend on HAS_DMA

2015-02-12 Thread Geert Uytterhoeven
’ VIDEO_AM437X_VPFE selects VIDEOBUF2_DMA_CONTIG, which bypasses its dependency on HAS_DMA. Make VIDEO_AM437X_VPFE depend on HAS_DMA to fix this. Signed-off-by: Geert Uytterhoeven ge...@linux-m68k.org --- drivers/media/platform/am437x/Kconfig | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git

[PATCH 2/2] [media] timberdale: VIDEO_TIMBERDALE should depend on HAS_DMA

2015-02-12 Thread Geert Uytterhoeven
depend on HAS_DMA to fix this. Signed-off-by: Geert Uytterhoeven ge...@linux-m68k.org --- 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 d9b872b9285a3460..2e30be55d17a7db2 100644

Re: [PATCH] soc-camera: fix device capabilities in multiple camera host drivers

2015-01-20 Thread Geert Uytterhoeven
didn't fill device_caps a compile-time warning is issued. Fix this non-compliance in several soc-camera camera host drivers. Thanks, works fine on r8a7740/armadillo-legacy! Reported-by: Geert Uytterhoeven ge...@linux-m68k.org Signed-off-by: Guennadi Liakhovetski g.liakhovet...@gmx.de Tested

SuperH Mobile CEU driver warning

2014-12-22 Thread Geert Uytterhoeven
, 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

[PATCH] [media] vb2-vmalloc: Protect DMA-specific code by #ifdef CONFIG_HAS_DMA

2014-12-15 Thread Geert Uytterhoeven
' videobuf2-vmalloc.c:(.text+0x6f1282): undefined reference to `dma_map_sg' As we don't want to make the core VIDEOBUF2_VMALLOC depend on HAS_DMA (it's v4l2 core code, and selected by a lot of drivers), stub out the DMA support if HAS_DMA is not set. Signed-off-by: Geert Uytterhoeven ge...@linux-m68k.org

[PATCH/RESEND] [media] VIDEO_CAFE_CCIC should select VIDEOBUF2_DMA_SG

2014-12-15 Thread Geert Uytterhoeven
': mcam-core.c:(.text+0x1c34bf): undefined reference to `vb2_dma_sg_cleanup_ctx' Reported-by: Fengguang Wu fengguang...@intel.com Signed-off-by: Geert Uytterhoeven ge...@linux-m68k.org --- This is a resend of a patch from 2013-09-30. It's still valid. drivers/media/platform/marvell-ccic/Kconfig | 1

Re: [PATCH/RESEND] [media] VIDEO_CAFE_CCIC should select VIDEOBUF2_DMA_SG

2014-12-15 Thread Geert Uytterhoeven
Hi Jon, On Mon, Dec 15, 2014 at 3:04 PM, Jonathan Corbet cor...@lwn.net wrote: On Mon, 15 Dec 2014 14:55:37 +0100 Geert Uytterhoeven ge...@linux-m68k.org wrote: If VIDEO_CAFE_CCIC=y, but VIDEOBUF2_DMA_SG=m: drivers/built-in.o: In function `mcam_v4l_open': mcam-core.c:(.text+0x1c2e81

[PATCH/TRIVIAL 2/4] [media] cx24117: Grammar s/if ... if/if ... is/

2014-11-27 Thread Geert Uytterhoeven
Signed-off-by: Geert Uytterhoeven geert+rene...@glider.be Cc: Mauro Carvalho Chehab mche...@osg.samsung.com Cc: linux-media@vger.kernel.org --- drivers/media/dvb-frontends/cx24117.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/drivers/media/dvb-frontends/cx24117.c b

Re: [PATCH 1/2] v4l: vsp1: Reset VSP1 RPF source address

2014-11-26 Thread Geert Uytterhoeven
, 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

Re: [PATCH] media: soc_camera: rcar_vin: Enable VSYNC field toggle mode

2014-10-21 Thread Geert Uytterhoeven
, 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

Re: [PATCH 1/3] media: soc_camera: rcar_vin: Add scaling support

2014-10-14 Thread Geert Uytterhoeven
= 0; struct v4l2_rect *cam_subrect = cam-subrect; + unsigned long value; u32, as it's written to a 32-bit register later. Gr{oetje,eeting}s, Geert -- Geert Uytterhoeven -- There's lots of Linux beyond ia32 -- ge...@linux-m68k.org In personal conversations

Re: [PATCH] [media] s5p-jpeg: Only build suspend/resume for PM

2014-10-13 Thread Geert Uytterhoeven
) is not enabled. Thanks! Despite the availability of your patch, this build warning has migrated to mainline. Signed-off-by: Thierry Reding tred...@nvidia.com Acked-by: Geert Uytterhoeven ge...@linux-m68k.org --- a/drivers/media/platform/s5p-jpeg/jpeg-core.c +++ b/drivers/media/platform/s5p-jpeg/jpeg

Re: [PATCH] [media] s5p-jpeg: Only build suspend/resume for PM

2014-10-13 Thread Geert Uytterhoeven
Hi Mauro, On Mon, Oct 13, 2014 at 6:24 PM, Mauro Carvalho Chehab m.che...@samsung.com wrote: Em Mon, 13 Oct 2014 18:16:10 +0200 Geert Uytterhoeven ge...@linux-m68k.org escreveu: On Thu, Oct 2, 2014 at 9:21 AM, Thierry Reding thierry.red...@gmail.com wrote: From: Thierry Reding tred

Re: [PATCH 1/6] V4L2: Add Renesas R-Car JPEG codec driver.

2014-09-23 Thread Geert Uytterhoeven
) +{ + u32 *addr = (u32 *)*p; + + *addr = cpu_to_be32(v); + *p += 4; +} Is the address in *p guaranteed to be aligned to 2 resp. 4 bytes? If not, you can use put_unaligned*(). Gr{oetje,eeting}s, Geert -- Geert Uytterhoeven -- There's lots of Linux beyond

Re: [PATCH] v4l: vsp1: fix driver dependencies

2014-09-01 Thread Geert Uytterhoeven
...@samsung.com Cc: Simon Horman ho...@verge.net.au Cc: Magnus Damm magnus.d...@gmail.com Acked-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

Re: [PATCH v2 6/6] devicetree: bindings: Document Renesas JPEG Processing Unit.

2014-08-26 Thread Geert Uytterhoeven
On Tue, Aug 26, 2014 at 1:57 AM, Simon Horman ho...@verge.net.au wrote: On Mon, Aug 25, 2014 at 02:59:46PM +0200, Geert Uytterhoeven wrote: Hi Mikhail, On Mon, Aug 25, 2014 at 2:35 PM, Mikhail Ulyanov mikhail.ulya...@cogentembedded.com wrote: + - compatible: should containg one

Re: [PATCH v2 6/6] devicetree: bindings: Document Renesas JPEG Processing Unit.

2014-08-26 Thread Geert Uytterhoeven
On Tue, Aug 26, 2014 at 11:01 AM, Simon Horman ho...@verge.net.au wrote: On Tue, Aug 26, 2014 at 10:03:34AM +0200, Geert Uytterhoeven wrote: On Tue, Aug 26, 2014 at 1:57 AM, Simon Horman ho...@verge.net.au wrote: On Mon, Aug 25, 2014 at 02:59:46PM +0200, Geert Uytterhoeven wrote: On Mon, Aug

Re: [PATCH v2 6/6] devicetree: bindings: Document Renesas JPEG Processing Unit.

2014-08-25 Thread Geert Uytterhoeven
, 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

[PATCH trivial 2/4] [media] cx23885: Spelling s/compuations/computations/

2014-08-08 Thread Geert Uytterhoeven
Signed-off-by: Geert Uytterhoeven ge...@linux-m68k.org Cc: Mauro Carvalho Chehab m.che...@samsung.com Cc: linux-media@vger.kernel.org --- drivers/media/pci/cx23885/cx23888-ir.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/drivers/media/pci/cx23885/cx23888-ir.c b/drivers

[PATCH trivial 1/4] [media] cx25840: Spelling s/compuations/computations/

2014-08-08 Thread Geert Uytterhoeven
Signed-off-by: Geert Uytterhoeven ge...@linux-m68k.org Cc: Mauro Carvalho Chehab m.che...@samsung.com Cc: linux-media@vger.kernel.org --- drivers/media/i2c/cx25840/cx25840-ir.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/drivers/media/i2c/cx25840/cx25840-ir.c b/drivers

dib7000p_get_stats: ‘i’ is used uninitialized (was: Re: [media] dib7000p: Add DVBv5 stats support)

2014-08-06 Thread Geert Uytterhoeven
= FE_SCALE_COUNTER; + c-block_count.stat[0].uvalue += blocks; + } + } + return 0; +} Gr{oetje,eeting}s, Geert -- Geert Uytterhoeven -- There's lots of Linux beyond ia32 -- ge...@linux-m68k.org In personal conversations

[PATCH] [media] staging/solo6x10: SOLO6X10 should select BITREVERSE

2014-07-06 Thread Geert Uytterhoeven
If CONFIG_SOLO6X10=y, but CONFIG_BITREVERSE=m: drivers/built-in.o: In function `solo_osd_print': (.text+0x1c7a1f): undefined reference to `byte_rev_table' make: *** [vmlinux] Error 1 Reported-by: kbuild test robot fengguang...@intel.com Signed-off-by: Geert Uytterhoeven ge...@linux

Re: [PATCH] [media] staging/solo6x10: SOLO6X10 should select BITREVERSE

2014-07-06 Thread Geert Uytterhoeven
CC kbuild test robot fengguang...@intel.com On Sun, Jul 6, 2014 at 10:58 AM, Geert Uytterhoeven ge...@linux-m68k.org wrote: If CONFIG_SOLO6X10=y, but CONFIG_BITREVERSE=m: drivers/built-in.o: In function `solo_osd_print': (.text+0x1c7a1f): undefined reference to `byte_rev_table

Re: [PATCH] [media] staging/solo6x10: SOLO6X10 should select BITREVERSE

2014-07-06 Thread Geert Uytterhoeven
On Sun, Jul 6, 2014 at 10:59 AM, Geert Uytterhoeven ge...@linux-m68k.org wrote: CC kbuild test robot fengguang...@intel.com Doh, this time for real... On Sun, Jul 6, 2014 at 10:58 AM, Geert Uytterhoeven ge...@linux-m68k.org wrote: If CONFIG_SOLO6X10=y, but CONFIG_BITREVERSE=m: drivers

Re: [PATCH 11/18] v4l: vsp1: wpf: Simplify cast to pipeline structure

2014-06-01 Thread Geert Uytterhoeven
On Sun, Jun 1, 2014 at 5:39 AM, Laurent Pinchart laurent.pinchart+rene...@ideasonboard.com wrote: USe the subdev pointer directly to_vsp1_pipeline() macro instead of Use Gr{oetje,eeting}s, Geert -- Geert Uytterhoeven -- There's lots of Linux beyond ia32 -- ge...@linux

Re: [PATCH 00/18] Renesas VSP1: alpha support

2014-06-01 Thread Geert Uytterhoeven
, for the rationale please see https://www.mail-archive.com/linux-media@vger.kernel.org/msg75449.html. mail-archive.com seems to be down. Do you have a link to another archiver? Thanks! Gr{oetje,eeting}s, Geert -- Geert Uytterhoeven -- There's lots of Linux beyond ia32 -- ge

Re: [PATCH 00/18] Renesas VSP1: alpha support

2014-06-01 Thread Geert Uytterhoeven
On Sun, Jun 1, 2014 at 10:51 AM, Geert Uytterhoeven ge...@linux-m68k.org wrote: On Sun, Jun 1, 2014 at 5:39 AM, Laurent Pinchart laurent.pinchart+rene...@ideasonboard.com wrote: The first two patch add new pixel formats for alpha and non-alpha RGB, and extend usage of the ALPHA_COMPONENT

Re: [PATCH 2/2] gpio: gpiolib: set gpiochip_remove retval to void

2014-05-30 Thread Geert Uytterhoeven
); panic? Is this likely to happen? 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

Re: mcam-core.c:undefined reference to `vb2_dma_sg_memops'

2014-03-10 Thread Geert Uytterhoeven
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

Re: [PATCH 5/5] rcar_vin: add devicetree support

2014-03-07 Thread Geert Uytterhoeven
On Fri, Mar 7, 2014 at 2:01 PM, Ben Dooks ben.do...@codethink.co.uk wrote: +++ b/Documentation/devicetree/bindings/media/rcar_vin.txt +family of devices. The current blocks are always slaves and suppot one input support Gr{oetje,eeting}s, Geert -- Geert Uytterhoeven

Re: [PATCH 00/16] sleep_on removal, second try

2014-02-27 Thread Geert Uytterhoeven
, 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

[PATCH] [media] v4l: VIDEO_SH_VOU should depend on HAS_DMA

2014-02-21 Thread Geert Uytterhoeven
to fix this. Signed-off-by: Geert Uytterhoeven ge...@linux-m68k.org --- 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 b2a4403940c5..c137abfa0c54 100644 --- a/drivers/media

Re: [Intel-gfx] [Mesa-dev] [PATCH] dri3, i915, i965: Add __DRI_IMAGE_FOURCC_SARGB8888

2013-11-25 Thread Geert Uytterhoeven
. Y, Cb, Cr, and alpha, giving a proper visual. Unfortunately the YCbCr visuals haven't made it into mainline. FOURCC unifies all of that in (not so) unique 32-bit IDs. Gr{oetje,eeting}s, Geert -- Geert Uytterhoeven -- There's lots of Linux beyond ia32 -- ge...@linux

[PATCH] [media] radio-shark: Mark shark_resume_leds() inline to kill compiler warning

2013-10-28 Thread Geert Uytterhoeven
the compiler warning. shark_resume_leds() is small and it has only one caller. Signed-off-by: Geert Uytterhoeven ge...@linux-m68k.org --- {cris,m68k,parisc,sparc,xtensa}-all{mod,yes}config drivers/media/radio/radio-shark.c |2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/drivers/media

Re: mcam-core.c:undefined reference to `vb2_dma_sg_memops'

2013-09-30 Thread Geert Uytterhoeven
of emailing random people to obtain a solution worked ;-) From 8a53ff3c33cfaa8641c9ba3e16bc5b0a35c74842 Mon Sep 17 00:00:00 2001 From: Geert Uytterhoeven ge...@linux-m68k.org Date: Mon, 30 Sep 2013 09:03:20 +0200 Subject: [PATCH] [media] VIDEO_CAFE_CCIC should select VIDEOBUF2_DMA_SG

[PATCH] [media] media/v4l2: VIDEO_RENESAS_VSP1 should depend on HAS_DMA

2013-09-06 Thread Geert Uytterhoeven
VIDEO_RENESAS_VSP1 (which doesn't have a platform dependency) selects VIDEOBUF2_DMA_CONTIG, but the latter depends on HAS_DMA. Make VIDEO_RENESAS_VSP1 depend on HAS_DMA to fix this. Signed-off-by: Geert Uytterhoeven ge...@linux-m68k.org --- drivers/media/platform/Kconfig |2 +- 1 file

Re: [PATCH] [media] media/v4l2: VIDEO_RENESAS_VSP1 should depend on HAS_DMA

2013-09-06 Thread Geert Uytterhoeven
On Fri, Sep 6, 2013 at 5:20 PM, Laurent Pinchart laurent.pinch...@ideasonboard.com wrote: On Friday 06 September 2013 14:43:56 Geert Uytterhoeven wrote: If NO_DMA=y: warning: (... VIDEO_RENESAS_VSP1 ...) selects VIDEOBUF2_DMA_CONTIG which has unmet direct dependencies (MEDIA_SUPPORT

[PATCH] [media] media/v4l2: VIDEO_SH_VEU should depend on HAS_DMA

2013-08-18 Thread Geert Uytterhoeven
. However, this got lost in the merge conflict resolution in commit df90e2258950fd631cdbf322c1ee1f22068391aa (Merge branch 'devel-for-v3.10' into v4l_for_linus). Re-add the dependency to fix this. Signed-off-by: Geert Uytterhoeven ge...@linux-m68k.org --- drivers/media/platform/Kconfig |2 +- 1

Re: [PATCH] usb: USB host support should depend on HAS_DMA

2013-08-18 Thread Geert Uytterhoeven
, 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

Re: [PATCH] usb: USB host support should depend on HAS_DMA

2013-07-11 Thread Geert Uytterhoeven
incorrectly doesn't depend on HAS_DMA. 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

[PATCH] usb: USB host support should depend on HAS_DMA

2013-07-10 Thread Geert Uytterhoeven
for other busses (e.g. MOUSE_SYNAPTICS_I2C) already handle this in a similar way. Signed-off-by: Geert Uytterhoeven ge...@linux-m68k.org --- drivers/input/joystick/Kconfig|3 +-- drivers/input/misc/Kconfig| 15 +-- drivers/input/mouse/Kconfig |9 +++-- drivers

Re: linux-next: Tree for Jun 3 (fonts.c vivi)

2013-06-03 Thread Geert Uytterhoeven
was to move the font code to lib/font, but I haven't done that yet. 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

[PATCH] console/font: Refactor font support code selection logic

2013-05-15 Thread Geert Uytterhoeven
to `.find_font' Signed-off-by: Geert Uytterhoeven ge...@linux-m68k.org --- drivers/media/platform/Kconfig |2 +- drivers/staging/media/solo6x10/Kconfig |2 ++ drivers/usb/misc/sisusbvga/Kconfig |1 + drivers/video/console/Kconfig | 12 ++-- drivers/video

[PATCH 1/2] [media] v4l2: SI476X MFD - Do not use binary constants

2013-05-08 Thread Geert Uytterhoeven
Gcc 4.3 doesn't understand binary constanrs (0b*): drivers/media/radio/radio-si476x.c:862:20: error: invalid suffix b1000 on integer constant Hence use a hexadecimal constant (0x*) instead. Signed-off-by: Geert Uytterhoeven ge...@linux-m68k.org Cc: Mauro Carvalho Chehab mche...@redhat.com

[PATCH] [media] dib8000: Fix dib8000_set_frontend() never setting ret

2013-05-03 Thread Geert Uytterhoeven
drivers/media/dvb-frontends/dib8000.c: In function ‘dib8000_set_frontend’: drivers/media/dvb-frontends/dib8000.c:3556: warning: ‘ret’ is used uninitialized in this function Remove the variable and return zero instead. Signed-off-by: Geert Uytterhoeven ge...@linux-m68k.org --- drivers/media/dvb

[PATCH 1/2] [media] anysee: Initialize ret = 0 in anysee_frontend_attach()

2013-04-24 Thread Geert Uytterhoeven
drivers/media/usb/dvb-usb-v2/anysee.c: In function ‘anysee_frontend_attach’: drivers/media/usb/dvb-usb-v2/anysee.c:641: warning: ‘ret’ may be used uninitialized in this function And gcc is right (see the ANYSEE_HW_507T case), so initialize ret to zero to fix this. Signed-off-by: Geert

[PATCH 2/2] [media] anysee: Grammar s/report the/report to/

2013-04-24 Thread Geert Uytterhoeven
Signed-off-by: Geert Uytterhoeven ge...@linux-m68k.org --- drivers/media/usb/dvb-usb-v2/anysee.c |2 +- 1 files changed, 1 insertions(+), 1 deletions(-) diff --git a/drivers/media/usb/dvb-usb-v2/anysee.c b/drivers/media/usb/dvb-usb-v2/anysee.c index 3a1f976..1760fee 100644 --- a/drivers

Re: [PATCH 10/10] drivers: misc: use module_platform_driver_probe()

2013-03-19 Thread Geert Uytterhoeven
drivers that (now) work perfectly fine with module_platform_driver_probe()/platform_driver_probe(), but will start failing suddenly in the future? I guess we need a big fat WARN_ON(-EPROBE_DEFER) in platform_driver_probe() to catch these? Gr{oetje,eeting}s, Geert -- Geert

[PATCH] media/v4l2: VIDEOBUF2_DMA_CONTIG should depend on HAS_DMA

2013-03-04 Thread Geert Uytterhoeven
/v4l2-core/videobuf2-dma-contig.c:387: error: implicit declaration of function ‘dma_get_sgtable’ Make VIDEOBUF2_DMA_CONTIG and VIDEO_SH_VEU (which selects the former and doesn't have a platform dependency) depend on HAS_DMA to fix this. Signed-off-by: Geert Uytterhoeven ge...@linux-m68k.org

Re: [PATCH/RFC 4/4] common: dma-mapping: Move dma_common_*() to linux/dma-mapping.h

2013-01-13 Thread Geert Uytterhoeven
Hi Marek, On Wed, Jan 2, 2013 at 3:22 PM, Marek Szyprowski m.szyprow...@samsung.com wrote: On 12/28/2012 8:23 PM, Geert Uytterhoeven wrote: dma_common_mmap() and dma_common_get_sgtable() are defined in drivers/base/dma-mapping.c, and always compiled if CONFIG_HAS_DMA=y. However

[PATCH 2/4] score: Remove unneeded asm/dma-mapping.h

2012-12-28 Thread Geert Uytterhoeven
It just includes asm-generic/dma-mapping-broken.h, which is already handled by linux/dma-mapping.h for the !CONFIG_HAS_DMA case (score sets CONFIG_NO_DMA=y). Signed-off-by: Geert Uytterhoeven ge...@linux-m68k.org Cc: Chen Liqin liqin.c...@sunplusct.com Cc: Lennox Wu lennox...@gmail.com --- arch

[PATCH/RFC 3/4] avr32/bfin/c6x/cris/frv/m68k/mn10300/parisc/xtensa: Add dummy get_dma_ops()

2012-12-28 Thread Geert Uytterhoeven
Provide dummy versions of get_dma_ops(), as dma_mmap_attrs() and dma_get_sgtable_attrs() (will) need these Signed-off-by: Geert Uytterhoeven ge...@linux-m68k.org --- arch/avr32/include/asm/dma-mapping.h|8 arch/blackfin/include/asm/dma-mapping.h |8 arch/c6x/include

[PATCH/RFC 4/4] common: dma-mapping: Move dma_common_*() to linux/dma-mapping.h

2012-12-28 Thread Geert Uytterhoeven
by returning NULL from their get_dma_ops(). Note that there are no BUG_ON(!ops) checks in other functions in asm-generic/dma-mapping-common.h, - Make struct dma_map_ops *ops const while we're at it. Signed-off-by: Geert Uytterhoeven ge...@linux-m68k.org --- include/asm-generic/dma-mapping

[PATCH 1/4] m68k: Sort out !CONFIG_MMU_SUN3 vs. CONFIG_HAS_DMA

2012-12-28 Thread Geert Uytterhoeven
In two places, we check !CONFIG_MMU_SUN3 while we should check CONFIG_HAS_DMA instead. While fixing this, the check in asm/dma-mapping.h became redundant (linux/dma-mapping.h already handles this case), so just remove it. Signed-off-by: Geert Uytterhoeven ge...@linux-m68k.org --- arch/m68k

<    1   2   3   4   >