Re: [RFC] mmc: tmio: fix tuning for stubborn cards

2017-10-11 Thread Wolfram Sang
On Wed, Oct 11, 2017 at 09:36:48AM +0200, Simon Horman wrote: > On Wed, Oct 11, 2017 at 02:08:14AM +0200, Niklas Söderlund wrote: > > The commit 43b0b361b0170030 ("mmc: tmio: always get number of taps") > > changed the behavior of the tuning. Before the commit the SCC was only > > enabled for the

[PATCH v2] ARM: dts: iwg20d-q7: Enable HS-USB

2017-10-11 Thread Biju Das
* Dropped "status" property,to disable host mode support on USB otg port by default. This patch has been tested against renesas-dev tag 20171011-v4.14-rc4. arch/arm/boot/dts/iwg20d-q7-common.dtsi | 7 ++- 1 file changed, 6 insertions(+), 1 deletion(-) diff --git a/ar

Re: [PATCH v5 2/4] phy: rcar-gen3-usb2: use enum phy_mode in the role_store()

2017-10-11 Thread Kishon Vijay Abraham I
Hi, On Tuesday 10 October 2017 03:11 PM, Yoshihiro Shimoda wrote: > This patch modifies the role_store() to use "enum phy_mode" instead > of the local "bool" for host/device mode selection. looks good except for one minor comment.. > > Signed-off-by: Yoshihiro Shimoda

Re: [PATCH v2 2/2] pinctrl: sh-pfc: r8a7796: fix trivial typo in comment

2017-10-11 Thread Geert Uytterhoeven
On Mon, Oct 9, 2017 at 10:37 AM, Wolfram Sang wrote: > Signed-off-by: Wolfram Sang Reviewed-by: Geert Uytterhoeven i.e. will queue in sh-pfc-for-v4.15. Gr{oetje,eeting}s,

Re: [PATCH v3 1/2] pinctrl: rza1: add support for RZ/A1L

2017-10-11 Thread Geert Uytterhoeven
Hi Linus, On Sat, Oct 7, 2017 at 1:19 PM, Linus Walleij wrote: > On Wed, Oct 4, 2017 at 11:07 PM, Chris Brandt > wrote: >> Aspects like the number of ports and the location where peripherals are >> brought out differ between the RZ/A1H and

Re: [RFC] mmc: tmio: fix tuning for stubborn cards

2017-10-11 Thread Simon Horman
On Wed, Oct 11, 2017 at 09:36:48AM +0200, Simon Horman wrote: > On Wed, Oct 11, 2017 at 02:08:14AM +0200, Niklas Söderlund wrote: > > The commit 43b0b361b0170030 ("mmc: tmio: always get number of taps") > > changed the behavior of the tuning. Before the commit the SCC was only > > enabled for the

[GIT PULL v2] Second Round of Renesas ARM Based SoC Updates for v4.15

2017-10-11 Thread Simon Horman
Hi Olof, Hi Kevin, Hi Arnd, Please consider these second round of Renesas ARM based SoC updates for v4.15. This pull request is based on the previous round of such requests, tagged as renesas-soc-for-v4.15, which I have already sent a pull-request for. This pull request fixes a bug introduced

[PATCH 1/1] ARM: shmobile: rcar-gen2: fix non-SMP build

2017-10-11 Thread Simon Horman
From: Arnd Bergmann A bugfix for the SMP case broke the build for the UP case: arch/arm/mach-shmobile/headsmp-apmu.o: In function `shmobile_boot_apmu': (.text+0x34): undefined reference to `secondary_startup' The assembler file mixes code that is used for SMP with code that we

Re: [PATCH 2/2 v2] pinctrl: sh-pfc: r8a77995: add Audio SSI pin support

2017-10-11 Thread Geert Uytterhoeven
Hi Morimoto-san, On Tue, Oct 10, 2017 at 9:57 AM, Kuninori Morimoto wrote: > From: Kuninori Morimoto > > Signed-off-by: Kuninori Morimoto Reviewed-by: Geert Uytterhoeven

