* [PATCH] ARM: shmobile: stout: enable R-Car Gen2 regulator quirk
@ 2018-02-13 23:28 ` Marek Vasut
0 siblings, 0 replies; 20+ messages in thread
From: Marek Vasut @ 2018-02-13 23:28 UTC (permalink / raw)
To: linux-renesas-soc
Cc: linux-arm-kernel, Marek Vasut, Geert Uytterhoeven,
Kuninori Morimoto, Simon Horman, Wolfram Sang
Regulator setup is suboptimal on H2 Stout too.
Signed-off-by: Marek Vasut <marek.vasut+renesas@gmail.com>
Cc: Geert Uytterhoeven <geert+renesas@glider.be>
Cc: Kuninori Morimoto <kuninori.morimoto.gx@renesas.com>
Cc: Simon Horman <horms+renesas@verge.net.au>
Cc: Wolfram Sang <wsa+renesas@sang-engineering.com>
---
arch/arm/mach-shmobile/regulator-quirk-rcar-gen2.c | 7 ++++---
1 file changed, 4 insertions(+), 3 deletions(-)
diff --git a/arch/arm/mach-shmobile/regulator-quirk-rcar-gen2.c b/arch/arm/mach-shmobile/regulator-quirk-rcar-gen2.c
index 44438f344dc8..b749450d361f 100644
--- a/arch/arm/mach-shmobile/regulator-quirk-rcar-gen2.c
+++ b/arch/arm/mach-shmobile/regulator-quirk-rcar-gen2.c
@@ -1,9 +1,9 @@
/*
* R-Car Generation 2 da9063/da9210 regulator quirk
*
- * The r8a7790/lager and r8a7791/koelsch development boards have da9063 and
- * da9210 regulators. Both regulators have their interrupt request lines tied
- * to the same interrupt pin (IRQ2) on the SoC.
+ * The r8a7790/lager,stout and r8a7791/koelsch development boards have da9063
+ * and da9210 regulators. Both regulators have their interrupt request lines
+ * tied to the same interrupt pin (IRQ2) on the SoC.
*
* After cold boot or da9063-induced restart, both the da9063 and da9210 seem
* to assert their interrupt request lines. Hence as soon as one driver
@@ -118,6 +118,7 @@ static int __init rcar_gen2_regulator_quirk(void)
if (!of_machine_is_compatible("renesas,koelsch") &&
!of_machine_is_compatible("renesas,lager") &&
+ !of_machine_is_compatible("renesas,stout") &&
!of_machine_is_compatible("renesas,gose"))
return -ENODEV;
--
2.15.1
^ permalink raw reply related [flat|nested] 20+ messages in thread
* [PATCH] ARM: shmobile: stout: enable R-Car Gen2 regulator quirk
@ 2018-02-13 23:28 ` Marek Vasut
0 siblings, 0 replies; 20+ messages in thread
From: Marek Vasut @ 2018-02-13 23:28 UTC (permalink / raw)
To: linux-arm-kernel
Regulator setup is suboptimal on H2 Stout too.
Signed-off-by: Marek Vasut <marek.vasut+renesas@gmail.com>
Cc: Geert Uytterhoeven <geert+renesas@glider.be>
Cc: Kuninori Morimoto <kuninori.morimoto.gx@renesas.com>
Cc: Simon Horman <horms+renesas@verge.net.au>
Cc: Wolfram Sang <wsa+renesas@sang-engineering.com>
---
arch/arm/mach-shmobile/regulator-quirk-rcar-gen2.c | 7 ++++---
1 file changed, 4 insertions(+), 3 deletions(-)
diff --git a/arch/arm/mach-shmobile/regulator-quirk-rcar-gen2.c b/arch/arm/mach-shmobile/regulator-quirk-rcar-gen2.c
index 44438f344dc8..b749450d361f 100644
--- a/arch/arm/mach-shmobile/regulator-quirk-rcar-gen2.c
+++ b/arch/arm/mach-shmobile/regulator-quirk-rcar-gen2.c
@@ -1,9 +1,9 @@
/*
* R-Car Generation 2 da9063/da9210 regulator quirk
*
- * The r8a7790/lager and r8a7791/koelsch development boards have da9063 and
- * da9210 regulators. Both regulators have their interrupt request lines tied
- * to the same interrupt pin (IRQ2) on the SoC.
+ * The r8a7790/lager,stout and r8a7791/koelsch development boards have da9063
+ * and da9210 regulators. Both regulators have their interrupt request lines
+ * tied to the same interrupt pin (IRQ2) on the SoC.
*
* After cold boot or da9063-induced restart, both the da9063 and da9210 seem
* to assert their interrupt request lines. Hence as soon as one driver
@@ -118,6 +118,7 @@ static int __init rcar_gen2_regulator_quirk(void)
if (!of_machine_is_compatible("renesas,koelsch") &&
!of_machine_is_compatible("renesas,lager") &&
+ !of_machine_is_compatible("renesas,stout") &&
!of_machine_is_compatible("renesas,gose"))
return -ENODEV;
--
2.15.1
^ permalink raw reply related [flat|nested] 20+ messages in thread
* Re: [PATCH] ARM: shmobile: stout: enable R-Car Gen2 regulator quirk
2018-02-13 23:28 ` Marek Vasut
@ 2018-02-14 5:58 ` Wolfram Sang
-1 siblings, 0 replies; 20+ messages in thread
From: Wolfram Sang @ 2018-02-14 5:58 UTC (permalink / raw)
To: Marek Vasut
Cc: linux-renesas-soc, linux-arm-kernel, Marek Vasut,
Geert Uytterhoeven, Kuninori Morimoto, Simon Horman,
Wolfram Sang
[-- Attachment #1: Type: text/plain, Size: 1126 bytes --]
> - * The r8a7790/lager and r8a7791/koelsch development boards have da9063 and
> - * da9210 regulators. Both regulators have their interrupt request lines tied
> - * to the same interrupt pin (IRQ2) on the SoC.
> + * The r8a7790/lager,stout and r8a7791/koelsch development boards have da9063
> + * and da9210 regulators. Both regulators have their interrupt request lines
> + * tied to the same interrupt pin (IRQ2) on the SoC.
I think listing the boards here doesn't scale well. Gose is already
missing. How about rephrasing the paragraph to something like "Some Gen2
development boards have..."?
> *
> * After cold boot or da9063-induced restart, both the da9063 and da9210 seem
> * to assert their interrupt request lines. Hence as soon as one driver
> @@ -118,6 +118,7 @@ static int __init rcar_gen2_regulator_quirk(void)
>
> if (!of_machine_is_compatible("renesas,koelsch") &&
> !of_machine_is_compatible("renesas,lager") &&
> + !of_machine_is_compatible("renesas,stout") &&
> !of_machine_is_compatible("renesas,gose"))
> return -ENODEV;
>
> --
> 2.15.1
>
[-- Attachment #2: signature.asc --]
[-- Type: application/pgp-signature, Size: 833 bytes --]
^ permalink raw reply [flat|nested] 20+ messages in thread
* [PATCH] ARM: shmobile: stout: enable R-Car Gen2 regulator quirk
@ 2018-02-14 5:58 ` Wolfram Sang
0 siblings, 0 replies; 20+ messages in thread
From: Wolfram Sang @ 2018-02-14 5:58 UTC (permalink / raw)
To: linux-arm-kernel
> - * The r8a7790/lager and r8a7791/koelsch development boards have da9063 and
> - * da9210 regulators. Both regulators have their interrupt request lines tied
> - * to the same interrupt pin (IRQ2) on the SoC.
> + * The r8a7790/lager,stout and r8a7791/koelsch development boards have da9063
> + * and da9210 regulators. Both regulators have their interrupt request lines
> + * tied to the same interrupt pin (IRQ2) on the SoC.
I think listing the boards here doesn't scale well. Gose is already
missing. How about rephrasing the paragraph to something like "Some Gen2
development boards have..."?
> *
> * After cold boot or da9063-induced restart, both the da9063 and da9210 seem
> * to assert their interrupt request lines. Hence as soon as one driver
> @@ -118,6 +118,7 @@ static int __init rcar_gen2_regulator_quirk(void)
>
> if (!of_machine_is_compatible("renesas,koelsch") &&
> !of_machine_is_compatible("renesas,lager") &&
> + !of_machine_is_compatible("renesas,stout") &&
> !of_machine_is_compatible("renesas,gose"))
> return -ENODEV;
>
> --
> 2.15.1
>
-------------- next part --------------
A non-text attachment was scrubbed...
Name: signature.asc
Type: application/pgp-signature
Size: 833 bytes
Desc: not available
URL: <http://lists.infradead.org/pipermail/linux-arm-kernel/attachments/20180214/5c6ad971/attachment.sig>
^ permalink raw reply [flat|nested] 20+ messages in thread
* Re: [PATCH] ARM: shmobile: stout: enable R-Car Gen2 regulator quirk
2018-02-14 5:58 ` Wolfram Sang
@ 2018-02-14 8:09 ` Geert Uytterhoeven
-1 siblings, 0 replies; 20+ messages in thread
From: Geert Uytterhoeven @ 2018-02-14 8:09 UTC (permalink / raw)
To: Wolfram Sang
Cc: Marek Vasut, Linux-Renesas, linux-arm-kernel, Marek Vasut,
Geert Uytterhoeven, Kuninori Morimoto, Simon Horman,
Wolfram Sang
On Wed, Feb 14, 2018 at 6:58 AM, Wolfram Sang <wsa@the-dreams.de> wrote:
>> - * The r8a7790/lager and r8a7791/koelsch development boards have da9063 and
>> - * da9210 regulators. Both regulators have their interrupt request lines tied
>> - * to the same interrupt pin (IRQ2) on the SoC.
>> + * The r8a7790/lager,stout and r8a7791/koelsch development boards have da9063
>> + * and da9210 regulators. Both regulators have their interrupt request lines
>> + * tied to the same interrupt pin (IRQ2) on the SoC.
>
> I think listing the boards here doesn't scale well. Gose is already
> missing. How about rephrasing the paragraph to something like "Some Gen2
> development boards have..."?
+1
>> *
>> * After cold boot or da9063-induced restart, both the da9063 and da9210 seem
>> * to assert their interrupt request lines. Hence as soon as one driver
>> @@ -118,6 +118,7 @@ static int __init rcar_gen2_regulator_quirk(void)
>>
>> if (!of_machine_is_compatible("renesas,koelsch") &&
>> !of_machine_is_compatible("renesas,lager") &&
>> + !of_machine_is_compatible("renesas,stout") &&
>> !of_machine_is_compatible("renesas,gose"))
>> return -ENODEV;
Have we reached critical mass to start using array-based matching with
of_device_compatible_match()?
Gr{oetje,eeting}s,
Geert
--
Geert Uytterhoeven -- There's lots of Linux beyond ia32 -- geert@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
^ permalink raw reply [flat|nested] 20+ messages in thread
* [PATCH] ARM: shmobile: stout: enable R-Car Gen2 regulator quirk
@ 2018-02-14 8:09 ` Geert Uytterhoeven
0 siblings, 0 replies; 20+ messages in thread
From: Geert Uytterhoeven @ 2018-02-14 8:09 UTC (permalink / raw)
To: linux-arm-kernel
On Wed, Feb 14, 2018 at 6:58 AM, Wolfram Sang <wsa@the-dreams.de> wrote:
>> - * The r8a7790/lager and r8a7791/koelsch development boards have da9063 and
>> - * da9210 regulators. Both regulators have their interrupt request lines tied
>> - * to the same interrupt pin (IRQ2) on the SoC.
>> + * The r8a7790/lager,stout and r8a7791/koelsch development boards have da9063
>> + * and da9210 regulators. Both regulators have their interrupt request lines
>> + * tied to the same interrupt pin (IRQ2) on the SoC.
>
> I think listing the boards here doesn't scale well. Gose is already
> missing. How about rephrasing the paragraph to something like "Some Gen2
> development boards have..."?
+1
>> *
>> * After cold boot or da9063-induced restart, both the da9063 and da9210 seem
>> * to assert their interrupt request lines. Hence as soon as one driver
>> @@ -118,6 +118,7 @@ static int __init rcar_gen2_regulator_quirk(void)
>>
>> if (!of_machine_is_compatible("renesas,koelsch") &&
>> !of_machine_is_compatible("renesas,lager") &&
>> + !of_machine_is_compatible("renesas,stout") &&
>> !of_machine_is_compatible("renesas,gose"))
>> return -ENODEV;
Have we reached critical mass to start using array-based matching with
of_device_compatible_match()?
Gr{oetje,eeting}s,
Geert
--
Geert Uytterhoeven -- There's lots of Linux beyond ia32 -- geert at 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
^ permalink raw reply [flat|nested] 20+ messages in thread
* Re: [PATCH] ARM: shmobile: stout: enable R-Car Gen2 regulator quirk
2018-02-13 23:28 ` Marek Vasut
@ 2018-02-14 9:14 ` Geert Uytterhoeven
-1 siblings, 0 replies; 20+ messages in thread
From: Geert Uytterhoeven @ 2018-02-14 9:14 UTC (permalink / raw)
To: Marek Vasut
Cc: Linux-Renesas, linux-arm-kernel, Marek Vasut, Geert Uytterhoeven,
Kuninori Morimoto, Simon Horman, Wolfram Sang
Hi Marek,
On Wed, Feb 14, 2018 at 12:28 AM, Marek Vasut <marek.vasut@gmail.com> wrote:
> Regulator setup is suboptimal on H2 Stout too.
Worse, Stout has 2 DA9210 regulators, so you have to add a check for a
DA9210 at address 0x70.
Gr{oetje,eeting}s,
Geert
--
Geert Uytterhoeven -- There's lots of Linux beyond ia32 -- geert@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
^ permalink raw reply [flat|nested] 20+ messages in thread
* [PATCH] ARM: shmobile: stout: enable R-Car Gen2 regulator quirk
@ 2018-02-14 9:14 ` Geert Uytterhoeven
0 siblings, 0 replies; 20+ messages in thread
From: Geert Uytterhoeven @ 2018-02-14 9:14 UTC (permalink / raw)
To: linux-arm-kernel
Hi Marek,
On Wed, Feb 14, 2018 at 12:28 AM, Marek Vasut <marek.vasut@gmail.com> wrote:
> Regulator setup is suboptimal on H2 Stout too.
Worse, Stout has 2 DA9210 regulators, so you have to add a check for a
DA9210 at address 0x70.
Gr{oetje,eeting}s,
Geert
--
Geert Uytterhoeven -- There's lots of Linux beyond ia32 -- geert at 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
^ permalink raw reply [flat|nested] 20+ messages in thread
* Re: [PATCH] ARM: shmobile: stout: enable R-Car Gen2 regulator quirk
2018-02-14 9:14 ` Geert Uytterhoeven
@ 2018-02-15 9:36 ` Marek Vasut
-1 siblings, 0 replies; 20+ messages in thread
From: Marek Vasut @ 2018-02-15 9:36 UTC (permalink / raw)
To: Geert Uytterhoeven
Cc: Linux-Renesas, linux-arm-kernel, Marek Vasut, Geert Uytterhoeven,
Kuninori Morimoto, Simon Horman, Wolfram Sang
On 02/14/2018 10:14 AM, Geert Uytterhoeven wrote:
> Hi Marek,
>
> On Wed, Feb 14, 2018 at 12:28 AM, Marek Vasut <marek.vasut@gmail.com> wrote:
>> Regulator setup is suboptimal on H2 Stout too.
>
> Worse, Stout has 2 DA9210 regulators, so you have to add a check for a
> DA9210 at address 0x70.
Fixed
--
Best regards,
Marek Vasut
^ permalink raw reply [flat|nested] 20+ messages in thread
* [PATCH] ARM: shmobile: stout: enable R-Car Gen2 regulator quirk
@ 2018-02-15 9:36 ` Marek Vasut
0 siblings, 0 replies; 20+ messages in thread
From: Marek Vasut @ 2018-02-15 9:36 UTC (permalink / raw)
To: linux-arm-kernel
On 02/14/2018 10:14 AM, Geert Uytterhoeven wrote:
> Hi Marek,
>
> On Wed, Feb 14, 2018 at 12:28 AM, Marek Vasut <marek.vasut@gmail.com> wrote:
>> Regulator setup is suboptimal on H2 Stout too.
>
> Worse, Stout has 2 DA9210 regulators, so you have to add a check for a
> DA9210 at address 0x70.
Fixed
--
Best regards,
Marek Vasut
^ permalink raw reply [flat|nested] 20+ messages in thread
* Re: [PATCH] ARM: shmobile: stout: enable R-Car Gen2 regulator quirk
2018-02-14 8:09 ` Geert Uytterhoeven
@ 2018-02-15 9:44 ` Marek Vasut
-1 siblings, 0 replies; 20+ messages in thread
From: Marek Vasut @ 2018-02-15 9:44 UTC (permalink / raw)
To: Geert Uytterhoeven, Wolfram Sang
Cc: Linux-Renesas, linux-arm-kernel, Marek Vasut, Geert Uytterhoeven,
Kuninori Morimoto, Simon Horman, Wolfram Sang
On 02/14/2018 09:09 AM, Geert Uytterhoeven wrote:
> On Wed, Feb 14, 2018 at 6:58 AM, Wolfram Sang <wsa@the-dreams.de> wrote:
>>> - * The r8a7790/lager and r8a7791/koelsch development boards have da9063 and
>>> - * da9210 regulators. Both regulators have their interrupt request lines tied
>>> - * to the same interrupt pin (IRQ2) on the SoC.
>>> + * The r8a7790/lager,stout and r8a7791/koelsch development boards have da9063
>>> + * and da9210 regulators. Both regulators have their interrupt request lines
>>> + * tied to the same interrupt pin (IRQ2) on the SoC.
>>
>> I think listing the boards here doesn't scale well. Gose is already
>> missing. How about rephrasing the paragraph to something like "Some Gen2
>> development boards have..."?
>
> +1
>
>>> *
>>> * After cold boot or da9063-induced restart, both the da9063 and da9210 seem
>>> * to assert their interrupt request lines. Hence as soon as one driver
>>> @@ -118,6 +118,7 @@ static int __init rcar_gen2_regulator_quirk(void)
>>>
>>> if (!of_machine_is_compatible("renesas,koelsch") &&
>>> !of_machine_is_compatible("renesas,lager") &&
>>> + !of_machine_is_compatible("renesas,stout") &&
>>> !of_machine_is_compatible("renesas,gose"))
>>> return -ENODEV;
>
> Have we reached critical mass to start using array-based matching with
> of_device_compatible_match()?
We're matching on machine , not device , here . I guess our device node
would be / ?
--
Best regards,
Marek Vasut
^ permalink raw reply [flat|nested] 20+ messages in thread
* [PATCH] ARM: shmobile: stout: enable R-Car Gen2 regulator quirk
@ 2018-02-15 9:44 ` Marek Vasut
0 siblings, 0 replies; 20+ messages in thread
From: Marek Vasut @ 2018-02-15 9:44 UTC (permalink / raw)
To: linux-arm-kernel
On 02/14/2018 09:09 AM, Geert Uytterhoeven wrote:
> On Wed, Feb 14, 2018 at 6:58 AM, Wolfram Sang <wsa@the-dreams.de> wrote:
>>> - * The r8a7790/lager and r8a7791/koelsch development boards have da9063 and
>>> - * da9210 regulators. Both regulators have their interrupt request lines tied
>>> - * to the same interrupt pin (IRQ2) on the SoC.
>>> + * The r8a7790/lager,stout and r8a7791/koelsch development boards have da9063
>>> + * and da9210 regulators. Both regulators have their interrupt request lines
>>> + * tied to the same interrupt pin (IRQ2) on the SoC.
>>
>> I think listing the boards here doesn't scale well. Gose is already
>> missing. How about rephrasing the paragraph to something like "Some Gen2
>> development boards have..."?
>
> +1
>
>>> *
>>> * After cold boot or da9063-induced restart, both the da9063 and da9210 seem
>>> * to assert their interrupt request lines. Hence as soon as one driver
>>> @@ -118,6 +118,7 @@ static int __init rcar_gen2_regulator_quirk(void)
>>>
>>> if (!of_machine_is_compatible("renesas,koelsch") &&
>>> !of_machine_is_compatible("renesas,lager") &&
>>> + !of_machine_is_compatible("renesas,stout") &&
>>> !of_machine_is_compatible("renesas,gose"))
>>> return -ENODEV;
>
> Have we reached critical mass to start using array-based matching with
> of_device_compatible_match()?
We're matching on machine , not device , here . I guess our device node
would be / ?
--
Best regards,
Marek Vasut
^ permalink raw reply [flat|nested] 20+ messages in thread
* Re: [PATCH] ARM: shmobile: stout: enable R-Car Gen2 regulator quirk
2018-02-15 9:44 ` Marek Vasut
@ 2018-02-15 10:08 ` Geert Uytterhoeven
-1 siblings, 0 replies; 20+ messages in thread
From: Geert Uytterhoeven @ 2018-02-15 10:08 UTC (permalink / raw)
To: Marek Vasut
Cc: Wolfram Sang, Linux-Renesas, linux-arm-kernel, Marek Vasut,
Geert Uytterhoeven, Kuninori Morimoto, Simon Horman,
Wolfram Sang
Hi Marek,
On Thu, Feb 15, 2018 at 10:44 AM, Marek Vasut <marek.vasut@gmail.com> wrote:
> On 02/14/2018 09:09 AM, Geert Uytterhoeven wrote:
>> On Wed, Feb 14, 2018 at 6:58 AM, Wolfram Sang <wsa@the-dreams.de> wrote:
>>>> - * The r8a7790/lager and r8a7791/koelsch development boards have da9063 and
>>>> - * da9210 regulators. Both regulators have their interrupt request lines tied
>>>> - * to the same interrupt pin (IRQ2) on the SoC.
>>>> + * The r8a7790/lager,stout and r8a7791/koelsch development boards have da9063
>>>> + * and da9210 regulators. Both regulators have their interrupt request lines
>>>> + * tied to the same interrupt pin (IRQ2) on the SoC.
>>>
>>> I think listing the boards here doesn't scale well. Gose is already
>>> missing. How about rephrasing the paragraph to something like "Some Gen2
>>> development boards have..."?
>>
>> +1
>>
>>>> *
>>>> * After cold boot or da9063-induced restart, both the da9063 and da9210 seem
>>>> * to assert their interrupt request lines. Hence as soon as one driver
>>>> @@ -118,6 +118,7 @@ static int __init rcar_gen2_regulator_quirk(void)
>>>>
>>>> if (!of_machine_is_compatible("renesas,koelsch") &&
>>>> !of_machine_is_compatible("renesas,lager") &&
>>>> + !of_machine_is_compatible("renesas,stout") &&
>>>> !of_machine_is_compatible("renesas,gose"))
>>>> return -ENODEV;
>>
>> Have we reached critical mass to start using array-based matching with
>> of_device_compatible_match()?
>
> We're matching on machine , not device , here . I guess our device node
> would be / ?
Correct, cfr. the implementation of of_machine_is_compatible().
BTW, several PPC platforms use of_device_compatible_match(of_root, ...),
but I believe of_root is not guaranteed to be set up.
Gr{oetje,eeting}s,
Geert
--
Geert Uytterhoeven -- There's lots of Linux beyond ia32 -- geert@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
^ permalink raw reply [flat|nested] 20+ messages in thread
* [PATCH] ARM: shmobile: stout: enable R-Car Gen2 regulator quirk
@ 2018-02-15 10:08 ` Geert Uytterhoeven
0 siblings, 0 replies; 20+ messages in thread
From: Geert Uytterhoeven @ 2018-02-15 10:08 UTC (permalink / raw)
To: linux-arm-kernel
Hi Marek,
On Thu, Feb 15, 2018 at 10:44 AM, Marek Vasut <marek.vasut@gmail.com> wrote:
> On 02/14/2018 09:09 AM, Geert Uytterhoeven wrote:
>> On Wed, Feb 14, 2018 at 6:58 AM, Wolfram Sang <wsa@the-dreams.de> wrote:
>>>> - * The r8a7790/lager and r8a7791/koelsch development boards have da9063 and
>>>> - * da9210 regulators. Both regulators have their interrupt request lines tied
>>>> - * to the same interrupt pin (IRQ2) on the SoC.
>>>> + * The r8a7790/lager,stout and r8a7791/koelsch development boards have da9063
>>>> + * and da9210 regulators. Both regulators have their interrupt request lines
>>>> + * tied to the same interrupt pin (IRQ2) on the SoC.
>>>
>>> I think listing the boards here doesn't scale well. Gose is already
>>> missing. How about rephrasing the paragraph to something like "Some Gen2
>>> development boards have..."?
>>
>> +1
>>
>>>> *
>>>> * After cold boot or da9063-induced restart, both the da9063 and da9210 seem
>>>> * to assert their interrupt request lines. Hence as soon as one driver
>>>> @@ -118,6 +118,7 @@ static int __init rcar_gen2_regulator_quirk(void)
>>>>
>>>> if (!of_machine_is_compatible("renesas,koelsch") &&
>>>> !of_machine_is_compatible("renesas,lager") &&
>>>> + !of_machine_is_compatible("renesas,stout") &&
>>>> !of_machine_is_compatible("renesas,gose"))
>>>> return -ENODEV;
>>
>> Have we reached critical mass to start using array-based matching with
>> of_device_compatible_match()?
>
> We're matching on machine , not device , here . I guess our device node
> would be / ?
Correct, cfr. the implementation of of_machine_is_compatible().
BTW, several PPC platforms use of_device_compatible_match(of_root, ...),
but I believe of_root is not guaranteed to be set up.
Gr{oetje,eeting}s,
Geert
--
Geert Uytterhoeven -- There's lots of Linux beyond ia32 -- geert at 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
^ permalink raw reply [flat|nested] 20+ messages in thread
* Re: [PATCH] ARM: shmobile: stout: enable R-Car Gen2 regulator quirk
2018-02-15 10:08 ` Geert Uytterhoeven
@ 2018-02-15 10:13 ` Marek Vasut
-1 siblings, 0 replies; 20+ messages in thread
From: Marek Vasut @ 2018-02-15 10:13 UTC (permalink / raw)
To: Geert Uytterhoeven
Cc: Wolfram Sang, Linux-Renesas, linux-arm-kernel, Marek Vasut,
Geert Uytterhoeven, Kuninori Morimoto, Simon Horman,
Wolfram Sang
On 02/15/2018 11:08 AM, Geert Uytterhoeven wrote:
> Hi Marek,
>
> On Thu, Feb 15, 2018 at 10:44 AM, Marek Vasut <marek.vasut@gmail.com> wrote:
>> On 02/14/2018 09:09 AM, Geert Uytterhoeven wrote:
>>> On Wed, Feb 14, 2018 at 6:58 AM, Wolfram Sang <wsa@the-dreams.de> wrote:
>>>>> - * The r8a7790/lager and r8a7791/koelsch development boards have da9063 and
>>>>> - * da9210 regulators. Both regulators have their interrupt request lines tied
>>>>> - * to the same interrupt pin (IRQ2) on the SoC.
>>>>> + * The r8a7790/lager,stout and r8a7791/koelsch development boards have da9063
>>>>> + * and da9210 regulators. Both regulators have their interrupt request lines
>>>>> + * tied to the same interrupt pin (IRQ2) on the SoC.
>>>>
>>>> I think listing the boards here doesn't scale well. Gose is already
>>>> missing. How about rephrasing the paragraph to something like "Some Gen2
>>>> development boards have..."?
>>>
>>> +1
>>>
>>>>> *
>>>>> * After cold boot or da9063-induced restart, both the da9063 and da9210 seem
>>>>> * to assert their interrupt request lines. Hence as soon as one driver
>>>>> @@ -118,6 +118,7 @@ static int __init rcar_gen2_regulator_quirk(void)
>>>>>
>>>>> if (!of_machine_is_compatible("renesas,koelsch") &&
>>>>> !of_machine_is_compatible("renesas,lager") &&
>>>>> + !of_machine_is_compatible("renesas,stout") &&
>>>>> !of_machine_is_compatible("renesas,gose"))
>>>>> return -ENODEV;
>>>
>>> Have we reached critical mass to start using array-based matching with
>>> of_device_compatible_match()?
>>
>> We're matching on machine , not device , here . I guess our device node
>> would be / ?
>
> Correct, cfr. the implementation of of_machine_is_compatible().
>
> BTW, several PPC platforms use of_device_compatible_match(of_root, ...),
> but I believe of_root is not guaranteed to be set up.
OK, subsequent patch then. Added to TODO.
--
Best regards,
Marek Vasut
^ permalink raw reply [flat|nested] 20+ messages in thread
* [PATCH] ARM: shmobile: stout: enable R-Car Gen2 regulator quirk
@ 2018-02-15 10:13 ` Marek Vasut
0 siblings, 0 replies; 20+ messages in thread
From: Marek Vasut @ 2018-02-15 10:13 UTC (permalink / raw)
To: linux-arm-kernel
On 02/15/2018 11:08 AM, Geert Uytterhoeven wrote:
> Hi Marek,
>
> On Thu, Feb 15, 2018 at 10:44 AM, Marek Vasut <marek.vasut@gmail.com> wrote:
>> On 02/14/2018 09:09 AM, Geert Uytterhoeven wrote:
>>> On Wed, Feb 14, 2018 at 6:58 AM, Wolfram Sang <wsa@the-dreams.de> wrote:
>>>>> - * The r8a7790/lager and r8a7791/koelsch development boards have da9063 and
>>>>> - * da9210 regulators. Both regulators have their interrupt request lines tied
>>>>> - * to the same interrupt pin (IRQ2) on the SoC.
>>>>> + * The r8a7790/lager,stout and r8a7791/koelsch development boards have da9063
>>>>> + * and da9210 regulators. Both regulators have their interrupt request lines
>>>>> + * tied to the same interrupt pin (IRQ2) on the SoC.
>>>>
>>>> I think listing the boards here doesn't scale well. Gose is already
>>>> missing. How about rephrasing the paragraph to something like "Some Gen2
>>>> development boards have..."?
>>>
>>> +1
>>>
>>>>> *
>>>>> * After cold boot or da9063-induced restart, both the da9063 and da9210 seem
>>>>> * to assert their interrupt request lines. Hence as soon as one driver
>>>>> @@ -118,6 +118,7 @@ static int __init rcar_gen2_regulator_quirk(void)
>>>>>
>>>>> if (!of_machine_is_compatible("renesas,koelsch") &&
>>>>> !of_machine_is_compatible("renesas,lager") &&
>>>>> + !of_machine_is_compatible("renesas,stout") &&
>>>>> !of_machine_is_compatible("renesas,gose"))
>>>>> return -ENODEV;
>>>
>>> Have we reached critical mass to start using array-based matching with
>>> of_device_compatible_match()?
>>
>> We're matching on machine , not device , here . I guess our device node
>> would be / ?
>
> Correct, cfr. the implementation of of_machine_is_compatible().
>
> BTW, several PPC platforms use of_device_compatible_match(of_root, ...),
> but I believe of_root is not guaranteed to be set up.
OK, subsequent patch then. Added to TODO.
--
Best regards,
Marek Vasut
^ permalink raw reply [flat|nested] 20+ messages in thread
* Re: [PATCH] ARM: shmobile: stout: enable R-Car Gen2 regulator quirk
2018-02-15 10:13 ` Marek Vasut
@ 2018-02-15 10:44 ` Wolfram Sang
-1 siblings, 0 replies; 20+ messages in thread
From: Wolfram Sang @ 2018-02-15 10:44 UTC (permalink / raw)
To: Marek Vasut
Cc: Geert Uytterhoeven, Linux-Renesas, linux-arm-kernel, Marek Vasut,
Geert Uytterhoeven, Kuninori Morimoto, Simon Horman,
Wolfram Sang
[-- Attachment #1: Type: text/plain, Size: 245 bytes --]
> > BTW, several PPC platforms use of_device_compatible_match(of_root, ...),
> > but I believe of_root is not guaranteed to be set up.
>
> OK, subsequent patch then. Added to TODO.
Can you place it below the PCIE patches, please? ;)
[-- Attachment #2: signature.asc --]
[-- Type: application/pgp-signature, Size: 833 bytes --]
^ permalink raw reply [flat|nested] 20+ messages in thread
* [PATCH] ARM: shmobile: stout: enable R-Car Gen2 regulator quirk
@ 2018-02-15 10:44 ` Wolfram Sang
0 siblings, 0 replies; 20+ messages in thread
From: Wolfram Sang @ 2018-02-15 10:44 UTC (permalink / raw)
To: linux-arm-kernel
> > BTW, several PPC platforms use of_device_compatible_match(of_root, ...),
> > but I believe of_root is not guaranteed to be set up.
>
> OK, subsequent patch then. Added to TODO.
Can you place it below the PCIE patches, please? ;)
-------------- next part --------------
A non-text attachment was scrubbed...
Name: signature.asc
Type: application/pgp-signature
Size: 833 bytes
Desc: not available
URL: <http://lists.infradead.org/pipermail/linux-arm-kernel/attachments/20180215/f8a717f5/attachment-0001.sig>
^ permalink raw reply [flat|nested] 20+ messages in thread
* Re: [PATCH] ARM: shmobile: stout: enable R-Car Gen2 regulator quirk
2018-02-15 10:44 ` Wolfram Sang
@ 2018-02-15 10:53 ` Marek Vasut
-1 siblings, 0 replies; 20+ messages in thread
From: Marek Vasut @ 2018-02-15 10:53 UTC (permalink / raw)
To: Wolfram Sang
Cc: Geert Uytterhoeven, Linux-Renesas, linux-arm-kernel, Marek Vasut,
Geert Uytterhoeven, Kuninori Morimoto, Simon Horman,
Wolfram Sang
On 02/15/2018 11:44 AM, Wolfram Sang wrote:
>
>>> BTW, several PPC platforms use of_device_compatible_match(of_root, ...),
>>> but I believe of_root is not guaranteed to be set up.
>>
>> OK, subsequent patch then. Added to TODO.
>
> Can you place it below the PCIE patches, please? ;)
Grumble, yeah.
--
Best regards,
Marek Vasut
^ permalink raw reply [flat|nested] 20+ messages in thread
* [PATCH] ARM: shmobile: stout: enable R-Car Gen2 regulator quirk
@ 2018-02-15 10:53 ` Marek Vasut
0 siblings, 0 replies; 20+ messages in thread
From: Marek Vasut @ 2018-02-15 10:53 UTC (permalink / raw)
To: linux-arm-kernel
On 02/15/2018 11:44 AM, Wolfram Sang wrote:
>
>>> BTW, several PPC platforms use of_device_compatible_match(of_root, ...),
>>> but I believe of_root is not guaranteed to be set up.
>>
>> OK, subsequent patch then. Added to TODO.
>
> Can you place it below the PCIE patches, please? ;)
Grumble, yeah.
--
Best regards,
Marek Vasut
^ permalink raw reply [flat|nested] 20+ messages in thread
end of thread, other threads:[~2018-02-15 13:08 UTC | newest]
Thread overview: 20+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2018-02-13 23:28 [PATCH] ARM: shmobile: stout: enable R-Car Gen2 regulator quirk Marek Vasut
2018-02-13 23:28 ` Marek Vasut
2018-02-14 5:58 ` Wolfram Sang
2018-02-14 5:58 ` Wolfram Sang
2018-02-14 8:09 ` Geert Uytterhoeven
2018-02-14 8:09 ` Geert Uytterhoeven
2018-02-15 9:44 ` Marek Vasut
2018-02-15 9:44 ` Marek Vasut
2018-02-15 10:08 ` Geert Uytterhoeven
2018-02-15 10:08 ` Geert Uytterhoeven
2018-02-15 10:13 ` Marek Vasut
2018-02-15 10:13 ` Marek Vasut
2018-02-15 10:44 ` Wolfram Sang
2018-02-15 10:44 ` Wolfram Sang
2018-02-15 10:53 ` Marek Vasut
2018-02-15 10:53 ` Marek Vasut
2018-02-14 9:14 ` Geert Uytterhoeven
2018-02-14 9:14 ` Geert Uytterhoeven
2018-02-15 9:36 ` Marek Vasut
2018-02-15 9:36 ` Marek Vasut
This is an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.