[<prev] [next>] [<thread-prev] [day] [month] [year] [list]
Message-ID: <0862C896-35D3-4032-8DA8-E4B7632CC333@fb.com>
Date: Fri, 2 Aug 2019 03:56:11 +0000
From: Tao Ren <taoren@...com>
To: Joel Stanley <joel@....id.au>
CC: Rob Herring <robh+dt@...nel.org>,
Mark Rutland <mark.rutland@....com>,
Andrew Jeffery <andrew@...id.au>,
devicetree <devicetree@...r.kernel.org>,
Linux ARM <linux-arm-kernel@...ts.infradead.org>,
"linux-aspeed@...ts.ozlabs.org" <linux-aspeed@...ts.ozlabs.org>,
"Linux Kernel Mailing List" <linux-kernel@...r.kernel.org>,
OpenBMC Maillist <openbmc@...ts.ozlabs.org>
Subject: Re: [PATCH] ARM: dts: aspeed: Add Facebook Wedge100 BMC
On 8/1/19, 8:02 PM, "Joel Stanley" <joel@....id.au> wrote:
> On Fri, 2 Aug 2019 at 01:02, Tao Ren <taoren@...com> wrote:
>> +
>> + chosen {
>> + stdout-path = &uart3;
>> + bootargs = "debug console=ttyS2,9600n8 root=/dev/ram rw";
>
> Are you sure you want 'debug' in your boot arguments?
>
> The rest lgtm. I can remove debug when applying, or leave it there if
> it was intentional.
Ahh, I copied bootargs from "/proc/cmdline" on my machine (running old kernel) but I don't think I need it.
Thank you for pointing it out. Let me send out v2 patches in a few minutes (so you could apply without extra changes).
Cheers,
Tao
Powered by blists - more mailing lists