[U-Boot] [PATCH 0/57] RFC: Move arch-specific global data into its own structure
Wolfgang Denk
wd at denx.de
Tue Nov 20 08:25:17 CET 2012
Dear Simon Glass,
In message <1353100842-20126-1-git-send-email-sjg at chromium.org> you wrote:
> The previous generic board series hit a snag in that we needed generic
> code to access some of the architecture-specific fields in global_data.
I missed that. Can you please summarize what exactly the problem was,
and how this modification is supposed to fix it?
> The solution eventually arrived at was to move these fields into a
> separate structure, so that global_data has the generic fields,
> and within that there is an arch_global_data structure holding the
> architecture-specific ones.
>
> This series makes that change. Assuming this is reasonable, the next
> step is to bring back the generic board patches on top of this.
This cover letter has a RFC in the subject,. but the following patch
series does not. This is actually bad!
General comments / questions:
- We always attempted to keep global data as small as possible. What
happens here appears to be a move in a totally wrong direction.
Instead of simplyfiyng it (and moving stuff out of global data), we
add more and more complexity to it. That's wrong. We should not
do that.
- The change makes the code less readable. Reading "gd->arch."
instead of plain "gd->" is no improvements, but rather vice versa.
If we really go this way, this should be improved.
- What exactly is the impact of this code changes on the memory
footprint?
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
I'm what passes for a Unix guru in my office. This is a frightening
concept. - Lee Ann Goldstein, in <3k55ba$c43 at butch.lmsc.lockheed.com>
More information about the U-Boot
mailing list