Re: [PATCH v2] pinctrl: sh-pfc: r8a7795: Add HSCIF pins, groups, and functions

2017-10-11 Thread Geert Uytterhoeven
Hi Wolfram, On Mon, Oct 9, 2017 at 10:20 AM, Wolfram Sang wrote: > Signed-off-by: Wolfram Sang > --- > > Change since V1: > * added third set of pins for HSCIF2 Thanks for the update! Reviewed-by: Geert Uytterhoeven

Re: [PATCH v2 04/18] arm64: dts: m3ulcb-kf: initial device tree

2017-10-11 Thread Simon Horman
On Tue, Oct 10, 2017 at 07:16:38PM +0300, Vladimir Barinov wrote: > Hi Simon, > > On 09.10.2017 10:18, Simon Horman wrote: > >On Fri, Oct 06, 2017 at 11:18:00AM +0200, Simon Horman wrote: > >>On Fri, Oct 06, 2017 at 05:47:25AM +0300, Vladimir Barinov wrote: > >>>Hi Simon, Geert, > >>> > >>>Do you

Re: [PATCH v2 1/2] pinctrl: sh-pfc: r8a7795: fix trivial typo in comment

2017-10-11 Thread Geert Uytterhoeven
Hi Wolfram, On Mon, Oct 9, 2017 at 10:37 AM, Wolfram Sang wrote: > Signed-off-by: Wolfram Sang > --- > > Change since V1 for both patches: > * corrected authorship email address (sorry for the noise!) > >

Re: [RFC] mmc: tmio: fix tuning for stubborn cards

2017-10-11 Thread Simon Horman
On Wed, Oct 11, 2017 at 02:08:14AM +0200, Niklas Söderlund wrote: > The commit 43b0b361b0170030 ("mmc: tmio: always get number of taps") > changed the behavior of the tuning. Before the commit the SCC was only > enabled for the first tuning attempt (host->init_tuning(host)), if that > failed the

Re: [PATCH] ARM: shmobile: rcar-gen2: fix non-SMP build

2017-10-11 Thread Simon Horman
On Tue, Oct 10, 2017 at 12:25:42PM +0300, Sergei Shtylyov wrote: > Hello! > > On 10/10/2017 11:12 AM, Simon Horman wrote: > > >From: Arnd Bergmann > > > >A bugfix for the SMP case broke the build for the UP case: > > > >arch/arm/mach-shmobile/headsmp-apmu.o: In function

Re: [RFC] mmc: tmio: fix tuning for stubborn cards

2017-10-11 Thread Niklas Söderlund
Hi Simon, On 2017-10-11 09:36:48 +0200, Simon Horman wrote: > On Wed, Oct 11, 2017 at 02:08:14AM +0200, Niklas Söderlund wrote: > > The commit 43b0b361b0170030 ("mmc: tmio: always get number of taps") > > changed the behavior of the tuning. Before the commit the SCC was only > > enabled for the

[PATCH] drm/shmob: Fix return value check in shmob_drm_probe()

2017-10-11 Thread Wei Yongjun
In case of error, the function devm_ioremap_resource() returns ERR_PTR() and never returns NULL. The NULL test in the return value check should be replaced with IS_ERR(). Signed-off-by: Wei Yongjun --- drivers/gpu/drm/shmobile/shmob_drm_drv.c | 4 ++-- 1 file changed, 2

Re: [PATCH 3/4] ARM: dts: r8a7743: Add USB-DMAC device nodes

2017-10-11 Thread Geert Uytterhoeven
On Mon, Oct 9, 2017 at 3:21 PM, Biju Das wrote: > Signed-off-by: Biju Das > Signed-off-by: Chris Paterson Reviewed-by: Geert Uytterhoeven Gr{oetje,eeting}s, Geert

