[PATCH] RISC-V: Cosmetic menuconfig changes
palmer at sifive.com
Mon Oct 1 09:56:18 PDT 2018
On Sat, 29 Sep 2018 13:45:05 PDT (-0700), mick at ics.forth.gr wrote:
> Στις 2018-09-29 04:44, Palmer Dabbelt έγραψε:
>> On Wed, 19 Sep 2018 15:48:15 PDT (-0700), mick at ics.forth.gr wrote:
>>> * Move the built-in cmdline configuration on a new menu entry "Boot
>>> options", it doesn't make much sense to be part of the debuging
>>> * Rename "Kernel Type" menu to "Kernel features" to be more consistent
>>> what other architectures are using, plus "type" is a bit misleading
>>> Signed-off-by: Nick Kossifidis <mick at ics.forth.gr>
>> This seems reasonable to me. I've added it to our for-next branch,
>> we'll see if anyone else has comments. Thanks!
>> Did you, by any chance, go and check to see if these boot options
>> actually work? We've got a bunch of subtly different boot methods and
>> they've been broken in various ones at various times, so I always like
>> to try and catalog what actually works now so at least I know where to
>> look when testing things.
> I assumed that since it was already there it worked but you are right,
> cmdline when passed from the bootloader through the generated device
> works but when passed through the config option doesn't. Do you want me
> to send a separate patch for that or re-send this one as well as a
> series ?
I think it's better to send that out as a separate patch. There's really no
reason to couple the two, it's just that this reminded me that we probably had
some bugs here.
More information about the linux-riscv