Re: [PATCH] clk: renesas: r8a7795: Move MODEMR to DT

2016-04-27 Thread Dirk Behme
Hi Geert, On 28.04.2016 08:52, Geert Uytterhoeven wrote: Hi Dirk, On Thu, Apr 28, 2016 at 8:08 AM, Dirk Behme wrote: From: Oleksij Rempel Map the Mode Monitor Register via the device tree instead of hard coding it in the code. This makes the mapping known to the device tree. Signed-off-by:

Re: [PATCH] clk: renesas: r8a7795: Move MODEMR to DT

2016-04-27 Thread Geert Uytterhoeven
Hi Dirk, On Thu, Apr 28, 2016 at 8:08 AM, Dirk Behme wrote: > From: Oleksij Rempel > > Map the Mode Monitor Register via the device tree instead of > hard coding it in the code. This makes the mapping known to the > device tree. > > Signed-off-by: Pooya Keshavarzi > Signed-off-by: Oleksij Rempe

Re: [PATCH v2 01/13] dt-bindings: Add Renesas R-Car FCP DT bindings

2016-04-27 Thread Laurent Pinchart
Hi Rob, On Wednesday 27 Apr 2016 21:54:29 Rob Herring wrote: > On Tue, Apr 26, 2016 at 12:36:26AM +0300, Laurent Pinchart wrote: > > The FCP is a companion module of video processing modules in the Renesas > > R-Car Gen3 SoCs. It provides data compression and decompression, data > > caching, and c

Re: [PATCH v2] can: rcar_canfd: Add Renesas R-Car CAN FD driver

2016-04-27 Thread Oliver Hartkopp
Hello Ramesh, please send out a new v3 patchset to trigger the process again :-) Best regards, Oliver On 04/13/2016 08:25 AM, Ramesh Shanmugasundaram wrote: HI Marc, Gentle reminder! Are you happy with the open comment's disposition? I can send a next version of patch if we have a closure on

Re: [PATCH 2/6] mmc: sh_mobile_sdhi: remove obsolete irq_by_name registration

2016-04-27 Thread Wolfram Sang
> I don't have any of the affected boards to test, but as long as > they're not actually using the include file you're removing, I don't > see any harm. It's my intent to remove all of these board files full > of platform devices as soon as we get working device tree bindings for > everything, any

[PATCH] mmc: sh_mobile_sdhi: remove obsolete include file

2016-04-27 Thread Wolfram Sang
From: Wolfram Sang A few SH boards include the file but don't make use of it (no named interrupts). The SDHI code removed support for this feature as well. So, drop the references and ultimately remove the unneeded file. Signed-off-by: Wolfram Sang Acked-by: Rich Felker Acked-by: Yoshinori Sat

[PATCH] clk: renesas: r8a7795: Move MODEMR to DT

2016-04-27 Thread Dirk Behme
From: Oleksij Rempel Map the Mode Monitor Register via the device tree instead of hard coding it in the code. This makes the mapping known to the device tree. Signed-off-by: Pooya Keshavarzi Signed-off-by: Oleksij Rempel Signed-off-by: Oleksij Rempel Signed-off-by: Dirk Behme --- .../device

Re: [PATCH] arm64: dts: r8a7795: Increase the size of GIC-400 mapped registers

2016-04-27 Thread Dirk Behme
Hi Simon, On 28.04.2016 01:30, Simon Horman wrote: Hi Dirk, I understand that there is an issue here but I'm not yet able to convince myself that this is the correct solution. In revision r0p1 of the CoreLink GIC-400 Generic Interrupt Controller Technical Reference Manual[1] I see in Section 3

Re: [PATCH 2/6] mmc: sh_mobile_sdhi: remove obsolete irq_by_name registration

2016-04-27 Thread Yoshinori Sato
On Thu, 28 Apr 2016 07:54:57 +0900, Rich Felker wrote: > > On Tue, Apr 19, 2016 at 11:37:13AM +0200, Wolfram Sang wrote: > > On Mon, Apr 18, 2016 at 10:21:41AM +0200, Ulf Hansson wrote: > > > On 6 April 2016 at 11:25, Wolfram Sang wrote: > > > > From: Wolfram Sang > > > > > > > > There is no use