Re: [PATCH 1/5] ARM: dts: r8a7745: Add internal PCI bridge nodes

2017-10-11 Thread Geert Uytterhoeven
On Mon, Oct 9, 2017 at 3:58 PM, Biju Das wrote: > Add device nodes for the r8a7745 internal PCI bridge devices. > > Signed-off-by: Biju Das > Signed-off-by: Chris Paterson Reviewed-by: Geert Uytterhoeven

Re: [PATCH 5/5] ARM: dts: iwg22d-sodimm: Enable USB PHY

2017-10-11 Thread Geert Uytterhoeven
On Mon, Oct 9, 2017 at 3:59 PM, Biju Das wrote: > Signed-off-by: Biju Das > Signed-off-by: Chris Paterson Reviewed-by: Geert Uytterhoeven Gr{oetje,eeting}s, Geert

Re: [PATCH v2] clocksource/drivers/ostm: Use unique device name instead of ostm

2017-10-11 Thread Geert Uytterhoeven
Hi Jacopo, On Wed, Oct 11, 2017 at 3:05 PM, Jacopo Mondi wrote: > From: Geert Uytterhoeven > > Currently all OSTM devices are called "ostm", also in kernel messages. > > As there can be multiple instances in an SoC, this can confuse the user.

Re: renesas sdhi driver and DMA

2017-10-11 Thread Wolfram Sang
> > Isn't it SYS-DMAC compatible (sigh)? > > And to answer your question, it's a completely different IP than the > SYS-DMAC. It's still a multi-channel linked-list descriptor based system > like SYS-DMAC, but a complete different set of registers. > > However, it's a nice piece of IP so

Re: [PATCH v5 2/4] phy: rcar-gen3-usb2: use enum phy_mode in the role_store()

2017-10-11 Thread Simon Horman
On Tue, Oct 10, 2017 at 06:41:28PM +0900, Yoshihiro Shimoda wrote: > This patch modifies the role_store() to use "enum phy_mode" instead > of the local "bool" for host/device mode selection. > > Signed-off-by: Yoshihiro Shimoda Thanks for the updates, this

Re: [PATCH 1/2 v2] pinctrl: sh-pfc: r8a77995: add Audio clock pin support

2017-10-11 Thread Geert Uytterhoeven
Hi Morimoto-san, On Tue, Oct 10, 2017 at 9:57 AM, Kuninori Morimoto wrote: > From: Kuninori Morimoto Thanks for the update! > Signed-off-by: Kuninori Morimoto Reviewed-by: Geert

Re: [PATCH 4/5] ARM: dts: iwg22d-sodimm: Enable internal PCI

2017-10-11 Thread Geert Uytterhoeven
On Mon, Oct 9, 2017 at 3:59 PM, Biju Das wrote: > Enable internal AHB-PCI bridges for the USB EHCI/OHCI controllers > attached to them. > > Signed-off-by: Biju Das > Signed-off-by: Chris Paterson Reviewed-by: Geert

[PATCH 0/2] drivers: firmware: psci: Extract psci_is_available() helper

2017-10-11 Thread Geert Uytterhoeven
Hi Mark, Lorenzo, PSCI support may be disabled at build time (by configuration) or at run-time (PSCI firmware not present). While CONFIG_ARM_PSCI_FW can be used to check for build time enablement, there is currently no simple way to check if PSCI is actually available and used. The

[PATCH 2/3] clk: renesas: r8a7796: Correct parent clock of INTC-AP

2017-10-11 Thread Geert Uytterhoeven
According to the R-Car Gen3 Hardware Manual Errata for Rev 0.55 of September 8, 2017, the parent clock of the INTC-AP module clock on R-Car M3-W is S0D3. This change has no functional impact. Signed-off-by: Geert Uytterhoeven --- drivers/clk/renesas/r8a7796-cpg-mssr.c

