* [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-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
* 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
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.