2018-09-27 16:57:50

by Maksym Kokhan

[permalink] [raw]
Subject: [PATCH 2/8] drivers: of: ifdef out cmdline section

From: Daniel Walker <[email protected]>

It looks like there's some seepage of cmdline stuff into
the generic device tree code. This conflicts with the
generic cmdline implementation so I remove it in the case
when that's enabled.

Cc: Daniel Walker <[email protected]>
Cc: Daniel Walker <[email protected]>
Signed-off-by: Daniel Walker <[email protected]>
Signed-off-by: Maksym Kokhan <[email protected]>
---
drivers/of/fdt.c | 2 +-
1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/drivers/of/fdt.c b/drivers/of/fdt.c
index 800ad25..74b85ad 100644
--- a/drivers/of/fdt.c
+++ b/drivers/of/fdt.c
@@ -1096,7 +1096,7 @@ int __init early_init_dt_scan_chosen(unsigned long node, const char *uname,
* managed to set the command line, unless CONFIG_CMDLINE_FORCE
* is set in which case we override whatever was found earlier.
*/
-#ifdef CONFIG_CMDLINE
+#if defined(CONFIG_CMDLINE) && !defined(CONFIG_GENERIC_CMDLINE)
#if defined(CONFIG_CMDLINE_EXTEND)
strlcat(data, " ", COMMAND_LINE_SIZE);
strlcat(data, CONFIG_CMDLINE, COMMAND_LINE_SIZE);
--
2.7.4



2018-09-27 21:08:07

by Rob Herring

[permalink] [raw]
Subject: Re: [PATCH 2/8] drivers: of: ifdef out cmdline section

On Thu, Sep 27, 2018 at 11:55 AM Maksym Kokhan
<[email protected]> wrote:
>
> From: Daniel Walker <[email protected]>
>
> It looks like there's some seepage of cmdline stuff into
> the generic device tree code. This conflicts with the
> generic cmdline implementation so I remove it in the case
> when that's enabled.

What's preventing removing this or simplifying the DT code to just be
"give me the cmdline data from DT" and the common code deals with all
the combinations of config options.

Rob

2018-10-08 17:59:57

by Maksym Kokhan

[permalink] [raw]
Subject: Re: [PATCH 2/8] drivers: of: ifdef out cmdline section

Hi, Rob

On Fri, Sep 28, 2018 at 12:07 AM Rob Herring <[email protected]> wrote:
>
> On Thu, Sep 27, 2018 at 11:55 AM Maksym Kokhan
> <[email protected]> wrote:
> >
> > From: Daniel Walker <[email protected]>
> >
> > It looks like there's some seepage of cmdline stuff into
> > the generic device tree code. This conflicts with the
> > generic cmdline implementation so I remove it in the case
> > when that's enabled.
>
> What's preventing removing this or simplifying the DT code to just be
> "give me the cmdline data from DT" and the common code deals with all
> the combinations of config options.

This code (handling built-in cmdline in old, non-generic way) is used by
some other architectures besides those, modified in our patches, so for
now we can't remove it.

Thanks,
Maksym