Re: [PATCH v2 01/13] dt-bindings: Add Renesas R-Car FCP DT bindings

2016-04-27 Thread Rob Herring
On Tue, Apr 26, 2016 at 12:36:26AM +0300, Laurent Pinchart wrote: > The FCP is a companion module of video processing modules in the Renesas > R-Car Gen3 SoCs. It provides data compression and decompression, data > caching, and conversion of AXI transactions in order to reduce the > memory bandwidt

Re: [PATCH] arm64: dts: r8a7795: Increase the size of GIC-400 mapped registers

2016-04-27 Thread Simon Horman
Hi Dirk, I understand that there is an issue here but I'm not yet able to convince myself that this is the correct solution. In revision r0p1 of the CoreLink GIC-400 Generic Interrupt Controller Technical Reference Manual[1] I see in Section 3.2. "GIC-400 register map" that the size of both the C

[PATCH 1/5] ARM: dts: r8a7778: Don't disable referenced optional clocks

2016-04-27 Thread Simon Horman
From: Geert Uytterhoeven clk_get() on a disabled clock node will return -EPROBE_DEFER, which can cause drivers to be deferred forever if such clocks are referenced in their devices' clocks properties. Update the various disabled external clock nodes to default to a frequency of 0, but don't disa

[PATCH 5/5] ARM: dts: r8a7794: Don't disable referenced optional clocks

2016-04-27 Thread Simon Horman
From: Geert Uytterhoeven clk_get() on a disabled clock node will return -EPROBE_DEFER, which can cause drivers to be deferred forever if such clocks are referenced in their devices' clocks properties. Update the various disabled external clock nodes to default to a frequency of 0, but don't disa

[PATCH 3/5] ARM: dts: r8a7790: Don't disable referenced optional clocks

2016-04-27 Thread Simon Horman
From: Geert Uytterhoeven clk_get() on a disabled clock node will return -EPROBE_DEFER, which can cause drivers to be deferred forever if such clocks are referenced in their devices' clocks properties. Update the various disabled external clock nodes to default to a frequency of 0, but don't disa

[GIT PULL] Second Round of Renesas ARM Based SoC DT Updates for v4.7

2016-04-27 Thread Simon Horman
Hi Olof, Hi Kevin, Hi Arnd, Please consider these second round of Renesas ARM based SoC DT updates for v4.7. This pull request is based on the previous round of such requests, tagged as renesas-dt-for-v4.7, which you have already pulled. The following changes since commit 6f92cb2f454c26d9bdada9

[PATCH 4/5] ARM: dts: r8a7793: Don't disable referenced optional clocks

2016-04-27 Thread Simon Horman
From: Geert Uytterhoeven clk_get() on a disabled clock node will return -EPROBE_DEFER, which can cause drivers to be deferred forever if such clocks are referenced in their devices' clocks properties. Update the various disabled external clock nodes to default to a frequency of 0, but don't disa

[PATCH 2/5] ARM: dts: r8a7779: Don't disable referenced optional clocks

2016-04-27 Thread Simon Horman
From: Geert Uytterhoeven clk_get() on a disabled clock node will return -EPROBE_DEFER, which can cause drivers to be deferred forever if such clocks are referenced in their devices' clocks properties. Update the various disabled external clock nodes to default to a frequency of 0, but don't disa

[PATCH] arm64: dts: r8a7795: Don't disable referenced optional scif clock

2016-04-27 Thread Simon Horman
From: Geert Uytterhoeven clk_get() on a disabled clock node will return -EPROBE_DEFER, which can cause drivers to be deferred forever if such clocks are referenced in their devices' clocks properties. Update the disabled external scif clock node so that it is not disabled to prevent this. Repor

[GIT PULL] Second Round of Renesas ARM Based SoC Fixes for v4.6

2016-04-27 Thread Simon Horman
Hi Olof, Hi Kevin, Hi Arnd, Please consider these second round of Renesas ARM based SoC fixes for v4.6. This fix allows serial to work on r8a7795-based boards where the optional scif clock is not populated. This pull request is based on the previous round of such requests, tagged as renesas-fixe