[PATCH 1/3] clk: renesas: r8a7795: Correct parent clock of INTC-AP

2017-10-11 Thread Geert Uytterhoeven
According to the R-Car Gen3 Hardware Manual Errata for Rev 0.55 of September 8, 2017, the parent clock of the INTC-AP module clock on R-Car H3 ES2.0 is S0D3. This change has no functional impact. Signed-off-by: Geert Uytterhoeven ---

[PATCH 3/3] clk: renesas: r8a77995: Correct parent clock of INTC-AP

2017-10-11 Thread Geert Uytterhoeven
According to the R-Car Gen3 Hardware Manual Errata for Rev 0.55 of September 8, 2017, the parent clock of the INTC-AP module clock on R-Car D3 is S1D2. This change has no functional impact. Signed-off-by: Geert Uytterhoeven --- drivers/clk/renesas/r8a77995-cpg-mssr.c |

[PATCH 0/3] clk: renesas: rcar-gen3: Correct parent clock of INTC-AP

2017-10-11 Thread Geert Uytterhoeven
Hi Mike, Stephen, This patch series corrects the parent clock of the INTC-AP module clock on R-Car H3 ES2.0, M3-W, and D3 SoCs, cfr. R-Car Gen3 Hardware Manual Errata for Rev 0.55 of September 8, 2017. These changes have no functional impact. As usual, I plan to queue these up in

[PATCH] pinctrl: sh-pfc: r8a77995: Remove USB0_IDIN and USB0_IDPU pins

2017-10-11 Thread Geert Uytterhoeven
R-Car Gen3 Hardware Manual Errata for Rev 0.55 of September 8, 2017 removed the USB0_IDIN and USB0_IDPU pins on R-Car D3. This change has no functional impact, as these definitions were unused. Signed-off-by: Geert Uytterhoeven --- To be queued up in sh-pfc-for-v4.15.

Re: [PATCH 1/2] ARM: dts: gr-peach: Add ETHER pin group

2017-10-11 Thread Geert Uytterhoeven
Hi Jacopo, On Mon, Oct 9, 2017 at 10:48 AM, Jacopo Mondi wrote: > Add pin configuration subnode for ETHER pin group. Thanks for the update! BTW, it would be good to have version numbers in the email subjects ("git format-patch -v2"). > Signed-off-by: Jacopo Mondi

[PATCH 2/2] drivers: firmware: psci: checker: Use psci_is_available()

2017-10-11 Thread Geert Uytterhoeven
Use the psci_is_available() helper instead of direct accesses to psci_ops internals. Signed-off-by: Geert Uytterhoeven --- drivers/firmware/psci_checker.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/drivers/firmware/psci_checker.c

mainline 4.14-rc4: renesas_sdhi_internal_dmac ee100000.sd: swiotlb buffer is full?

2017-10-11 Thread Dirk Behme
Hi, trying to boot recent mainline v4.14-rc4 on a custom H3 ES2.0 board with rootfs on SD card I'm getting [1]. Last time I think I used v4.13 in the same environment and I think it worked fine, most probably because renesas_sdhi_internal_dmac wasn't there, yet ;) I checked

Re: mainline 4.14-rc4: renesas_sdhi_internal_dmac ee100000.sd: swiotlb buffer is full?

2017-10-11 Thread Dirk Behme
Hi Geert, On 11.10.2017 14:42, Geert Uytterhoeven wrote: Hi Dirk, On Wed, Oct 11, 2017 at 2:23 PM, Dirk Behme wrote: trying to boot recent mainline v4.14-rc4 on a custom H3 ES2.0 board with rootfs on SD card I'm getting [1]. Last time I think I used v4.13 in the

Re: mainline 4.14-rc4: renesas_sdhi_internal_dmac ee100000.sd: swiotlb buffer is full?

