[U-Boot] [PATCH v2 11/58] keymile: Unroll km/km83xx-common.h

Mario Six mario.six at gdsys.cc
Mon Nov 26 06:35:09 UTC 2018


Hi Holger,

On Tue, Nov 13, 2018 at 11:42 AM Holger Brunck <holger.brunck at ch.abb.com> wrote:
>
> Hi Mario,
>
> > Simplify the keymile config files once more by unrolling the
> > km/km83xx-common.h, and resolve the #ifdef logic as needed.
> >
> > Signed-off-by: Mario Six <mario.six at gdsys.cc>
> >
> > ---
> >
> > v1 -> v2:
> > No changes
> >
> > ---
>  > include/configs/km8360.h        | 291 ++++++++++++++++++++++++++++++++++++++-
>  > include/configs/km83xx-common.h | 296 ----------------------------------------
>  > include/configs/kmopti2.h       | 289 ++++++++++++++++++++++++++++++++++++++-
>  > include/configs/kmsupx5.h       | 289 ++++++++++++++++++++++++++++++++++++++-
>  > include/configs/kmtegr1.h       | 288 +++++++++++++++++++++++++++++++++++++-
>  > include/configs/kmtepr2.h       | 289 ++++++++++++++++++++++++++++++++++++++-
>  > include/configs/kmvect1.h       | 288 +++++++++++++++++++++++++++++++++++++-
>  > include/configs/suvd3.h         | 291 ++++++++++++++++++++++++++++++++++++++-
>  > include/configs/tuge1.h         | 289 ++++++++++++++++++++++++++++++++++++++-
>  > include/configs/tuxx1.h         | 289 ++++++++++++++++++++++++++++++++++++++-
>  > 10 files changed, 2579 insertions(+), 320 deletions(-)
>  > delete mode 100644 include/configs/km83xx-common.h
>
> could you elaborate why you need to do this? Turning 320 lines into 2579
> does not sound like simplifying the code.  I understand that you
> try to move a lot of config options to Kconfig for 83xx, which is good. But
> why can't we keep common header files for things which are common for
> some of our boards? For example duplicating CS configurations or SDRAM
> settings for similar boards seems to be unneeded to me and increases
> maintenance in the future.
>
The purpose of the unrolling is to use semi-automatic conversion tools (i.e. a
bunch of Python scripts I threw together) to move most of the config options
over to Kconfig; mostly for the "bit-field" options, like CONFIG_SYS_{BR,OR}_*
that are actually a bunch of options rolled into one, which are turned into
multiple Kconfig options.

But, yes, I'll re-instate the common includes, no problem. I made the series
under the initial series impression that the keymile boards were essentially
unmaintained, so I thought nobody would care. But now that you're actively
maintaining again, I'll put the configs back in a readable state. :-)

Concerning the boards, though: Are there any plans for migrating them to DM? I
see at least a kmeter1 device tree in the Linux kernel, but the other boards
would need device trees as well (or you could do everything with platdata, I
guess). There are DM drivers for a lot of MPC83xx functions upstream now, so
quite a few things should work already.

> Best regards
> Holger Brunck

Best regards,
Mario


More information about the U-Boot mailing list