Re: [PATCH v3] arm64: dts: renesas: salvator-common: add GPIO extender

2018-01-03 Thread Geert Uytterhoeven
uot;; > + reg = <0x20>; > + gpio-controller; > + #gpio-cells = <2>; interrupts? You can test them using CP16/17 and gpio-keys. > + }; > + > csa_vdd: adc@7c { > compatible = "maxim,max961

Re: [PATCH v4 2/6] clk: renesas: rcar-gen3: Add Z2 clock divider support

2018-01-03 Thread Geert Uytterhoeven
the Z or Z2 registers? Or is that handled automatically by cpufreq during system resume, for both the primary and the secondary CPU cores? 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

Re: [PATCH v3 1/3] arm64: dts: renesas: r8a7795: move scif node into alphabetical order

2018-01-03 Thread Geert Uytterhoeven
On Wed, Jan 3, 2018 at 1:41 PM, Simon Horman wrote: > Move scif node so that sub-nodes of the root node are in > alphabetical order. > > Signed-off-by: Simon Horman Reviewed-by: Geert Uytterhoeven Gr{oetje,eeting}s, Geert -- Geert Uytterhoeven -- The

Re: [PATCH v3 2/3] arm64: dts: renesas: r8a7795: Add OPPs table for cpu devices

2018-01-03 Thread Geert Uytterhoeven
Simon Horman Looks good and matches the bindings. I cannot verify the operating point tuples, though. Reviewed-by: Geert Uytterhoeven Gr{oetje,eeting}s, Geert -- Geert Uytterhoeven -- There's lots of Linux beyond ia32 -- ge...@linux-m68k.org In personal conversations w

Re: [PATCH v3 3/3] arm64: dts: renesas: r8a7796: Add OPPs table for cpu devices

2018-01-03 Thread Geert Uytterhoeven
Simon Horman Looks good and matches the bindings. I cannot verify the operating point tuples, though. Reviewed-by: Geert Uytterhoeven Gr{oetje,eeting}s, Geert -- Geert Uytterhoeven -- There's lots of Linux beyond ia32 -- ge...@linux-m68k.org In personal conversations w

[PATCH] spi: sh-msiof: Fix timeout failures for TX-only DMA transfers

2018-01-03 Thread Geert Uytterhoeven
74x164 spi2.0: SPI transfer failed: -110 spi_master spi2: failed to transfer one message from queue 74x164 spi2.0: Failed writing: -110 Fix this by waiting explicitly until the TX FIFO has been emptied. Based on a patch in the BSP by Hiromitsu Yamasaki. Signed-off-by: Geert

Re: [PATCH 2/8] spi: sh-msiof: Fix DMA transfer size check

2018-01-03 Thread Geert Uytterhoeven
Hi Dirk, On Thu, Sep 7, 2017 at 11:12 AM, Geert Uytterhoeven wrote: > On Thu, Sep 7, 2017 at 11:05 AM, Dirk Behme wrote: >> On 07.09.2017 10:59, Geert Uytterhoeven wrote: >>> On Thu, Sep 7, 2017 at 10:42 AM, Dirk Behme >>> wrote: >>>> On 07.09.2017 10:39,

Re: [PATCH 2/8] spi: sh-msiof: Fix DMA transfer size check

2018-01-04 Thread Geert Uytterhoeven
Hi Dirk, On Thu, Jan 4, 2018 at 8:19 AM, Dirk Behme wrote: > On 03.01.2018 18:25, Geert Uytterhoeven wrote: >> On Thu, Sep 7, 2017 at 11:12 AM, Geert Uytterhoeven >> wrote: >>> On Thu, Sep 7, 2017 at 11:05 AM, Dirk Behme >>> wrote: >>>> On 07.09.2

Re: Applied "spi: sh-msiof: Fix timeout failures for TX-only DMA transfers" to the spi tree

2018-01-04 Thread Geert Uytterhoeven
automatically, as it has the magic word "Fix" in the one-line summary. 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 wh

Re: [PATCH] clk: renesas: Stop enabling legacy DT clock support by default

2018-01-04 Thread Geert Uytterhoeven
On Tue, Jan 2, 2018 at 4:18 PM, Geert Uytterhoeven wrote: > Since v4.15-rc1, the DTS files for all R-Car Gen2 SoCs have been > converted to the new CPG/MSSR bindings. Hence it is now safe to no > longer enable legacy DT clock support by default. > > Signed-off-by: Geert Uytterhoe

Re: [PATCH] clk: renesas: r8a7796: Add FDP clock

2018-01-04 Thread Geert Uytterhoeven
On Tue, Jan 2, 2018 at 4:54 PM, Laurent Pinchart wrote: > On Tuesday, 2 January 2018 17:21:43 EET Geert Uytterhoeven wrote: >> From: ABE Hiroshige >> >> This patch adds FDP1-0 clock to the R8A7796 SoC. >> >> Signed-off-by: ABE Hiroshige >> Signed-off-by: T

Re: [PATCH 1/2] arm64: dts: r8a7795: update register size for thermal

