* [PATCH v2] thermal: rcar: Add binding docs for new R-Car Gen2 SoCs @ 2014-08-28 8:12 Geert Uytterhoeven 2014-09-06 16:13 ` Eduardo Valentin 2014-09-09 14:51 ` Eduardo Valentin 0 siblings, 2 replies; 6+ messages in thread From: Geert Uytterhoeven @ 2014-08-28 8:12 UTC (permalink / raw) To: Zhang Rui, Eduardo Valentin Cc: devicetree, linux-pm, linux-sh, Geert Uytterhoeven - r8a7792 (R-Car V2H) - r8a7793 (R-Car M2-N) - r8a7794 (R-Car E2) r8a7791 is now called "R-Car M2-W". Signed-off-by: Geert Uytterhoeven <geert+renesas@glider.be> --- v2: Drop RFC --- Documentation/devicetree/bindings/thermal/rcar-thermal.txt | 5 ++++- 1 file changed, 4 insertions(+), 1 deletion(-) diff --git a/Documentation/devicetree/bindings/thermal/rcar-thermal.txt b/Documentation/devicetree/bindings/thermal/rcar-thermal.txt index 0ef00be44b01..43404b197933 100644 --- a/Documentation/devicetree/bindings/thermal/rcar-thermal.txt +++ b/Documentation/devicetree/bindings/thermal/rcar-thermal.txt @@ -7,7 +7,10 @@ Required properties: - "renesas,thermal-r8a73a4" (R-Mobile AP6) - "renesas,thermal-r8a7779" (R-Car H1) - "renesas,thermal-r8a7790" (R-Car H2) - - "renesas,thermal-r8a7791" (R-Car M2) + - "renesas,thermal-r8a7791" (R-Car M2-W) + - "renesas,thermal-r8a7792" (R-Car V2H) + - "renesas,thermal-r8a7793" (R-Car M2-N) + - "renesas,thermal-r8a7794" (R-Car E2) - reg : Address range of the thermal registers. The 1st reg will be recognized as common register if it has "interrupts". -- 1.9.1 ^ permalink raw reply related [flat|nested] 6+ messages in thread
* Re: [PATCH v2] thermal: rcar: Add binding docs for new R-Car Gen2 SoCs 2014-08-28 8:12 [PATCH v2] thermal: rcar: Add binding docs for new R-Car Gen2 SoCs Geert Uytterhoeven @ 2014-09-06 16:13 ` Eduardo Valentin 2014-09-06 18:10 ` Geert Uytterhoeven 2014-09-09 14:51 ` Eduardo Valentin 1 sibling, 1 reply; 6+ messages in thread From: Eduardo Valentin @ 2014-09-06 16:13 UTC (permalink / raw) To: Geert Uytterhoeven; +Cc: Zhang Rui, devicetree, linux-pm, linux-sh Hello Geert, On Thu, Aug 28, 2014 at 10:12:32AM +0200, Geert Uytterhoeven wrote: > - r8a7792 (R-Car V2H) > - r8a7793 (R-Car M2-N) > - r8a7794 (R-Car E2) > > r8a7791 is now called "R-Car M2-W". > > Signed-off-by: Geert Uytterhoeven <geert+renesas@glider.be> I don't have any concerns on this patch and will be adding to my next branch. But before that, I have one simple question. Although the binding description mention about soc types, the rcar thermal driver do not differentiate soc types, only supports the "renesas,rcar-thermal" compatible. Do you care explaining why we need the soc specific compatibles? > --- > v2: Drop RFC > --- > Documentation/devicetree/bindings/thermal/rcar-thermal.txt | 5 ++++- > 1 file changed, 4 insertions(+), 1 deletion(-) > > diff --git a/Documentation/devicetree/bindings/thermal/rcar-thermal.txt b/Documentation/devicetree/bindings/thermal/rcar-thermal.txt > index 0ef00be44b01..43404b197933 100644 > --- a/Documentation/devicetree/bindings/thermal/rcar-thermal.txt > +++ b/Documentation/devicetree/bindings/thermal/rcar-thermal.txt > @@ -7,7 +7,10 @@ Required properties: > - "renesas,thermal-r8a73a4" (R-Mobile AP6) > - "renesas,thermal-r8a7779" (R-Car H1) > - "renesas,thermal-r8a7790" (R-Car H2) > - - "renesas,thermal-r8a7791" (R-Car M2) > + - "renesas,thermal-r8a7791" (R-Car M2-W) > + - "renesas,thermal-r8a7792" (R-Car V2H) > + - "renesas,thermal-r8a7793" (R-Car M2-N) > + - "renesas,thermal-r8a7794" (R-Car E2) > - reg : Address range of the thermal registers. > The 1st reg will be recognized as common register > if it has "interrupts". > -- > 1.9.1 > ^ permalink raw reply [flat|nested] 6+ messages in thread
* Re: [PATCH v2] thermal: rcar: Add binding docs for new R-Car Gen2 SoCs 2014-09-06 16:13 ` Eduardo Valentin @ 2014-09-06 18:10 ` Geert Uytterhoeven 2014-09-08 10:48 ` Eduardo Valentin 0 siblings, 1 reply; 6+ messages in thread From: Geert Uytterhoeven @ 2014-09-06 18:10 UTC (permalink / raw) To: Eduardo Valentin Cc: Geert Uytterhoeven, Zhang Rui, devicetree@vger.kernel.org, Linux PM list, Linux-sh list Hi Eduardo, On Sat, Sep 6, 2014 at 6:13 PM, Eduardo Valentin <edubezval@gmail.com> wrote: > On Thu, Aug 28, 2014 at 10:12:32AM +0200, Geert Uytterhoeven wrote: >> - r8a7792 (R-Car V2H) >> - r8a7793 (R-Car M2-N) >> - r8a7794 (R-Car E2) >> >> r8a7791 is now called "R-Car M2-W". >> >> Signed-off-by: Geert Uytterhoeven <geert+renesas@glider.be> > > I don't have any concerns on this patch and will be adding to my next > branch. But before that, I have one simple question. Although the Thanks! > binding description mention about soc types, the rcar thermal driver do > not differentiate soc types, only supports the "renesas,rcar-thermal" > compatible. > > Do you care explaining why we need the soc specific compatibles? The reason we also have the SoC-specific values is that we are not 100% sure the thermal IP cores in the Renesas R-Car Gen1 and R-Car Gen2 series are really identical. Currently the driver treats all of them the same, but if we ever discover a difference, and the driver needs to start distinguishing among different versions, we will need to have the SoC-specific values in the DTSes to do so. I hope this explains our rationale. 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] 6+ messages in thread
* Re: [PATCH v2] thermal: rcar: Add binding docs for new R-Car Gen2 SoCs 2014-09-06 18:10 ` Geert Uytterhoeven @ 2014-09-08 10:48 ` Eduardo Valentin 2014-09-08 11:02 ` Geert Uytterhoeven 0 siblings, 1 reply; 6+ messages in thread From: Eduardo Valentin @ 2014-09-08 10:48 UTC (permalink / raw) To: Geert Uytterhoeven Cc: Geert Uytterhoeven, Zhang Rui, devicetree@vger.kernel.org, Linux PM list, Linux-sh list Hi Geert, On Sat, Sep 06, 2014 at 08:10:13PM +0200, Geert Uytterhoeven wrote: > Hi Eduardo, > > On Sat, Sep 6, 2014 at 6:13 PM, Eduardo Valentin <edubezval@gmail.com> wrote: > > On Thu, Aug 28, 2014 at 10:12:32AM +0200, Geert Uytterhoeven wrote: > >> - r8a7792 (R-Car V2H) > >> - r8a7793 (R-Car M2-N) > >> - r8a7794 (R-Car E2) > >> > >> r8a7791 is now called "R-Car M2-W". > >> > >> Signed-off-by: Geert Uytterhoeven <geert+renesas@glider.be> > > > > I don't have any concerns on this patch and will be adding to my next > > branch. But before that, I have one simple question. Although the > > Thanks! > > > binding description mention about soc types, the rcar thermal driver do > > not differentiate soc types, only supports the "renesas,rcar-thermal" > > compatible. > > > > Do you care explaining why we need the soc specific compatibles? > > The reason we also have the SoC-specific values is that we are not > 100% sure the thermal IP cores in the Renesas R-Car Gen1 and R-Car Gen2 > series are really identical. > > Currently the driver treats all of them the same, but if we ever discover > a difference, and the driver needs to start distinguishing among different > versions, we will need to have the SoC-specific values in the DTSes to > do so. In this case, wouldn't make sense to find out first the differences before changing DT documentation? Otherwise, you may also figure out that there is no difference and then send a patch to remove the changes you are sending now (or even removing the -<soc specific> parts). > > I hope this explains our rationale. > > 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] 6+ messages in thread
* Re: [PATCH v2] thermal: rcar: Add binding docs for new R-Car Gen2 SoCs 2014-09-08 10:48 ` Eduardo Valentin @ 2014-09-08 11:02 ` Geert Uytterhoeven 0 siblings, 0 replies; 6+ messages in thread From: Geert Uytterhoeven @ 2014-09-08 11:02 UTC (permalink / raw) To: Eduardo Valentin Cc: Geert Uytterhoeven, Zhang Rui, devicetree@vger.kernel.org, Linux PM list, Linux-sh list Hi Eduardo, On Mon, Sep 8, 2014 at 12:48 PM, Eduardo Valentin <edubezval@gmail.com> wrote: > On Sat, Sep 06, 2014 at 08:10:13PM +0200, Geert Uytterhoeven wrote: >> On Sat, Sep 6, 2014 at 6:13 PM, Eduardo Valentin <edubezval@gmail.com> wrote: >> > On Thu, Aug 28, 2014 at 10:12:32AM +0200, Geert Uytterhoeven wrote: >> >> - r8a7792 (R-Car V2H) >> >> - r8a7793 (R-Car M2-N) >> >> - r8a7794 (R-Car E2) >> >> >> >> r8a7791 is now called "R-Car M2-W". >> >> >> >> Signed-off-by: Geert Uytterhoeven <geert+renesas@glider.be> >> > >> > I don't have any concerns on this patch and will be adding to my next >> > branch. But before that, I have one simple question. Although the >> >> Thanks! >> >> > binding description mention about soc types, the rcar thermal driver do >> > not differentiate soc types, only supports the "renesas,rcar-thermal" >> > compatible. >> > >> > Do you care explaining why we need the soc specific compatibles? >> >> The reason we also have the SoC-specific values is that we are not >> 100% sure the thermal IP cores in the Renesas R-Car Gen1 and R-Car Gen2 >> series are really identical. >> >> Currently the driver treats all of them the same, but if we ever discover >> a difference, and the driver needs to start distinguishing among different >> versions, we will need to have the SoC-specific values in the DTSes to >> do so. > > In this case, wouldn't make sense to find out first the differences > before changing DT documentation? That may be too late, if a DTS is in active use. We cannot have new SoC-specific values in a DTS if they're not documented, as checkpatch would complain. > Otherwise, you may also figure out that there is no difference and > then send a patch to remove the changes you are sending now > (or even removing the -<soc specific> parts). Removing SoC-specific parts cannot be done, as some of they are already in use in existing DTSes. These are supposed to be backwards-compatible. We are doing the same for (most) other Renesas hardware blocks. Thanks! 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] 6+ messages in thread
* Re: [PATCH v2] thermal: rcar: Add binding docs for new R-Car Gen2 SoCs 2014-08-28 8:12 [PATCH v2] thermal: rcar: Add binding docs for new R-Car Gen2 SoCs Geert Uytterhoeven 2014-09-06 16:13 ` Eduardo Valentin @ 2014-09-09 14:51 ` Eduardo Valentin 1 sibling, 0 replies; 6+ messages in thread From: Eduardo Valentin @ 2014-09-09 14:51 UTC (permalink / raw) To: Geert Uytterhoeven; +Cc: Zhang Rui, devicetree, linux-pm, linux-sh On Thu, Aug 28, 2014 at 10:12:32AM +0200, Geert Uytterhoeven wrote: > - r8a7792 (R-Car V2H) > - r8a7793 (R-Car M2-N) > - r8a7794 (R-Car E2) > > r8a7791 is now called "R-Car M2-W". > > Signed-off-by: Geert Uytterhoeven <geert+renesas@glider.be> Pulled into my next branch. Thanks. > --- > v2: Drop RFC > --- > Documentation/devicetree/bindings/thermal/rcar-thermal.txt | 5 ++++- > 1 file changed, 4 insertions(+), 1 deletion(-) > > diff --git a/Documentation/devicetree/bindings/thermal/rcar-thermal.txt b/Documentation/devicetree/bindings/thermal/rcar-thermal.txt > index 0ef00be44b01..43404b197933 100644 > --- a/Documentation/devicetree/bindings/thermal/rcar-thermal.txt > +++ b/Documentation/devicetree/bindings/thermal/rcar-thermal.txt > @@ -7,7 +7,10 @@ Required properties: > - "renesas,thermal-r8a73a4" (R-Mobile AP6) > - "renesas,thermal-r8a7779" (R-Car H1) > - "renesas,thermal-r8a7790" (R-Car H2) > - - "renesas,thermal-r8a7791" (R-Car M2) > + - "renesas,thermal-r8a7791" (R-Car M2-W) > + - "renesas,thermal-r8a7792" (R-Car V2H) > + - "renesas,thermal-r8a7793" (R-Car M2-N) > + - "renesas,thermal-r8a7794" (R-Car E2) > - reg : Address range of the thermal registers. > The 1st reg will be recognized as common register > if it has "interrupts". > -- > 1.9.1 > ^ permalink raw reply [flat|nested] 6+ messages in thread
end of thread, other threads:[~2014-09-09 14:51 UTC | newest] Thread overview: 6+ messages (download: mbox.gz follow: Atom feed -- links below jump to the message on this page -- 2014-08-28 8:12 [PATCH v2] thermal: rcar: Add binding docs for new R-Car Gen2 SoCs Geert Uytterhoeven 2014-09-06 16:13 ` Eduardo Valentin 2014-09-06 18:10 ` Geert Uytterhoeven 2014-09-08 10:48 ` Eduardo Valentin 2014-09-08 11:02 ` Geert Uytterhoeven 2014-09-09 14:51 ` Eduardo Valentin
This is a public inbox, see mirroring instructions for how to clone and mirror all data and code used for this inbox; as well as URLs for NNTP newsgroup(s).