2017-10-11 Thread Geert Uytterhoeven
Hi Dirk, On Wed, Oct 11, 2017 at 2:57 PM, Dirk Behme wrote: > On 11.10.2017 14:42, Geert Uytterhoeven wrote: >> On Wed, Oct 11, 2017 at 2:23 PM, Dirk Behme >> wrote: >>> trying to boot recent mainline v4.14-rc4 on a custom H3 ES2.0 board with

Re: [PATCH 2/2] ARM: dts: gr-peach: Enable ostm0 and ostm1 timers

2017-10-11 Thread Geert Uytterhoeven
On Mon, Oct 9, 2017 at 10:48 AM, Jacopo Mondi wrote: > Enable ostm0 and ostm1 timers to be used as clock source and clockevent > source. The timers provides greater accuracy than the already enabled > mtu2 one. > > With these enabled: > > clocksource: ostm: mask:

Re: [PATCH v2] ARM: dts: iwg20d-q7: Enable HS-USB

2017-10-11 Thread Geert Uytterhoeven
On Wed, Oct 11, 2017 at 11:04 AM, Biju Das wrote: > Enable HS-USB device for the iWave G20D-Q7 carrier board based on > RZ/G1M. > Also disable the host mode support on usb otg port by default to avoid > pin conflicts. > > Signed-off-by: Biju Das

Re: [PATCH 2/5] ARM: dts: r8a7745: Add USB PHY DT support

2017-10-11 Thread Geert Uytterhoeven
On Mon, Oct 9, 2017 at 3:58 PM, Biju Das wrote: > Define the r8a7745 generic part of the USB PHY device node. > > Signed-off-by: Biju Das > Signed-off-by: Chris Paterson Reviewed-by: Geert Uytterhoeven

RE: [PATCH v2] clocksource/drivers/ostm: Use unique device name instead of ostm

2017-10-11 Thread Chris Brandt
On Wednesday, October 11, 2017, Jacopo Mondi wrote: > From: Geert Uytterhoeven > > Currently all OSTM devices are called "ostm", also in kernel messages. > > As there can be multiple instances in an SoC, this can confuse the user. > Hence construct a unique name from

Re: [PATCH/RFC] arm64: dts: renesas: salvator-common: Add BD9571 PMIC

2017-10-11 Thread Marek Vasut
On 10/10/2017 04:58 PM, Geert Uytterhoeven wrote: > Add a device node for the ROHM BD9571MWV PMIC, based on the example in > the DT binding documentation, but using INTC-EX instead. > > Signed-off-by: Geert Uytterhoeven > --- > Do we need to describe more regulators? To

Re: [PATCH 04/24] media: v4l2-mediabus: convert flags to enums and document them

2017-10-11 Thread Pavel Machek
On Mon 2017-10-09 07:19:10, Mauro Carvalho Chehab wrote: > There is a mess with media bus flags: there are two sets of > flags, one used by parallel and ITU-R BT.656 outputs, > and another one for CSI2. > > Depending on the type, the same bit has different meanings. > > @@ -86,11 +125,22 @@

Re: [PATCH v2 2/3] mmc: parse new binding for eMMC fixed driver type

2017-10-11 Thread Simon Horman
On Wed, Oct 04, 2017 at 07:39:19PM +0200, Wolfram Sang wrote: > Parse the new binding and store it in the host struct after doing some > sanity checks. The code is designed to support fixed SD driver type if > we ever need that. > > Signed-off-by: Wolfram Sang

RE: [PATCH v2] i2c: riic: remove fixed clock restriction

2017-10-11 Thread Chris Brandt
On Friday, September 29, 2017, Chris Brandt wrote: > Most systems with this i2c are going to have a clock of either 33.3MHz or > 32MHz. That 4% difference is not reason enough to warrant that the driver > to completely fail. > > Signed-off-by: Chris Brandt > --- > v2: >

[PATCH 1/2] drivers: firmware: psci: Add psci_is_available()

