[GIT PULL] Renesas ARM Based SoC Updates for v5.1

[GIT PULL] Renesas ARM Based SoC Updates for v5.1

[Simon Horman]

Hi Olof, Hi Kevin, Hi Arnd,

Please consider these Renesas ARM based SoC updates for v5.1.


The following changes since commit bfeffd155283772bbe78c6a05dec7c0128ee500c:

  Linux 5.0-rc1 (2019-01-06 17:08:20 -0800)

are available in the git repository at:

  https://git.kernel.org/pub/scm/linux/kernel/git/horms/renesas.git tags/renesas-arm-soc-for-v5.1

for you to fetch changes up to 5347a0203709d5039a74d7c94e23519eee478094:

  ARM: shmobile: Fix R-Car Gen2 regulator quirk (2019-01-07 13:24:07 +0100)

----------------------------------------------------------------
Renesas ARM Based SoC Updates for v5.1

* Correct shared IRQ handling of R-Car Gen2 Regulator quirk
* Add missing dts files to MAINTAINERS

----------------------------------------------------------------
Marek Vasut (1):
      ARM: shmobile: Fix R-Car Gen2 regulator quirk

Simon Horman (1):
      ARM: shmobile: Add missing dts files to MAINTAINERS

 MAINTAINERS                                        | 2 ++
 arch/arm/mach-shmobile/regulator-quirk-rcar-gen2.c | 5 +++++
 2 files changed, 7 insertions(+)

_______________________________________________
linux-arm-kernel mailing list
linux-arm-kernel@lists.infradead.org
http://lists.infradead.org/mailman/listinfo/linux-arm-kernel

[PATCH 1/2] ARM: shmobile: Add missing dts files to MAINTAINERS

[Simon Horman]

Add some more patterns to the ARM/SHMOBILE ARM ARCHITECTURE
section of MAINTAINERS to complete the list of maintained dts files.

As of writing dts files covered are:
* arch/arm/boot/dts/gr-peach-audiocamerashield.dtsi
* arch/arm/boot/dts/iwg20d-q7-common.dtsi
* arch/arm/boot/dts/iwg20d-q7-dbcm-ca.dtsi

Signed-off-by: Simon Horman <horms+renesas@verge.net.au>
Reviewed-by: Geert Uytterhoeven <geert+renesas@glider.be>
---
 MAINTAINERS | 2 ++
 1 file changed, 2 insertions(+)

diff --git a/MAINTAINERS b/MAINTAINERS
index 32d444476a90..f4c805c1a34b 100644
--- a/MAINTAINERS
+++ b/MAINTAINERS
@@ -2109,6 +2109,8 @@ Q:	http://patchwork.kernel.org/project/linux-renesas-soc/list/
 T:	git git://git.kernel.org/pub/scm/linux/kernel/git/horms/renesas.git next
 S:	Supported
 F:	arch/arm/boot/dts/emev2*
+F:	arch/arm/boot/dts/gr-peach*
+F:	arch/arm/boot/dts/iwg20d-q7*
 F:	arch/arm/boot/dts/r7s*
 F:	arch/arm/boot/dts/r8a*
 F:	arch/arm/boot/dts/r9a*
-- 
2.11.0


_______________________________________________
linux-arm-kernel mailing list
linux-arm-kernel@lists.infradead.org
http://lists.infradead.org/mailman/listinfo/linux-arm-kernel

[PATCH 2/2] ARM: shmobile: Fix R-Car Gen2 regulator quirk

[Simon Horman]

From: Marek Vasut <marek.vasut@gmail.com>

The quirk code currently detects all compatible I2C chips with a shared
IRQ line on all I2C busses, adds them into a list, and registers a bus
notifier. For every chip for which the bus notifier triggers, the quirk
code performs I2C transfer on that I2C bus for all addresses in the list.
The problem is that this may generate transfers to non-existing chips on
systems with multiple I2C busses.

This patch adds a check to verify that the I2C bus to which the chip with
shared IRQ is attached to matches the I2C bus of the chip which triggered
the bus notifier and only starts the I2C transfer if they match.

Signed-off-by: Marek Vasut <marek.vasut+renesas@gmail.com>
Tested-by: Nguyen Viet Dung <dung.nguyen.aj@renesas.com>
Signed-off-by: Simon Horman <horms+renesas@verge.net.au>
---
 arch/arm/mach-shmobile/regulator-quirk-rcar-gen2.c | 5 +++++
 1 file changed, 5 insertions(+)

