Re: [RFC PATCH-for-5.2 4/4] docs/system/arm: Document the Sharp Zaurus SL-6000

[Philippe Mathieu-Daudé]

On 11/20/20 5:52 PM, Peter Maydell wrote:
> On Fri, 20 Nov 2020 at 15:22, Philippe Mathieu-Daudé <f4bug@amsat.org> wrote:
>>
>> List the 'tosa' machine with the XScale-based PDAs models.
>>
>> Signed-off-by: Philippe Mathieu-Daudé <f4bug@amsat.org>
>> ---
>>  docs/system/arm/xscale.rst | 13 ++++++++-----
>>  1 file changed, 8 insertions(+), 5 deletions(-)
>>
>> diff --git a/docs/system/arm/xscale.rst b/docs/system/arm/xscale.rst
>> index 89ec93e904e..2dd2f8f9a56 100644
>> --- a/docs/system/arm/xscale.rst
>> +++ b/docs/system/arm/xscale.rst
>> @@ -1,16 +1,19 @@
>> -Sharp XScale-based PDA models (``akita``, ``borzoi``, ``spitz``, 
>> ``terrier``)
>> -=============================================================================
>> +Sharp XScale-based PDA models (``tosa``, ``spitz``, ``akita``, ``borzoi``, 
>> ``terrier``)
>> +=======================================================================================
> 
> These were in alphabetical order -- why the rearrangement ?

Ah I didn't notice, I sorted by chronological order. I'll add tosa
at the end (alphabetically last).

> 
>> +
>> +The Sharp Zaurus SL-6000 (``tosa``), released in 2005, was a PDA based on 
>> the
>> +PXA255.
>>
>>  The XScale-based clamshell PDA models (\"Spitz\", \"Akita\", \"Borzoi\"
>>  and \"Terrier\") emulation includes the following peripherals:
> 
> If you want to add tosa to this file (which seems reasonable) then
> you need to rewrite this para that introduces the bulleted list
> because it currently only describes spitz/akita/borzoi/terrier,
> so that it is instead a suitable introductory paragraph that
> covers all of the boards.

OK, I think added \"Tosa\" at the end is enough.

> 
>> --  Intel PXA270 System-on-chip (ARMv5TE core)
>> +-  Intel PXA255/PXA270 System-on-chip (ARMv5TE core)
>>
>> --  NAND Flash memory
>> +-  NAND Flash memory - not in \"Tosa\"
>>
>>  -  IBM/Hitachi DSCM microdrive in a PXA PCMCIA slot - not in \"Akita\"
>>
>> --  On-chip OHCI USB controller
>> +-  On-chip OHCI USB controller - not in \"Tosa\"
>>
>>  -  On-chip LCD controller
> 
> 
> thanks
> -- PMM
>