[U-Boot] [u-boot-release] [Patch v3 7/7] powerpc/8xxx: Add support for interactive DDR programming interface

[Timur Tabi]

York Sun wrote:

> I think the interactive command is self-explained. 

Why do people say things like that?  If I say that it needs to be better
documented, then obviously it isn't self-explanatory.

> I can add some
> examples if needed. But I am afraid the example will be either too short
> or too long.
> 
> For example, I can add the following
> 
> First step, run "compute" command, it returns with the DIMM part number
> 
> FSL DDR>compute
> Detected UDIMM UG51U6400N8SU-ACF
> 
> Second step, users can run 'print' command with arguments. Without
> argument, a help message will print out

This is not an example.  This is a walk-through.  An example shows how
individual commands can be used, and what the output could look like.

> I can further show the examples of print/edit dimmparams, commonparams,
> opts, regs. It will be too long. It is not difficult to use the
> self-guided interface. Agree?

How could it be too long?  Is there not enough room on the git server to store a
text file?  This file could be 200KB is size, and that would be okay.

-- 
Timur Tabi
Linux kernel developer at Freescale