[U-Boot] [PATCH 1/4] doc: Add documentation for mpc85xx debugger support

Wolfgang Denk wd at denx.de
Wed Mar 7 07:24:52 CET 2012


Dear Kushwaha Prabhakar-B32579,

In message <071A08F2C6A57E4E94D980ECA553F874575244 at 039-SN1MPN1-005.039d.mgd.msft.net> you wrote:
>
> Regarding CONFIG_E500_V1_V2, Its description is also part of this
> patch or is it not cleared ?

First, documentation of CONFIG_ options belongs into the central
README, so we have it all in a single place.

Second, "Enables code taking care of above mentioned rule" is not
really helpful to understand what it's actually doing.

The name of the cvariable suggests that this defiens a E500 core
based system, but it does not even contain a slight hint that it has
something to do with debugging.

Also, what's the "V1_V2" ? Are there also other systems (say, e500 v3
cores), and are this not affected? We already have CONFIG_E500 and
CONFIG_E500MC so CONFIG_E500_V1_V2 appears to belong to this group,
but if I understand your intentions it does something completely
unrelated.

All this is highly confusing.

Best regards,

Wolfgang Denk

-- 
DENX Software Engineering GmbH,     MD: Wolfgang Denk & Detlev Zundel
HRB 165235 Munich, Office: Kirchenstr.5, D-82194 Groebenzell, Germany
Phone: (+49)-8142-66989-10 Fax: (+49)-8142-66989-80 Email: wd at denx.de
If you hear an onion ring, answer it.


More information about the U-Boot mailing list