[U-Boot] [PATCH v8] Marvell Kirkwood family SOC support

Wolfgang Denk wd at denx.de
Wed May 20 21:00:10 CEST 2009


Dear Prafulla,

In message <73173D32E9439E4ABB5151606C3E19E201CF9E64AE at SC-VEXCH1.marvell.com> you wrote:
> 
...
> > > > +#define KW_EGIGA0_BASE			(KW_REGISTER(0x72000))
> > > > +#define KW_EGIGA1_BASE			(KW_REGISTER(0x76000))
> > > 
> > > Use a C struct?
> > These are the Base address referred by register structures.

I am aware of this.

> > Generally this type of declaration used for other cpu/socs.

Well, "generally" is a weak argument - you find examples for both.
Coming from the PowerPC world, and haveing been using the code these
sinde > 10 years, I'm more accustomed to see something like a big IMMR
sturcture here.

> I have almost done with other changes except this one
> Do this really need to be converted C struct?

No, you do not *have* to. That's why I asked it as a question. If I
had to write the code, I would use a C struct, and I think it would be
easier to read.

But I don't insist on such a change. You can find arguments for both
solutions, so decide what deems best for you.

> I will have to put some efforts to get it done :-(.

Well, changing it would actually be trivial. But as mentioned above: I
don't insist. It's largely a matter of taste, I think.

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 people are good only because they fear punishment, and  hope  for
reward, then we are a sorry lot indeed."            - Albert Einstein


More information about the U-Boot mailing list