[U-Boot] [PATCH 09/30] riscv: move target selection into separate file

[Bin Meng]

On Tue, Oct 23, 2018 at 10:48 AM Rick Chen <rickchen36 at gmail.com> wrote:
>
> > > Subject: Re: [PATCH 09/30] riscv: move target selection into separate file
> > >
> > > Hi Lukas,
> > >
> > > On Sat, Oct 20, 2018 at 6:10 AM Lukas Auer <lukas.auer at aisec.fraunhofer.de>
> > > wrote:
> > > >
> > > > Move the target selection into a separate file (Kconfig.board) to
> > > > avoid clutter once we support more boards.
> > > >
> > > > Signed-off-by: Lukas Auer <lukas.auer at aisec.fraunhofer.de>
> > > > ---
> > > >
> > > >  arch/riscv/Kconfig       | 17 ++---------------
> > > >  arch/riscv/Kconfig.board | 14 ++++++++++++++
> > > >  2 files changed, 16 insertions(+), 15 deletions(-)  create mode
> > > > 100644 arch/riscv/Kconfig.board
> > > >
> > > > diff --git a/arch/riscv/Kconfig b/arch/riscv/Kconfig index
> > > > ce07fb4b55..10d17a0e18 100644
> > > > --- a/arch/riscv/Kconfig
> > > > +++ b/arch/riscv/Kconfig
> > > > @@ -4,21 +4,6 @@ menu "RISC-V architecture"
> > > >  config SYS_ARCH
> > > >         default "riscv"
> > > >
> > > > -choice
> > > > -       prompt "Target select"
> > > > -       optional
> > > > -
> > > > -config TARGET_AX25_AE350
> > > > -       bool "Support ax25-ae350"
> > > > -
> > > > -config TARGET_QEMU_VIRT
> > > > -       bool "Support QEMU Virt Board"
> > > > -
> > > > -endchoice
> > > > -
> > > > -source "board/AndesTech/ax25-ae350/Kconfig"
> > > > -source "board/emulation/qemu-riscv/Kconfig"
> > > > -
> > > >  choice
> > > >         prompt "Base ISA"
> > > >         default ARCH_RV32I
> > > > @@ -72,4 +57,6 @@ config 32BIT
> > > >  config 64BIT
> > > >         bool
> > > >
> > > > +source "arch/riscv/Kconfig.board"
> > > > +
> > >
> > > I am OK with moving board one to a separate file, though it looks no other arch
> > > uses scuh convention in U-Boot :)
> > >
>
> Though no other arch use Kconfig.board here.
> Because RISC-V is an open architechture.
> Maybe separate board option from Kconfig to Kconfig.board is an good idea.
>

I suggest we keep current kconfig structure for now to be in
consistent with other archs.

Regards,
Bin