2017-10-11 Thread Geert Uytterhoeven
PSCI support may be disabled at build time (by configuration) or at run-time (PSCI firmware not present). While CONFIG_ARM_PSCI_FW can be used to check for build time enablement, there is currently no simple way to check if PSCI is actually available and used. Hence add a helper function to

Re: [PATCH v2 1/2] pinctrl: sh-pfc: r8a7795: fix trivial typo in comment

2017-10-11 Thread Wolfram Sang
> > - * R8A7795 has 8 banks with 32 PGIOS in each => 256 GPIOs. > > + * R8A7795 has 8 banks with 32 GPIOs IN EACH => 256 GPIOs. > > I don't think the capitalization of "in each" had to be changed? > (overzealous use of vim ~?) Huh, how did this slip through? Sorry, Geert, and thanks for

[PATCH/RFC] clocksource/drivers/ostm: Use unique device name instead of ostm

2017-10-11 Thread Geert Uytterhoeven
Currently all OSTM devices are called "ostm", also in kernel messages. As there can be multiple instances in an SoC, this can confuse the user. Hence construct a unique name from the DT node name, like is done for platform devices. On GR-Peach, the boot log changes like: -clocksource: ostm:

Re: mainline 4.14-rc4: renesas_sdhi_internal_dmac ee100000.sd: swiotlb buffer is full?

2017-10-11 Thread Geert Uytterhoeven
Hi Dirk, On Wed, Oct 11, 2017 at 2:23 PM, Dirk Behme wrote: > trying to boot recent mainline v4.14-rc4 on a custom H3 ES2.0 board with > rootfs on SD card I'm getting [1]. > > Last time I think I used v4.13 in the same environment and I think it worked > fine, most

Re: [PATCH 1/4] ARM: dts: r8a7743: Add HS-USB device node

2017-10-11 Thread Geert Uytterhoeven
On Mon, Oct 9, 2017 at 3:21 PM, Biju Das wrote: > Define the R8A7743 generic part of the HS-USB device node. It is up to the > board file to enable the device. > > Signed-off-by: Biju Das > Signed-off-by: Chris Paterson

Re: [PATCH 1/2] drivers: firmware: psci: Add psci_is_available()

2017-10-11 Thread Jisheng Zhang
On Wed, 11 Oct 2017 10:03:01 +0200 Geert Uytterhoeven wrote: > PSCI support may be disabled at build time (by configuration) or at > run-time (PSCI firmware not present). While CONFIG_ARM_PSCI_FW can be > used to check for build time enablement, there is currently no simple > way to check if

Re: [PATCH v2] extcon: Split out extcon header file for consumer and provider device

2017-10-11 Thread Chanwoo Choi
Dear Kishon, Could you please review this patch? After that, I'll make the immutable brand and then send the pull request for power_supply, mfd, phy, usb and extcon framework. On 2017년 10월 10일 19:17, Chanwoo Choi wrote: > The extcon has two type of extcon devices as following. > - 'extcon

[PATCH] dt-bindings: PCI: rcar: Correct example to match reality

2017-10-11 Thread Geert Uytterhoeven
Correct the USB subnodes in the example, cfr. commit f7d569c1e6a6fa73 ("ARM: dts: r8a779x: Fix PCI bus dtc warnings"): 1. Drop the bogus 'device_type = "pci"' properties, 2. Correct the unit addresses. Update other bits in the example to match real use: 1. Rename the USB subnodes from "pci"

Re: [PATCH] dt-bindings: PCI: rcar: Correct example to match reality

2017-10-11 Thread Sergei Shtylyov
On 10/11/2017 04:50 PM, Geert Uytterhoeven wrote: Correct the USB subnodes in the example, cfr. commit f7d569c1e6a6fa73 ("ARM: dts: r8a779x: Fix PCI bus dtc warnings"): 1. Drop the bogus 'device_type = "pci"' properties, 2. Correct the unit addresses. Update other bits in the example to