diff --git a/arch/arm/mach-shmobile/regulator-quirk-rcar-gen2.c b/arch/arm/mach-shmobile/regulator-quirk-rcar-gen2.c
index 8e50daa99151..dc526ef2e9b3 100644
--- a/arch/arm/mach-shmobile/regulator-quirk-rcar-gen2.c
+++ b/arch/arm/mach-shmobile/regulator-quirk-rcar-gen2.c
@@ -40,6 +40,7 @@
 struct regulator_quirk {
 	struct list_head		list;
 	const struct of_device_id	*id;
+	struct device_node		*np;
 	struct of_phandle_args		irq_args;
 	struct i2c_msg			i2c_msg;
 	bool				shared;	/* IRQ line is shared */
@@ -101,6 +102,9 @@ static int regulator_quirk_notify(struct notifier_block *nb,
 		if (!pos->shared)
 			continue;
 
+		if (pos->np->parent != client->dev.parent->of_node)
+			continue;
+
 		dev_info(&client->dev, "clearing %s@0x%02x interrupts\n",
 			 pos->id->compatible, pos->i2c_msg.addr);
 
@@ -165,6 +169,7 @@ static int __init rcar_gen2_regulator_quirk(void)
 		memcpy(&quirk->i2c_msg, id->data, sizeof(quirk->i2c_msg));
 
 		quirk->id = id;
+		quirk->np = np;
 		quirk->i2c_msg.addr = addr;
 
 		ret = of_irq_parse_one(np, 0, argsa);
-- 
2.11.0


_______________________________________________
linux-arm-kernel mailing list
linux-arm-kernel@lists.infradead.org
http://lists.infradead.org/mailman/listinfo/linux-arm-kernel

Re: [GIT PULL] Renesas ARM Based SoC Updates for v5.1

[Arnd Bergmann]

On Fri, Feb 8, 2019 at 12:14 PM Simon Horman <horms+renesas@verge.net.au> wrote:
>
> Hi Olof, Hi Kevin, Hi Arnd,
>
> Please consider these Renesas ARM based SoC updates for v5.1.
>
>
> The following changes since commit bfeffd155283772bbe78c6a05dec7c0128ee500c:
>
>   Linux 5.0-rc1 (2019-01-06 17:08:20 -0800)
>
> are available in the git repository at:
>
>   https://git.kernel.org/pub/scm/linux/kernel/git/horms/renesas.git tags/renesas-arm-soc-for-v5.1
>
> for you to fetch changes up to 5347a0203709d5039a74d7c94e23519eee478094:
>
>   ARM: shmobile: Fix R-Car Gen2 regulator quirk (2019-01-07 13:24:07 +0100)
>
> ----------------------------------------------------------------
> Renesas ARM Based SoC Updates for v5.1
>
> * Correct shared IRQ handling of R-Car Gen2 Regulator quirk
> * Add missing dts files to MAINTAINERS

Should the bugfix be marked Cc:stable and and Fixes:... ?

Pulled into arm/soc, thanks!

    Arnd

_______________________________________________
linux-arm-kernel mailing list
linux-arm-kernel@lists.infradead.org
http://lists.infradead.org/mailman/listinfo/linux-arm-kernel

Re: [GIT PULL] Renesas ARM Based SoC Updates for v5.1

[Simon Horman]

On Fri, Feb 15, 2019 at 05:49:32PM +0100, Arnd Bergmann wrote:
> On Fri, Feb 8, 2019 at 12:14 PM Simon Horman <horms+renesas@verge.net.au> wrote:
> >
> > Hi Olof, Hi Kevin, Hi Arnd,
> >
> > Please consider these Renesas ARM based SoC updates for v5.1.
> >
> >
> > The following changes since commit bfeffd155283772bbe78c6a05dec7c0128ee500c:
> >
> >   Linux 5.0-rc1 (2019-01-06 17:08:20 -0800)
> >
> > are available in the git repository at:
> >
> >   https://git.kernel.org/pub/scm/linux/kernel/git/horms/renesas.git tags/renesas-arm-soc-for-v5.1
> >
> > for you to fetch changes up to 5347a0203709d5039a74d7c94e23519eee478094:
> >
> >   ARM: shmobile: Fix R-Car Gen2 regulator quirk (2019-01-07 13:24:07 +0100)
> >
> > ----------------------------------------------------------------
> > Renesas ARM Based SoC Updates for v5.1
> >
> > * Correct shared IRQ handling of R-Car Gen2 Regulator quirk
> > * Add missing dts files to MAINTAINERS
> 
> Should the bugfix be marked Cc:stable and and Fixes:... ?
> 
> Pulled into arm/soc, thanks!

Thanks,

Sorry for missing that, I think that the bugfix should have had the following.
Let me know if/how you want to handle updating the patch's changelog.

Fixes: 6d14d4d313d09dab ("ARM: shmobile: Rework the PMIC IRQ line quirk")

_______________________________________________
linux-arm-kernel mailing list
linux-arm-kernel@lists.infradead.org
http://lists.infradead.org/mailman/listinfo/linux-arm-kernel

Re: [GIT PULL] Renesas ARM Based SoC Updates for v5.1

[Arnd Bergmann]

On Mon, Feb 18, 2019 at 12:27 PM Simon Horman <horms@verge.net.au> wrote:
> On Fri, Feb 15, 2019 at 05:49:32PM +0100, Arnd Bergmann wrote:
> > On Fri, Feb 8, 2019 at 12:14 PM Simon Horman <horms+renesas@verge.net.au> wrote:
> > > Renesas ARM Based SoC Updates for v5.1
> > >
> > > * Correct shared IRQ handling of R-Car Gen2 Regulator quirk
> > > * Add missing dts files to MAINTAINERS
> >
> > Should the bugfix be marked Cc:stable and and Fixes:... ?
> >
> Sorry for missing that, I think that the bugfix should have had the following.
> Let me know if/how you want to handle updating the patch's changelog.
>
> Fixes: 6d14d4d313d09dab ("ARM: shmobile: Rework the PMIC IRQ line quirk")

I think the easiest way is to send an email to stable@kernel.org asking for
a backport of commit 5347a0203709 ("ARM: shmobile: Fix R-Car Gen2 regulator
quirk") after that has hit the mainline kernel and asking for a backport.

I've added it to Cc here, but I don't know if that's sufficient.

If you'd rather have me pick up the patch for 5.0 as well, I can also do
that and add the Cc/Fixes lines there. We will then have a duplicate
commit in 5.1, but it should not cause any merge issues.

      Arnd

_______________________________________________
linux-arm-kernel mailing list
linux-arm-kernel@lists.infradead.org
http://lists.infradead.org/mailman/listinfo/linux-arm-kernel

Re: [GIT PULL] Renesas ARM Based SoC Updates for v5.1

[Simon Horman]

On Mon, Feb 18, 2019 at 02:12:07PM +0100, Arnd Bergmann wrote:
> On Mon, Feb 18, 2019 at 12:27 PM Simon Horman <horms@verge.net.au> wrote:
> > On Fri, Feb 15, 2019 at 05:49:32PM +0100, Arnd Bergmann wrote:
> > > On Fri, Feb 8, 2019 at 12:14 PM Simon Horman <horms+renesas@verge.net.au> wrote:
> > > > Renesas ARM Based SoC Updates for v5.1
> > > >
> > > > * Correct shared IRQ handling of R-Car Gen2 Regulator quirk
> > > > * Add missing dts files to MAINTAINERS
> > >
> > > Should the bugfix be marked Cc:stable and and Fixes:... ?
> > >
> > Sorry for missing that, I think that the bugfix should have had the following.
> > Let me know if/how you want to handle updating the patch's changelog.
> >
> > Fixes: 6d14d4d313d09dab ("ARM: shmobile: Rework the PMIC IRQ line quirk")
> 
> I think the easiest way is to send an email to stable@kernel.org asking for
> a backport of commit 5347a0203709 ("ARM: shmobile: Fix R-Car Gen2 regulator
> quirk") after that has hit the mainline kernel and asking for a backport.
> 
> I've added it to Cc here, but I don't know if that's sufficient.
> 
> If you'd rather have me pick up the patch for 5.0 as well, I can also do
> that and add the Cc/Fixes lines there. We will then have a duplicate
> commit in 5.1, but it should not cause any merge issues.

Thanks Arnd,

I'm happy to handle this via stable and avoid the duplicate.
If I don't hear back from them in a few days I'll make a formal
submission there.

_______________________________________________
linux-arm-kernel mailing list
linux-arm-kernel@lists.infradead.org
http://lists.infradead.org/mailman/listinfo/linux-arm-kernel

Re: [GIT PULL] Renesas ARM Based SoC Updates for v5.1

[Arnd Bergmann]

On Mon, Feb 18, 2019 at 3:15 PM Simon Horman <horms@verge.net.au> wrote:
>
> On Mon, Feb 18, 2019 at 02:12:07PM +0100, Arnd Bergmann wrote:
> > On Mon, Feb 18, 2019 at 12:27 PM Simon Horman <horms@verge.net.au> wrote:
> > > On Fri, Feb 15, 2019 at 05:49:32PM +0100, Arnd Bergmann wrote:
> > > > On Fri, Feb 8, 2019 at 12:14 PM Simon Horman <horms+renesas@verge.net.au> wrote:
> > > > > Renesas ARM Based SoC Updates for v5.1
> > > > >
> > > > > * Correct shared IRQ handling of R-Car Gen2 Regulator quirk
> > > > > * Add missing dts files to MAINTAINERS
> > > >
> > > > Should the bugfix be marked Cc:stable and and Fixes:... ?
> > > >
> > > Sorry for missing that, I think that the bugfix should have had the following.
> > > Let me know if/how you want to handle updating the patch's changelog.
> > >
> > > Fixes: 6d14d4d313d09dab ("ARM: shmobile: Rework the PMIC IRQ line quirk")
> >
> > I think the easiest way is to send an email to stable@kernel.org asking for
> > a backport of commit 5347a0203709 ("ARM: shmobile: Fix R-Car Gen2 regulator
> > quirk") after that has hit the mainline kernel and asking for a backport.
> >
> > I've added it to Cc here, but I don't know if that's sufficient.
> >
> > If you'd rather have me pick up the patch for 5.0 as well, I can also do
> > that and add the Cc/Fixes lines there. We will then have a duplicate
> > commit in 5.1, but it should not cause any merge issues.
>
> Thanks Arnd,
>
> I'm happy to handle this via stable and avoid the duplicate.
> If I don't hear back from them in a few days I'll make a formal
> submission there.

Ok, sounds good.

To clarify: I think you should only write them /after/ the commit
is in mainline then, i.e. near the end of the 5.1 merge window.

      Arnd

_______________________________________________
linux-arm-kernel mailing list
linux-arm-kernel@lists.infradead.org
http://lists.infradead.org/mailman/listinfo/linux-arm-kernel

Re: [GIT PULL] Renesas ARM Based SoC Updates for v5.1

[Simon Horman]

On Mon, Feb 18, 2019 at 03:21:14PM +0100, Arnd Bergmann wrote:
> On Mon, Feb 18, 2019 at 3:15 PM Simon Horman <horms@verge.net.au> wrote:
> >
> > On Mon, Feb 18, 2019 at 02:12:07PM +0100, Arnd Bergmann wrote:
> > > On Mon, Feb 18, 2019 at 12:27 PM Simon Horman <horms@verge.net.au> wrote:
> > > > On Fri, Feb 15, 2019 at 05:49:32PM +0100, Arnd Bergmann wrote:
> > > > > On Fri, Feb 8, 2019 at 12:14 PM Simon Horman <horms+renesas@verge.net.au> wrote:
> > > > > > Renesas ARM Based SoC Updates for v5.1
> > > > > >
> > > > > > * Correct shared IRQ handling of R-Car Gen2 Regulator quirk
> > > > > > * Add missing dts files to MAINTAINERS
> > > > >
> > > > > Should the bugfix be marked Cc:stable and and Fixes:... ?
> > > > >
> > > > Sorry for missing that, I think that the bugfix should have had the following.
> > > > Let me know if/how you want to handle updating the patch's changelog.
> > > >
> > > > Fixes: 6d14d4d313d09dab ("ARM: shmobile: Rework the PMIC IRQ line quirk")
> > >
> > > I think the easiest way is to send an email to stable@kernel.org asking for
> > > a backport of commit 5347a0203709 ("ARM: shmobile: Fix R-Car Gen2 regulator
> > > quirk") after that has hit the mainline kernel and asking for a backport.
> > >
> > > I've added it to Cc here, but I don't know if that's sufficient.
> > >
> > > If you'd rather have me pick up the patch for 5.0 as well, I can also do
> > > that and add the Cc/Fixes lines there. We will then have a duplicate
> > > commit in 5.1, but it should not cause any merge issues.
> >
> > Thanks Arnd,
> >
> > I'm happy to handle this via stable and avoid the duplicate.
> > If I don't hear back from them in a few days I'll make a formal
> > submission there.
> 
> Ok, sounds good.
> 
> To clarify: I think you should only write them /after/ the commit
> is in mainline then, i.e. near the end of the 5.1 merge window.

Thanks Arnd, got it.

_______________________________________________
linux-arm-kernel mailing list
linux-arm-kernel@lists.infradead.org
http://lists.infradead.org/mailman/listinfo/linux-arm-kernel