Re: [GIT PULL] Second Round of Renesas ARM Based SoC R-Car SYSC Updates for v4.7

2016-04-27 Thread Simon Horman
On Tue, Apr 26, 2016 at 12:44:48PM +0200, Arnd Bergmann wrote: > On Tuesday 26 April 2016 11:28:38 Simon Horman wrote: > > This pull request is based on a merge of: > > > > * Thee previous round of such requests, tagged as > > renesas-rcar-sysc-for-v4.7, > > which you have previously pulled. >

Re: [PATCH 2/6] mmc: sh_mobile_sdhi: remove obsolete irq_by_name registration

2016-04-27 Thread Rich Felker
On Tue, Apr 19, 2016 at 11:37:13AM +0200, Wolfram Sang wrote: > On Mon, Apr 18, 2016 at 10:21:41AM +0200, Ulf Hansson wrote: > > On 6 April 2016 at 11:25, Wolfram Sang wrote: > > > From: Wolfram Sang > > > > > > There is no user left in the kernel, so this code can be removed. > > > (Legacy, non-

Re: [ANNOUNCE] Renesas tree closed for v4.7

2016-04-27 Thread Simon Horman
On Wed, Apr 27, 2016 at 01:51:40PM +0300, Sergei Shtylyov wrote: > Hello. > > On 4/27/2016 3:54 AM, Simon Horman wrote: > > >>I would like to stop accepting non-bug-fix patches for v4.6 and get > >v4.7? > > >>the last pull requests posted by the end of this week. This is in order > >>for t

[PATCH v3 1/5] mmc: tmio: give read32/write32 functions more descriptive names

2016-04-27 Thread Wolfram Sang
From: Wolfram Sang Looking at the backlogs, I am not the only one who missed that the above functions do not read u32 from one register, but create a virtual u32 from reading to adjacent u16 registers (which depending on 'bus_shift' can be up to 8 byte apart). Because this driver supports old har

[PATCH v3 0/5] mmc: tmio: make CTL_STATUS handling consistent

2016-04-27 Thread Wolfram Sang
It took me a little to discover that CTL_STATUS has a different handling than the other registers. CTL_STATUS and CTL_STATUS2, both u16, are merged into a virtual u32. However, CTL_STATUS2 was also directly accessed. Clean this up, make this consistent, and document this. Making the driver less co

[PATCH v3 4/5] mmc: tmio/sdhi: distinguish between SCLKDIVEN and ILL_FUNC

2016-04-27 Thread Wolfram Sang
From: Wolfram Sang This bit has a different meaning in SDHI and original TMIO. Document that and use the proper naming. Reviewed-by: Simon Horman Signed-off-by: Wolfram Sang --- drivers/mmc/host/sh_mobile_sdhi.c | 3 ++- drivers/mmc/host/tmio_mmc.h | 3 ++- drivers/mmc/host/tmio_mmc_pio

[PATCH v3 2/5] mmc: tmio: use BIT() within defines

2016-04-27 Thread Wolfram Sang
From: Wolfram Sang BIT() makes it easier to match the bits to the datasheet. This is especially important here, since some variants have different names in their datasheets (like with Renesas R-Car). Reviewed-by: Simon Horman Signed-off-by: Wolfram Sang --- drivers/mmc/host/tmio_mmc.h | 44 ++

[PATCH v3 3/5] mmc: tmio: use CTL_STATUS consistently

2016-04-27 Thread Wolfram Sang
From: Wolfram Sang To prevent confusion, use the virtual u32 CTL_STATUS in card_busy() the same way as in other parts of this driver. Reviewed-by: Simon Horman Signed-off-by: Wolfram Sang --- drivers/mmc/host/tmio_mmc.h | 3 +-- drivers/mmc/host/tmio_mmc_pio.c | 2 +- 2 files changed, 2 i

[PATCH v3 5/5] mmc: tmio: document CTL_STATUS handling

2016-04-27 Thread Wolfram Sang
From: Wolfram Sang Now that reading CTL_STATUS is consistent, we can remove CTL_STATUS2 and document how this is handled internally. Reviewed-by: Simon Horman Signed-off-by: Wolfram Sang --- drivers/mmc/host/tmio_mmc.h | 3 ++- 1 file changed, 2 insertions(+), 1 deletion(-) diff --git a/driv

Re: [PATCH v2 08/13] v4l: vsp1: Make vsp1_entity_get_pad_compose() more generic

2016-04-27 Thread Laurent Pinchart
Hi Sergei, On Tuesday 26 Apr 2016 21:09:10 Sergei Shtylyov wrote: > On 04/26/2016 12:36 AM, Laurent Pinchart wrote: > > Turn the helper into a function that can retrieve crop and compose > > selection rectangles. > > > > Signed-off-by: Laurent Pinchart > > > > --- > > > > drivers/media/platfo

[PATCH] sh-sci: SysRq support on EARLYCON

2016-04-27 Thread Yoshinori Sato
Add missing flag condition. Signed-off-by: Yoshinori Sato --- drivers/tty/serial/sh-sci.c | 4 +++- 1 file changed, 3 insertions(+), 1 deletion(-) diff --git a/drivers/tty/serial/sh-sci.c b/drivers/tty/serial/sh-sci.c index 0130feb..5a73620 100644 --- a/drivers/tty/serial/sh-sci.c +++ b/drivers

Re: [PATCH 2/4] clk: renesas: mstp: Use always-on governor for Clock Domain

2016-04-27 Thread Ulf Hansson
On 26 April 2016 at 09:09, Geert Uytterhoeven wrote: > As a pure Clock Domain does not have the concept of powering the domain > itself, the CPG/MSTP driver does not provide power_off() and power_on() > callbacks. > However, the genpd core may still perform a dummy power down, causing > /sys/kerne

Re: [PATCH] pinctrl: ish-pfc: avoid unused variable warning

2016-04-27 Thread Sergei Shtylyov
Hello. s/ish-/sh-/ in the subject. On 4/27/2016 12:56 PM, Arnd Bergmann wrote: After the conversion to devm_pinctrl_register, we get a warning in sh_pfc_remove when CONFIG_PINCTRL_SH_PFC_GPIO is disabled: drivers/pinctrl/sh-pfc/core.c: In function 'sh_pfc_remove': drivers/pinctrl/sh-pfc/co

Re: [PATCH 0/4] clk: renesas: Clock Domain fixes

2016-04-27 Thread Ulf Hansson
On 26 April 2016 at 09:09, Geert Uytterhoeven wrote: > Hi Mike, Stephen, > > This patch series contains fixes for the Renesas CPG/MSTP and CPG/MSSR > Clock Domains: > 1. Postpone calls to pm_genpd_init(), as all local setup of the > generic_pm_domain structure should have been compl

Re: [PATCH 2/4] clk: renesas: mstp: Use always-on governor for Clock Domain

2016-04-27 Thread Ulf Hansson
On 27 April 2016 at 16:04, Geert Uytterhoeven wrote: > Hi Ulf, > > On Wed, Apr 27, 2016 at 3:59 PM, Ulf Hansson wrote: >> On 26 April 2016 at 09:09, Geert Uytterhoeven >> wrote: >>> As a pure Clock Domain does not have the concept of powering the domain >>> itself, the CPG/MSTP driver does not

Re: [PATCH 2/4] clk: renesas: mstp: Use always-on governor for Clock Domain

2016-04-27 Thread Geert Uytterhoeven
Hi Ulf, On Wed, Apr 27, 2016 at 3:59 PM, Ulf Hansson wrote: > On 26 April 2016 at 09:09, Geert Uytterhoeven wrote: >> As a pure Clock Domain does not have the concept of powering the domain >> itself, the CPG/MSTP driver does not provide power_off() and power_on() >> callbacks. >> However, the g

Re: [ANNOUNCE] Renesas tree closed for v4.7

2016-04-27 Thread Sergei Shtylyov
Hello. On 4/27/2016 3:54 AM, Simon Horman wrote: I would like to stop accepting non-bug-fix patches for v4.6 and get v4.7? the last pull requests posted by the end of this week. This is in order for them to be sent before the release of v4.5-rc6, the deadline set by the The version me

Re: [PATCH] pinctrl: ish-pfc: avoid unused variable warning

2016-04-27 Thread Laxman Dewangan
On Wednesday 27 April 2016 03:26 PM, Arnd Bergmann wrote: After the conversion to devm_pinctrl_register, we get a warning in sh_pfc_remove when CONFIG_PINCTRL_SH_PFC_GPIO is disabled: drivers/pinctrl/sh-pfc/core.c: In function 'sh_pfc_remove': drivers/pinctrl/sh-pfc/core.c:603:17: unused variab

[PATCH] pinctrl: ish-pfc: avoid unused variable warning

2016-04-27 Thread Arnd Bergmann
After the conversion to devm_pinctrl_register, we get a warning in sh_pfc_remove when CONFIG_PINCTRL_SH_PFC_GPIO is disabled: drivers/pinctrl/sh-pfc/core.c: In function 'sh_pfc_remove': drivers/pinctrl/sh-pfc/core.c:603:17: unused variable 'pfc' [-Werror=unused-variable] struct sh_pfc *pfc = pl

Re: [PATCH v2 0/5] mmc: tmio: make CTL_STATUS handling consistent

2016-04-27 Thread Ulf Hansson
On 26 April 2016 at 18:53, Wolfram Sang wrote: > It took me a little to discover that CTL_STATUS has a different handling than > the other registers. CTL_STATUS and CTL_STATUS2, both u16, are merged into a > virtual u32. However, CTL_STATUS2 was also directly accessed. > > Clean this up, make this

Re: [PATCH] mmc: mmcif: remove obsolete support for sh7372

2016-04-27 Thread Ulf Hansson
On 26 April 2016 at 22:34, Wolfram Sang wrote: > From: Wolfram Sang > > There is no support for this platform in the kernel anymore. Make the > Kconfig text more generic, so it won't get stale anymore. > > Reported-by: Geert Uytterhoeven > Signed-off-by: Wolfram Sang Thanks, applied for next!

Re: [PATCH] mmc: sdhi: remove obsolete support for sh7372

2016-04-27 Thread Ulf Hansson
On 26 April 2016 at 22:29, Wolfram Sang wrote: > From: Wolfram Sang > > There is no support for this platform in the kernel anymore. > > Reported-by: Geert Uytterhoeven > Signed-off-by: Wolfram Sang Thanks, applied for next! Kind regards Uffe > --- > drivers/mmc/host/sh_mobile_sdhi.c | 1 -

Re: [PATCH V2 3/6] mmc: tmio: remove now unneeded seperate irq handlers

2016-04-27 Thread Ulf Hansson
On 26 April 2016 at 20:03, Sergei Shtylyov wrote: > Hello. > > On 04/26/2016 06:55 PM, Wolfram Sang wrote: > >> From: Wolfram Sang >> >> We removed installation of seperate handlers previously, so we can also > > >Separate. > >> remove the seperate handlers. > > >Again. :-) > >> Signed-of

Re: [PATCH] mmc: mmcif: remove obsolete support for sh7372

2016-04-27 Thread Geert Uytterhoeven
On Tue, Apr 26, 2016 at 10:34 PM, Wolfram Sang wrote: > From: Wolfram Sang > > There is no support for this platform in the kernel anymore. Make the > Kconfig text more generic, so it won't get stale anymore. > > Reported-by: Geert Uytterhoeven > Signed-off-by: Wolfram Sang Acked-by: Geert Uyt

Re: [PATCH] mmc: sdhi: remove obsolete support for sh7372

2016-04-27 Thread Geert Uytterhoeven
On Tue, Apr 26, 2016 at 10:29 PM, Wolfram Sang wrote: > From: Wolfram Sang > > There is no support for this platform in the kernel anymore. > > Reported-by: Geert Uytterhoeven > Signed-off-by: Wolfram Sang Acked-by: Geert Uytterhoeven Gr{oetje,eeting}s, Geert -- Gee