2018-01-05 Thread Geert Uytterhoeven
fused or not. > > Signed-off-by: Niklas Söderlund Reviewed-by: Geert Uytterhoeven > --- a/arch/arm64/boot/dts/renesas/r8a7795.dtsi > +++ b/arch/arm64/boot/dts/renesas/r8a7795.dtsi > @@ -2385,7 +2385,7 @@ > > tsc: thermal@e6198000 { > c

Re: [PATCH 2/2] arm64: dts: r8a7796: update register size for thermal

2018-01-05 Thread Geert Uytterhoeven
fused or not. > > Signed-off-by: Niklas Söderlund Reviewed-by: 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.

Re: [PATCH 2/3] dt-bindings: usb: renesas_usbhs: Add support for RZ/A1

2018-01-05 Thread Geert Uytterhoeven
On Thu, Jan 4, 2018 at 9:01 PM, Chris Brandt wrote: > Document support for RZ/A1 SoCs > > Signed-off-by: Chris Brandt Reviewed-by: Geert Uytterhoeven Gr{oetje,eeting}s, Geert -- Geert Uytterhoeven -- There's lots of Linux beyond ia32 -- ge...@linu

Re: [PATCH 3/3] ARM: dts: r7s72100: add USB device to device tree

2018-01-05 Thread Geert Uytterhoeven
On Thu, Jan 4, 2018 at 9:01 PM, Chris Brandt wrote: > Add USB device support. > > Signed-off-by: Chris Brandt Reviewed-by: Geert Uytterhoeven Minor nits below. > --- a/arch/arm/boot/dts/r7s72100.dtsi > +++ b/arch/arm/boot/dts/r7s72100.dtsi > @@ -667,4 +667,24 @@ >

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

2018-01-05 Thread Geert Uytterhoeven
nesas,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 different (yes I

Re: [PATCH v2 0/3] renesas: irqchip: Use WAKEUP_PATH driver PM flag

2018-01-05 Thread Geert Uytterhoeven
Hi Rafael, On Sun, Dec 31, 2017 at 12:47 PM, Rafael J. Wysocki wrote: > On Sun, Dec 31, 2017 at 10:22 AM, Geert Uytterhoeven > wrote: >> On Sun, Dec 31, 2017 at 1:56 AM, Rafael J. Wysocki wrote: >>> On Fri, Dec 29, 2017 at 2:31 PM, Ulf Hansson wrote: >>

Re: [PATCH] clk: renesas: Stop enabling legacy DT clock support by default

2018-01-05 Thread Geert Uytterhoeven
On Thu, Jan 4, 2018 at 11:35 AM, Geert Uytterhoeven wrote: > On Tue, Jan 2, 2018 at 4:18 PM, Geert Uytterhoeven > wrote: >> Since v4.15-rc1, the DTS files for all R-Car Gen2 SoCs have been >> converted to the new CPG/MSSR bindings. Hence it is now safe to no >> longe

[git pull] clk: renesas: Updates for v4.16 (take two)

2018-01-05 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 "

[git pull] pinctrl: sh-pfc: Updates for v4.16 (take two)

2018-01-05 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

Re: [PATCH 1/2] arm64: dts: r8a7795: update register size for thermal

2018-01-05 Thread Geert Uytterhoeven
Hi Niklas, On Fri, Jan 5, 2018 at 1:24 PM, Niklas Söderlund wrote: > On 2018-01-05 09:47:11 +0100, Geert Uytterhoeven wrote: >> On Thu, Jan 4, 2018 at 10:31 PM, Niklas Söderlund >> wrote: >> > To be able to read fused calibration values from hardware the size of >&

Re: [PATCH 3/3] ARM: dts: r7s72100: add USB device to device tree

2018-01-05 Thread Geert Uytterhoeven
Hi Chris, On Fri, Jan 5, 2018 at 1:54 PM, Chris Brandt wrote: > On Friday, January 05, 2018, Geert Uytterhoeven wrote: >> > + interrupts = ; >> >> "41", all other interrupt properties already have the SPI offset >> subtracted? > > The

Re: [PATCH v4 2/6] clk: renesas: rcar-gen3: Add Z2 clock divider support

2018-01-05 Thread Geert Uytterhoeven
Hi Simon, On Fri, Jan 5, 2018 at 3:04 PM, Simon Horman wrote: > On Wed, Jan 03, 2018 at 01:47:08PM +0100, Geert Uytterhoeven wrote: >> On Wed, Jan 3, 2018 at 1:18 PM, Simon Horman >> wrote: >> > From: Takeshi Kihara >> > >> > This patch adds Z

Re: [PATCH v2 3/3] ARM: dts: r7s72100: add USB device to device tree

2018-01-05 Thread Geert Uytterhoeven
On Fri, Jan 5, 2018 at 7:46 PM, Chris Brandt wrote: > Add USB device support. > > Signed-off-by: Chris Brandt > Reviewed-by: Geert Uytterhoeven > --- a/arch/arm/boot/dts/r7s72100.dtsi > +++ b/arch/arm/boot/dts/r7s72100.dtsi > @@ -667,4 +667,24 @@ > powe

Re: [PATCH v2 1/3] usb: renesas_usbhs: Add support for RZ/A1

2018-01-05 Thread Geert Uytterhoeven
_usbhs/rza.c > @@ -0,0 +1,52 @@ > +// SPDX-License-Identifier: GPL-1.0+ Not GPL-2.0? > +++ b/drivers/usb/renesas_usbhs/rza.h > @@ -0,0 +1,4 @@ > +// SPDX-License-Identifier: GPL-2.0 like here? Gr{oetje,eeting}s, Geert -- Geert Uytterhoeven -- There's lots of L

Re: [PATCH v3 0/3] usb: renesas_usbhs: Add RZ/A1 support

2018-01-07 Thread Geert Uytterhoeven
applicable_ compatible values, ordered from most-specific to least-specific, 3. Drivers should match against the least-specific compatible value to get the job done. Gr{oetje,eeting}s, Geert -- Geert Uytterhoeven -- There's lots of Linux beyond ia32 -- ge...

Re: [PATCH v3 2/3] dt-bindings: usb: renesas_usbhs: Add support for RZ/A1

2018-01-07 Thread Geert Uytterhoeven
Hi Chris, On Fri, Jan 5, 2018 at 11:46 PM, Chris Brandt wrote: > Document support for RZ/A1 SoCs > > Signed-off-by: Chris Brandt > Reviewed-by: Geert Uytterhoeven > --- > v3: > * Removed "renesas,usbhs-r7s72100" Please don't, cfr. my reply to

Re: [PATCH v3 3/3] ARM: dts: r7s72100: add USB device to device tree

2018-01-07 Thread Geert Uytterhoeven
On Fri, Jan 5, 2018 at 11:46 PM, Chris Brandt wrote: > Add USB device support. > > Signed-off-by: Chris Brandt > Reviewed-by: Geert Uytterhoeven > --- > v3: > * Changed from "renesas,usbhs-r7s72100" to "renesas,rza1-usbhs" Please use both, cfr. my repl

Re: [PATCH v2 0/2] arm64: dts: r8a779[56]: update register size for thermal

2018-01-08 Thread Geert Uytterhoeven
e policy #1, but covered a register not documented in the datasheet. Note that some drivers (SD?) use the register block lengths to check if a feature is available or not. Ideally they should use the compatible value instead. Gr{oetje,eeting}s, Geert -- Geert Uytterhoeven -

Re: [PATCH v4 2/6] clk: renesas: rcar-gen3: Add Z2 clock divider support

2018-01-08 Thread Geert Uytterhoeven
Hi Simon, On Mon, Jan 8, 2018 at 9:02 AM, Simon Horman wrote: > On Fri, Jan 05, 2018 at 03:35:13PM +0100, Geert Uytterhoeven wrote: >> On Fri, Jan 5, 2018 at 3:04 PM, Simon Horman wrote: >> > On Wed, Jan 03, 2018 at 01:47:08PM +0100, Geert Uytterhoeven wrote: >> >&g

Re: [PATCH v3 0/3] usb: renesas_usbhs: Add RZ/A1 support

2018-01-08 Thread Geert Uytterhoeven
On Mon, Jan 8, 2018 at 12:59 PM, Chris Brandt wrote: > Hi Geert and Simon, > > > On Monday, January 08, 2018, Geert Uytterhoeven wrote: >> Thanks for the update, but I think there has been a misunderstanding. >> I didn't mean to drop "renesas,usbhs-r7s72100"

Re: [PATCH v3 0/3] usb: renesas_usbhs: Add RZ/A1 support

2018-01-08 Thread Geert Uytterhoeven
Hi Chris, On Mon, Jan 8, 2018 at 12:59 PM, Chris Brandt wrote: > On Monday, January 08, 2018, Geert Uytterhoeven wrote: >> Thanks for the update, but I think there has been a misunderstanding. >> I didn't mean to drop "renesas,usbhs-r7s72100" everywhere, only from

[PATCH] net: phy: Fix phy_modify() semantic difference fallout

2018-01-09 Thread Geert Uytterhoeven
: fea23fb591cce995 ("net: phy: convert read-modify-write to phy_modify()") Signed-off-by: Geert Uytterhoeven --- Alternatively, __phy_modify() could be changed to follow __phy_write() semantics? --- drivers/net/phy/at803x.c | 4 +++- drivers/net/phy/phy_dev

Re: [PATCH v2] PM / runtime: Rework pm_runtime_force_suspend/resume()

2018-01-09 Thread Geert Uytterhoeven
nstead does not fix the crash. With more debug code added, it seems the EHCI module clocks (701-703) are enabled earlier than before. I guess this triggers the workqueue to perform an operation while another related device (HSUSB 704?) is still disabled? Gr{oetje,eeting}s,

Re: [PATCH] pinctrl: sh-pfc: Use seq_puts() in sh_pfc_pin_dbg_show()

2018-01-09 Thread Geert Uytterhoeven
> This issue was detected by using the Coccinelle software. > > Signed-off-by: Markus Elfring Reviewed-by: Geert Uytterhoeven i.e. will queue in sh-pfc-for-v4.17. Gr{oetje,eeting}s, Geert -- Geert Uytterhoeven -- There's lots of Linux beyond ia32 -- ge...

Re: [PATCH v2] PM / runtime: Rework pm_runtime_force_suspend/resume()

2018-01-09 Thread Geert Uytterhoeven
Hi Ulf, On Tue, Jan 9, 2018 at 3:27 PM, Ulf Hansson wrote: > On 9 January 2018 at 14:37, Geert Uytterhoeven wrote: >> On Wed, Jan 3, 2018 at 12:06 PM, Rafael J. Wysocki >> wrote: >>> From: Rafael J. Wysocki >>> One of the limitations of pm_runtime_force_

Re: [PATCH] net: phy: Fix phy_modify() semantic difference fallout

2018-01-09 Thread Geert Uytterhoeven
Hi Russell, On Tue, Jan 9, 2018 at 3:22 PM, Russell King - ARM Linux wrote: > On Tue, Jan 09, 2018 at 03:10:08PM +0100, Andrew Lunn wrote: >> On Tue, Jan 09, 2018 at 12:11:21PM +0100, Geert Uytterhoeven wrote: >> > In case of success, the return values of (__)phy_write() and &

Re: [PATCH v2] PM / runtime: Rework pm_runtime_force_suspend/resume()

2018-01-09 Thread Geert Uytterhoeven
Hi Rafael, CC usb On Tue, Jan 9, 2018 at 4:00 PM, Rafael J. Wysocki wrote: > On Tue, Jan 9, 2018 at 2:37 PM, Geert Uytterhoeven > wrote: >> On Wed, Jan 3, 2018 at 12:06 PM, Rafael J. Wysocki >> wrote: >>> From: Rafael J. Wysocki >>> >>> One o

renesas-drivers-2018-01-09-v4.15-rc7

2018-01-09 Thread Geert Uytterhoeven
ext 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 "

[PATCH] gpio: Add missing open drain/source handling to gpiod_set_value_cansleep()

2018-01-09 Thread Geert Uytterhoeven
a new common helper __gpiod_set_value_nocheck(). Fixes: 02e479808b5d62f8 ("gpio: Alter semantics of *raw* operations to actually be raw") Signed-off-by: Geert Uytterhoeven --- The bad gpio commit is in v4.15-rc1 and later. The i2c commit exposing this is not yet upstream, but in i2c/for-next. --

Re: [PATCH] net: phy: Fix phy_modify() semantic difference fallout

2018-01-09 Thread Geert Uytterhoeven
Hi Russell, On Tue, Jan 9, 2018 at 3:22 PM, Russell King - ARM Linux wrote: > On Tue, Jan 09, 2018 at 03:10:08PM +0100, Andrew Lunn wrote: >> On Tue, Jan 09, 2018 at 12:11:21PM +0100, Geert Uytterhoeven wrote: >> > In case of success, the return values of (__)phy_write() and &

Re: [PATCH] net: phy: Fix phy_modify() semantic difference fallout

2018-01-09 Thread Geert Uytterhoeven
Hi Russell, On Tue, Jan 9, 2018 at 7:31 PM, Russell King - ARM Linux wrote: > On Tue, Jan 09, 2018 at 07:25:40PM +0100, Geert Uytterhoeven wrote: >> On Tue, Jan 9, 2018 at 3:22 PM, Russell King - ARM Linux >> wrote: >> > On Tue, Jan 09, 2018 at 03:10:08PM +0100, Andrew

Re: [PATCH] gpio: Add missing open drain/source handling to gpiod_set_value_cansleep()

2018-01-10 Thread Geert Uytterhoeven
Hi Linus, On Wed, Jan 10, 2018 at 2:17 PM, Linus Walleij wrote: > On Tue, Jan 9, 2018 at 7:08 PM, Geert Uytterhoeven > wrote: > >> Since commit f11a04464ae57e8d ("i2c: gpio: Enable working over slow >> can_sleep GPIOs"), probing the i2c RTC connected to an i2c-g

Re: [PATCH] net: phy: Fix phy_modify() semantic difference fallout

2018-01-11 Thread Geert Uytterhoeven
On Thu, Jan 11, 2018 at 4:53 PM, Russell King - ARM Linux wrote: > On Thu, Jan 11, 2018 at 10:48:35AM -0500, David Miller wrote: >> From: Geert Uytterhoeven >> Date: Tue, 9 Jan 2018 12:11:21 +0100 >> >> > In case of success, the return values of (__)phy_write() and

Re: [PATCH] net: phy: Fix phy_modify() semantic difference fallout

2018-01-11 Thread Geert Uytterhoeven
On Thu, Jan 11, 2018 at 4:54 PM, Geert Uytterhoeven wrote: > On Thu, Jan 11, 2018 at 4:53 PM, Russell King - ARM Linux > wrote: >> On Thu, Jan 11, 2018 at 10:48:35AM -0500, David Miller wrote: >>> From: Geert Uytterhoeven >>> Date: Tue, 9 Jan 2018 12:11:21 +010

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

2018-01-12 Thread Geert Uytterhoeven
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, Geert -- G

Re: [PATCH 01/10] dt-bindings: display: renesas: Add R-Car LVDS encoder DT bindings

2018-01-12 Thread Geert Uytterhoeven
eg: Base address and length for the memory-mapped registers > +- clocks: A phandle + clock-specifier pair for the functional clock Please document the "resets" property, too, and add it to the example. According to the rcar-3.6.0 BSP, reset support is needed desperately. Gr{oetje,eetin

Re: [PATCH 00/10] R-Car DU: Convert LVDS code to bridge driver

2018-01-12 Thread Geert Uytterhoeven
ice > tree at runtime to convert the legacy bindings to the new ones. Looks like we will have to postpone the R-Car Gen2 Modern DT flag day again by a few kernel releases? Gr{oetje,eeting}s, Geert -- Geert Uytterhoeven -- There's lots of Linux beyond ia32 -- g

Re: [PATCH 02/10] dt-bindings: display: renesas: Deprecate LVDS support in the DU bindings

2018-01-12 Thread Geert Uytterhoeven
, but removing it completely. Do you want to keep a reference instead? 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 talkin

Re: [PATCH 03/10] drm: rcar-du: Fix legacy DT to create LVDS encoder nodes

2018-01-12 Thread Geert Uytterhoeven
verlay__/ports/port@0"); Many strings with similar prefixes or substrings? Would it make sense to e.g. locate "/fragment@0/__overlay__/lvds/ports" first, and continue from there? 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

Re: [PATCH 01/10] dt-bindings: display: renesas: Add R-Car LVDS encoder DT bindings

2018-01-12 Thread Geert Uytterhoeven
inding, please use "renesas,-lvds". BTW, would it make sense to use "renesas,-du" for the new DU binding, too? Or have you reserved that for the future version that will have a one-to-one mapping between device nodes and DU channels? ;-) Gr{oetje,eeting}s,

Re: [PATCH v2] PM / runtime: Rework pm_runtime_force_suspend/resume()

2018-01-12 Thread Geert Uytterhoeven
Hi Rafael, On Tue, Jan 9, 2018 at 5:25 PM, Rafael J. Wysocki wrote: > On Tuesday, January 9, 2018 4:00:35 PM CET Rafael J. Wysocki wrote: >> On Tue, Jan 9, 2018 at 2:37 PM, Geert Uytterhoeven >> wrote: >> > On Wed, Jan 3, 2018 at 12:06 PM, Rafael J. Wysocki >>

Re: [PATCH v2] PM / runtime: Rework pm_runtime_force_suspend/resume()

2018-01-12 Thread Geert Uytterhoeven
Hi Rafael, On Tue, Jan 9, 2018 at 5:28 PM, Rafael J. Wysocki wrote: > On Tuesday, January 9, 2018 5:03:18 PM CET Rafael J. Wysocki wrote: >> On Tue, Jan 9, 2018 at 4:30 PM, Geert Uytterhoeven >> wrote: >> > On Tue, Jan 9, 2018 at 4:00 PM, Rafael J. Wysocki >>

Re: [PATCH 0/2] PM / core: genpd fix and pm_runtime_force_suspend|resume() rework

2018-01-12 Thread Geert Uytterhoeven
ssue seems to be caused by patch [1/2]. But I have no idea what's really happening, and why the two systems behave differently. Oh well, have a nice weekend! Gr{oetje,eeting}s, Geert -- Geert Uytterhoeven -- There's lots of Linux beyond ia32 -- ge...@linux-m68k.org

Re: [PATCH 0/2] PM / core: genpd fix and pm_runtime_force_suspend|resume() rework

2018-01-14 Thread Geert Uytterhoeven
Hi Rafael, On Sat, Jan 13, 2018 at 1:38 AM, Rafael J. Wysocki wrote: > On Friday, January 12, 2018 3:31:09 PM CET Geert Uytterhoeven wrote: >> On Fri, Jan 12, 2018 at 2:00 PM, Rafael J. Wysocki >> wrote: >> > This comes from the recent discussion/testing eff

Re: [PATCH 01/10] dt-bindings: display: renesas: Add R-Car LVDS encoder DT bindings

2018-01-15 Thread Geert Uytterhoeven
Hi Laurent, On Mon, Jan 15, 2018 at 7:59 AM, Laurent Pinchart wrote: > On Monday, 15 January 2018 08:55:29 EET Simon Horman wrote: >> On Fri, Jan 12, 2018 at 03:29:48PM +0200, Laurent Pinchart wrote: >> > On Friday, 12 January 2018 12:13:18 EET Geert Uytterhoeven wrote: >

Re: [PATCH 1/2] sh_eth: gather all TSU init code in one place

2018-01-15 Thread Geert Uytterhoeven
gt; simplify the probing code a bit... > > Signed-off-by: Sergei Shtylyov Reviewed-by: 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

Re: [PATCH 2/2] sh_eth: get Ether port # only when needed

2018-01-15 Thread Geert Uytterhoeven
d' > only when we actually need it (and not recalculate it each time)... > > Signed-off-by: Sergei Shtylyov Reviewed-by: Geert Uytterhoeven Gr{oetje,eeting}s, Geert -- Geert Uytterhoeven -- There's lots of Linux beyond ia32 -- ge...@linux-m68k.or

Re: [PATCH 0/2] PM / core: genpd fix and pm_runtime_force_suspend|resume() rework

2018-01-15 Thread Geert Uytterhoeven
Hi Rafael, On Mon, Jan 15, 2018 at 1:04 AM, Rafael J. Wysocki wrote: > On Sun, Jan 14, 2018 at 10:48 AM, Geert Uytterhoeven > wrote: >> On Sat, Jan 13, 2018 at 1:38 AM, Rafael J. Wysocki >> wrote: >>> On Friday, January 12, 2018 3:31:09 PM CET Geert Uytterhoeven

Re: [PATCH v2 04/12] drm: rcar-du: Convert LVDS encoder code to bridge driver

2018-01-15 Thread Geert Uytterhoeven
d thus calling pm_runtime_get_sync() here instead? That will make the driver future-proof w.r.t. LVDS blocks in power domains. You do need a "power-domains" property, though, which may complicate DT runtime patching. Gr{oetje,eeting}s, Geert -- Geert Uytterh

Re: [PATCH 0/2] PM / core: genpd fix and pm_runtime_force_suspend|resume() rework

2018-01-15 Thread Geert Uytterhoeven
Hi Rafael, On Mon, Jan 15, 2018 at 9:16 AM, Geert Uytterhoeven wrote: > On Mon, Jan 15, 2018 at 1:04 AM, Rafael J. Wysocki wrote: >> On Sun, Jan 14, 2018 at 10:48 AM, Geert Uytterhoeven >> wrote: >>> On Sat, Jan 13, 2018 at 1:38 AM, Rafael J. Wysocki >>>

Re: [PATCH v2 03/12] drm: rcar-du: Fix legacy DT to create LVDS encoder nodes

2018-01-16 Thread Geert Uytterhoeven
t;> OF_OVERLAY should probably select OF_FLATTREE. I guess in theory, we >> could have an overlay from a non-FDT source... Currently the select is needed for of_fdt_unflatten_tree() only, which is not used by the core OF_OVERLAY code. So you could build an overlay in memory yourself, and pa

Re: [PATCH v2 03/12] drm: rcar-du: Fix legacy DT to create LVDS encoder nodes

2018-01-16 Thread Geert Uytterhoeven
Hi Rob, On Tue, Jan 16, 2018 at 4:08 PM, Rob Herring wrote: > On Tue, Jan 16, 2018 at 2:56 AM, Geert Uytterhoeven > wrote: >> On Mon, Jan 15, 2018 at 7:01 PM, Laurent Pinchart >> wrote: >>> On Monday, 15 January 2018 19:09:53 EET Rob Herring wrote: >>>> O

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

2018-01-17 Thread Geert Uytterhoeven
err_unlock; > + } > + } > + > INIT_LIST_HEAD(¬ifier->waiting); > INIT_LIST_HEAD(¬ifier->done); > > - mutex_lock(&list_lock); > - > for (i = 0; i < notifier->num_subdevs; i++) { > asd = notifier->s

Re: [PATCH 0/2] PM / core: genpd fix and pm_runtime_force_suspend|resume() rework

2018-01-17 Thread Geert Uytterhoeven
Hi Rafael, On Mon, Jan 15, 2018 at 5:17 PM, Rafael J. Wysocki wrote: > On Mon, Jan 15, 2018 at 3:26 PM, Ulf Hansson wrote: >> On 15 January 2018 at 14:22, Geert Uytterhoeven wrote: > > [cut] > >>> >>> I did miss a small difference in topology: in pm/linux-ne

[PATCH] dmaengine: rcar-dmac: Make DMAC reinit during system resume explicit

2018-01-17 Thread Geert Uytterhoeven
{suspend,resume}() as the system sleep callbacks instead. Use SET_LATE_SYSTEM_SLEEP_PM_OPS() as DMA engines must be initialized before all DMA slave devices. Suggested-by: Ulf Hansson Signed-off-by: Geert Uytterhoeven --- This is a dependency for "[PATCH 1/2] PM / genpd: Stop/start devices wi

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

2018-01-17 Thread Geert Uytterhoeven
ned-off-by: Wei Yongjun Reviewed-by: 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

Re: [PATCH 1/2] clk: renesas: r8a7796: Add ZG clock

2018-01-17 Thread Geert Uytterhoeven
On Wed, Jan 17, 2018 at 4:41 PM, Ulrich Hecht wrote: > This patch adds ZG clock for R8A7796 SoC. > > Signed-off-by: Takeshi Kihara > Signed-off-by: Ulrich Hecht Reviewed-by: Geert Uytterhoeven > --- a/drivers/clk/renesas/r8a7796-cpg-mssr.c > +++ b/drivers/clk/renesas/

Re: [PATCH 2/2] clk: renesas: rcar-gen3: Add ZG clock divider support

2018-01-17 Thread Geert Uytterhoeven
similar, both differences need to be taken into account. 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

Re: [RFC] arm64: renesas_defconfig: add driver for PCA9654

2018-01-19 Thread Geert Uytterhoeven
RMAL=y > CONFIG_THERMAL_GOV_POWER_ALLOCATOR=y 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

Re: [PATCH v2 01/16] ARM: dts: r8a7790: consistently use single space after =

2018-01-19 Thread Geert Uytterhoeven
> > This patch should not introduce any functional change. > > Signed-off-by: Simon Horman Reviewed-by: Geert Uytterhoeven Gr{oetje,eeting}s, Geert -- Geert Uytterhoeven -- There's lots of Linux beyond ia32 -- ge...@linux-m68k.org In personal conver

Re: [PATCH v2 02/16] ARM: dts: r8a7790: add soc node

2018-01-19 Thread Geert Uytterhoeven
node. > > This patch should not introduce any functional change. > > Signed-off-by: Simon Horman Reviewed-by: Geert Uytterhoeven Gr{oetje,eeting}s, Geert -- Geert Uytterhoeven -- There's lots of Linux beyond ia32 -- ge...@linux-m68k.org In personal

Re: [PATCH v2 02/16] ARM: dts: r8a7790: add soc node

2018-01-19 Thread Geert Uytterhoeven
checked the > documentation and as far as I can make out this is how it should be. But > I'm not certain why :-) Please see section 2.3.8 in the Device Tree Specification, either from ePAPR, or at https://www.devicetree.org/. Gr{oetje,eeting}s, Geert -- Gee

Re: [PATCH v2 04/16] ARM: dts: r8a7790: sort subnodes of root node

2018-01-19 Thread Geert Uytterhoeven
On Wed, Jan 17, 2018 at 5:17 PM, Simon Horman wrote: > Sort subnodes of root node to aid maintenance. > > This patch should not introduce any functional change. > > Signed-off-by: Simon Horman Reviewed-by: Geert Uytterhoeven Gr{oetje,eeting}s,

Re: [PATCH v2 05/16] ARM: dts: r8a7791: consistently use single space after =

2018-01-19 Thread Geert Uytterhoeven
> > This patch should not introduce any functional change. > > Signed-off-by: Simon Horman Reviewed-by: Geert Uytterhoeven Gr{oetje,eeting}s, Geert -- Geert Uytterhoeven -- There's lots of Linux beyond ia32 -- ge...@linux-m68k.org In personal conver

Re: [PATCH v2 03/16] ARM: dts: r8a7790: sort subnodes of soc node

2018-01-19 Thread Geert Uytterhoeven
t; > Signed-off-by: Simon Horman Reviewed-by: Geert Uytterhoeven The sort order for the various SCIF variants is still a bit odd: scifa0: serial@e6c4 { scifa1: serial@e6c5 { scifa2: serial@e6c6 { scifb0: serial@e6c2 { scifb1: serial@e6c3

Re: [PATCH v2 06/16] ARM: dts: r8a7791: add soc node

2018-01-19 Thread Geert Uytterhoeven
node. > > This patch should not introduce any functional change. > > Signed-off-by: Simon Horman Reviewed-by: Geert Uytterhoeven Gr{oetje,eeting}s, Geert -- Geert Uytterhoeven -- There's lots of Linux beyond ia32 -- ge...@linux-m68k.org In personal

Re: [PATCH v2 07/16] ARM: dts: r8a7791: sort subnodes of root node

2018-01-19 Thread Geert Uytterhoeven
On Wed, Jan 17, 2018 at 5:17 PM, Simon Horman wrote: > Sort subnodes of root node to aid maintenance. > > This patch should not introduce any functional change. > > Signed-off-by: Simon Horman Reviewed-by: Geert Uytterhoeven Gr{oetje,eeting}s,

Re: [PATCH v2 08/16] ARM: dts: r8a7792: sort subnodes of soc node

2018-01-19 Thread Geert Uytterhoeven
Signed-off-by: Simon Horman Reviewed-by: Geert Uytterhoeven Albeit the same comment about SCIF variant sort order as for r8a7790. Gr{oetje,eeting}s, Geert -- Geert Uytterhoeven -- There's lots of Linux beyond ia32 -- ge...@linux-m68k.org In personal conversation

Re: [PATCH v2 09/16] ARM: dts: r8a7793: consistently use single space after =

2018-01-19 Thread Geert Uytterhoeven
should not introduce any functional change. > > Signed-off-by: Simon Horman Reviewed-by: Geert Uytterhoeven Gr{oetje,eeting}s, Geert -- Geert Uytterhoeven -- There's lots of Linux beyond ia32 -- ge...@linux-m68k.org In personal conversations with technica

Re: [PATCH v2 10/16] ARM: dts: r8a7793: add soc node

2018-01-19 Thread Geert Uytterhoeven
node. > > This patch should not introduce any functional change. > > Signed-off-by: Simon Horman Reviewed-by: Geert Uytterhoeven Gr{oetje,eeting}s, Geert -- Geert Uytterhoeven -- There's lots of Linux beyond ia32 -- ge...@linux-m68k.org In personal

Re: [PATCH v2 13/16] ARM: dts: r8a7794: consistently use single space after =

2018-01-19 Thread Geert Uytterhoeven
On Wed, Jan 17, 2018 at 5:17 PM, Simon Horman wrote: > Consistently use a single space after a =. > > This patch removes instances where a tab is used instead. > > This patch should not introduce any functional change. > > Signed-off-by: Simon Horman Reviewed-by: Geert Uy

Re: [PATCH v2 12/16] ARM: dts: r8a7793: sort subnodes of root node

2018-01-19 Thread Geert Uytterhoeven
Signed-off-by: Simon Horman Reviewed-by: 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 ta

Re: [PATCH v2 11/16] ARM: dts: r8a7793: sort subnodes of soc node

2018-01-19 Thread Geert Uytterhoeven
Signed-off-by: Simon Horman Reviewed-by: Geert Uytterhoeven Same comment for SCIF variant sort order. 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

Re: [PATCH v2 14/16] ARM: dts: r8a7794: add soc node

2018-01-19 Thread Geert Uytterhoeven
node. > > This patch should not introduce any functional change. > > Signed-off-by: Simon Horman Reviewed-by: Geert Uytterhoeven Gr{oetje,eeting}s, Geert -- Geert Uytterhoeven -- There's lots of Linux beyond ia32 -- ge...@linux-m68k.org In personal

Re: [PATCH v2 15/16] ARM: dts: r8a7794: sort subnodes of soc node

2018-01-19 Thread Geert Uytterhoeven
Signed-off-by: Simon Horman Reviewed-by: Geert Uytterhoeven SCIF sort order? 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 hacke

Re: [PATCH v2 16/16] ARM: dts: r8a7794: sort subnodes of root node

2018-01-19 Thread Geert Uytterhoeven
Signed-off-by: Simon Horman Reviewed-by: 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 ta

Re: [PATCH v2 0/16] ARM: r8a7790: add soc node

2018-01-19 Thread Geert Uytterhoeven
fi ;; esac done if [ "$dts1" == "" -o "$dts2" == "" ]; then usage fi soc1=$(basename $dts1) soc2=$(basename $dts2) soc1=${soc1%%[-.]*} soc2=${soc2%%[-.]*} colordiff -u $options \ --label $dts1 <(sed -e "s/$soc

Re: [PATCH] arm64: renesas_defconfig: add driver for PCA9654

2018-01-19 Thread Geert Uytterhoeven
On Fri, Jan 19, 2018 at 12:38 PM, Wolfram Sang wrote: > We enabled it in the DTS, so we should have it here, too. > > Signed-off-by: Wolfram Sang Reviewed-by: Geert Uytterhoeven Gr{oetje,eeting}s, Geert -- Geert Uytterhoeven -- There's lots of Linux bey

Re: [PATCH 4/5] arm64: dts: renesas: r8a77970: add LVDS support

2018-01-19 Thread Geert Uytterhoeven
t;1>; > + lvds_out: endpoint { > }; > }; > }; > Gr{oetje,eeting}s, Geert -- Geert Uytterhoeven -- There'

Re: [GIT PULL FOR renesas-drivers] iommu/ipmmu-vmsa: Add r8a779(6|70|95) support

2018-01-23 Thread Geert Uytterhoeven
F_DECLARE(ipmmu_r8a77995_iommu_of, "renesas,ipmmu-r8a77995"); MODULE_DESCRIPTION("IOMMU API for Renesas VMSA-compatible IPMMU"); MODULE_AUTHOR("Laurent Pinchart "); 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

renesas-drivers-2018-01-23-v4.15-rc9

2018-01-23 Thread Geert Uytterhoeven
el.org/pub/scm/linux/kernel/git/lee/mfd.git#for-mfd-next - git://git.kernel.org/pub/scm/linux/kernel/git/robh/linux.git#for-next Gr{oetje,eeting}s, Geert -- Geert Uytterhoeven -- There's lots of Linux beyond ia32 -- ge...@linux-m68k.o

Re: [PATCH 1/3] dt-bindings: iommu: ipmmu-vmsa: Add device tree support for r8a774[35]

2018-01-24 Thread Geert Uytterhoeven
On Wed, Jan 24, 2018 at 4:42 PM, Biju Das wrote: > Document r8a774[35] specific compatible strings. The Renesas RZ/G1[ME] > (r8a774[35]) IPMMU are identical to the R-Car Gen2 family. > > Signed-off-by: Biju Das > Reviewed-by: Chris Paterson Reviewed-by: Geert Uytterhoeven Gr

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

2018-01-25 Thread Geert Uytterhoeven
*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: [RFC PATCH v2 1/1] of: introduce event tracepoints for dynamic device_node lifecyle

2018-01-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 Torvalds

Re: [PATCH v2 1/3] dt-bindings: timer: renesas, cmt: Document r8a774[35] CMT support

2018-01-25 Thread Geert Uytterhoeven
t;>> r8a774[35] CMT support >>> >>> Document SoC specific compatible strings for r8a7743 and r8a7745. >>> No driver change is needed as the fallback strings will activate >>> the right code. >>> >>> Signed-off-by: Fabrizio Castro >>>

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

2018-01-25 Thread Geert Uytterhoeven
r 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 -- Geert Uytterhoeven -- There's lots of Linux beyond ia32 --

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

2018-01-25 Thread Geert Uytterhoeven
Hi Jacopo, On Thu, Jan 25, 2018 at 3:14 PM, jacopo mondi 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) >> >> On Th

Re: [RFC 01/37] ARM: shmobile: Add watchdog support

2018-01-26 Thread Geert Uytterhoeven
r0, r1, #RWTCSRA_WOVF > + cmp r0, #RWTCSRA_WOVF > + bne shmobile_smp_continue > + > +/* > + goto bootrom; > +*/ > + ldr r0, bootrom > + bx r0 > + > +shmobile_smp_continue: This should become ENTRY(shmobile_

Re: [RFC 01/37] ARM: shmobile: Add watchdog support

2018-01-26 Thread Geert Uytterhoeven
On Fri, Jan 26, 2018 at 10:46 AM, Geert Uytterhoeven wrote: > On Thu, Jan 25, 2018 at 7:02 PM, Fabrizio Castro > wrote: >> SMP bringup, CPU hot plug, and suspend to RAM work as normal. > > shmobile_boot_vector is not only used for R-Car Gen2 and RZ/G1 SoCs, but > also

<    1   2   3   4   5   6   7   8   9   10   >