[U-Boot-Users] [PATCH] Added support for multiple serial for MPC8XX
Wolfgang Denk
wd at denx.de
Wed Jun 6 01:49:49 CEST 2007
Dear Stefano,
in message <200706052139.16320.sbabic at denx.de> you wrote:
>
> > I really dislike this mess of #ifdef's
>
> I agree with you and I inserted this code against my will :)
;-)
> However, I didn't want to have two different coding styles with different
> meaning inside the same file. In fact, some lines later the whole mess of
> #ifdef was already inserted for the ppc4xx architecture using
> CONFIG_CONS_INDEX to switch among the interfaces.
Please feel free to clean this up the same was as I suggested. I'm
sure Stefan Roese will ACK such a change.
We should get rid of all this CONFIG_CONS_INDEX stuff.
> I didn't want to change this part because this would break all ppc4xx boards
> that are not part of the U-Boot tree. So I preferred to add a lot of #ifdef
> using the same CONFIG defines as in ppc4xx instead of creating a new one.
I'm afraid I don't understand what you mean. Usually we don't care
about boards that are not part of the U-Boot tree.
> I will resend the patch with the modifications for the mpc8xx, without
> touching ppc4xx part. This would not break any custom board.
Please feel free to fix it for the existing 4xx boards as well. I'd
really appreciate this, and I even volunteer to test it on a few of
them.
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
The best things in life are for a fee.
More information about the U-Boot
mailing list