Messages in this thread | | | Date | Tue, 28 Jul 2015 10:54:50 +0200 | From | Nicolas Ferre <> | Subject | Re: [PATCH 1/3] ARM: at91/soc: add basic support for new sama5d2 SoC |
| |
Le 29/06/2015 17:04, Alexandre Belloni a écrit : > On 29/06/2015 at 14:46:51 +0200, Nicolas Ferre wrote : >> Le 18/06/2015 17:21, Alexandre Belloni a écrit : >>> Hi, >>> >>> On 18/06/2015 at 15:18:28 +0200, Nicolas Ferre wrote : >>>> diff --git a/Documentation/arm/Atmel/README b/Documentation/arm/Atmel/README >>>> index c53a19b4aab2..6650a8ffaa64 100644 >>>> --- a/Documentation/arm/Atmel/README >>>> +++ b/Documentation/arm/Atmel/README >>>> @@ -90,6 +90,12 @@ the Atmel website: http://www.atmel.com. >>>> + Datasheet >>>> http://www.atmel.com/Images/Atmel-11238-32-bit-Cortex-A5-Microcontroller-SAMA5D4_Datasheet.pdf >>>> >>>> + * ARM Cortex-A5 + NEON based SoCs >>>> + - sama5d2 family >>>> + - sama5d27 >>>> + + Datasheet >>>> + Coming soon >>> >>> Shouldn't that go directly before or after the sama5d4 family ? >> >> This is "after" sama5d4: I sorted this file according to SoC release >> date whereas I sorted the other entries in alpa-numerial order (sama5d2 >> placed before sama5d3). >> >> Is it what you answered? >> > > No, I meant doing something like: > > * ARM Cortex-A5 + NEON based SoCs > - sama5d4 family > - sama5d41 > - sama5d42 > - sama5d43 > - sama5d44 (device superset) > + Datasheet > http://www.atmel.com/Images/Atmel-11238-32-bit-Cortex-A5-Microcontroller-SAMA5D4_Datasheet.pdf > - sama5d2 family > - sama5d27 > + Datasheet > Coming soon > > Else, you end up with two "ARM Cortex-A5 + NEON based SoCs" sections.
Oh, yes. Sure, I'll modify this.
Do you think that it's worth sending another series or I can take it for the pull-request when it's modified?
Bye, -- Nicolas Ferre
| |