diff mbox series

[01/03] dt-bindings: arm: consistently name SH-Mobile AG5 as R8A73A00

Message ID 153717357067.480.7556586771925219502.sendpatchset@octo (mailing list archive)
State Deferred
Delegated to: Simon Horman
Headers show
Series Further product name updates | expand

Commit Message

Magnus Damm Sept. 17, 2018, 8:39 a.m. UTC
From: Magnus Damm <damm@opensource.se>

Update the description of the AG5 entry to follow same style as other SoCs.

The actual DT binding remains unchanged.

Signed-off-by: Magnus Damm <damm@opensource.se>
---

 Developed on top of renesas-devel-20180914-v4.19-rc3

 Documentation/devicetree/bindings/arm/shmobile.txt |    2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

Comments

Geert Uytterhoeven Sept. 17, 2018, 11:20 a.m. UTC | #1
Hi Magnus,

On Mon, Sep 17, 2018 at 10:39 AM Magnus Damm <magnus.damm@gmail.com> wrote:
> From: Magnus Damm <damm@opensource.se>
>
> Update the description of the AG5 entry to follow same style as other SoCs.
>
> The actual DT binding remains unchanged.
>
> Signed-off-by: Magnus Damm <damm@opensource.se>

Thanks for your patch!

Do you plan to update the other occurrences under
Documentation/devicetree/bindings/, too?

Gr{oetje,eeting}s,

                        Geert
Simon Horman Sept. 19, 2018, 8:30 a.m. UTC | #2
On Mon, Sep 17, 2018 at 01:20:57PM +0200, Geert Uytterhoeven wrote:
> Hi Magnus,
> 
> On Mon, Sep 17, 2018 at 10:39 AM Magnus Damm <magnus.damm@gmail.com> wrote:
> > From: Magnus Damm <damm@opensource.se>
> >
> > Update the description of the AG5 entry to follow same style as other SoCs.
> >
> > The actual DT binding remains unchanged.
> >
> > Signed-off-by: Magnus Damm <damm@opensource.se>
> 
> Thanks for your patch!
> 
> Do you plan to update the other occurrences under
> Documentation/devicetree/bindings/, too?

Which occurrences are you concerned about?
Geert Uytterhoeven Sept. 19, 2018, 8:43 a.m. UTC | #3
On Wed, Sep 19, 2018 at 10:31 AM Simon Horman <horms@verge.net.au> wrote:
> On Mon, Sep 17, 2018 at 01:20:57PM +0200, Geert Uytterhoeven wrote:
> > On Mon, Sep 17, 2018 at 10:39 AM Magnus Damm <magnus.damm@gmail.com> wrote:
> > > From: Magnus Damm <damm@opensource.se>
> > >
> > > Update the description of the AG5 entry to follow same style as other SoCs.
> > >
> > > The actual DT binding remains unchanged.
> > >
> > > Signed-off-by: Magnus Damm <damm@opensource.se>
> >
> > Thanks for your patch!
> >
> > Do you plan to update the other occurrences under
> > Documentation/devicetree/bindings/, too?
>
> Which occurrences are you concerned about?

git grep SH73A0 -- Documentation/devicetree/bindings/

I have mixed feelings about this patch. AFAIU, it was called SH73A0 originally,
but renamed R8A73A0 after SuperHitachi and successors morphed into Renesas?

Gr{oetje,eeting}s,

                        Geert
Simon Horman Sept. 19, 2018, 9:24 a.m. UTC | #4
On Wed, Sep 19, 2018 at 10:43:53AM +0200, Geert Uytterhoeven wrote:
> On Wed, Sep 19, 2018 at 10:31 AM Simon Horman <horms@verge.net.au> wrote:
> > On Mon, Sep 17, 2018 at 01:20:57PM +0200, Geert Uytterhoeven wrote:
> > > On Mon, Sep 17, 2018 at 10:39 AM Magnus Damm <magnus.damm@gmail.com> wrote:
> > > > From: Magnus Damm <damm@opensource.se>
> > > >
> > > > Update the description of the AG5 entry to follow same style as other SoCs.
> > > >
> > > > The actual DT binding remains unchanged.
> > > >
> > > > Signed-off-by: Magnus Damm <damm@opensource.se>
> > >
> > > Thanks for your patch!
> > >
> > > Do you plan to update the other occurrences under
> > > Documentation/devicetree/bindings/, too?
> >
> > Which occurrences are you concerned about?
> 
> git grep SH73A0 -- Documentation/devicetree/bindings/
> 
> I have mixed feelings about this patch. AFAIU, it was called SH73A0 originally,
> but renamed R8A73A0 after SuperHitachi and successors morphed into Renesas?

Thanks.

I agree that we should strive to be consistent.
Lets see if anyone has some more information regarding the
history of the name for this SoC.
diff mbox series

Patch

--- 0001/Documentation/devicetree/bindings/arm/shmobile.txt
+++ work/Documentation/devicetree/bindings/arm/shmobile.txt	2018-09-17 13:00:41.773502304 +0900
@@ -9,7 +9,7 @@  SoCs:
     compatible = "renesas,r7s72100"
   - RZ/A2 (R7S9210)
     compatible = "renesas,r7s9210"
-  - SH-Mobile AG5 (R8A73A00/SH73A0)
+  - SH-Mobile AG5 (R8A73A00)
     compatible = "renesas,sh73a0"
   - R-Mobile APE6 (R8A73A40)
     compatible = "renesas,r8a73a4"