[PATCH] aspeed/ast-g5: Simplify default bootargs

[Rick Altherr]

I had intended to remove the Change-Id tag but the gerrit hooks in a git
repo seem to run deep.  I believe Xo sent a PR with the same patch.  I
prefer to drop my patch and follow up on his.

On Tue, Jun 14, 2016 at 8:49 PM, Joel Stanley <joel at jms.id.au> wrote:

> On Wed, Jun 15, 2016 at 5:20 AM, Rick Altherr <raltherr at google.com> wrote:
> > From: Xo Wang <xow at google.com>
> >
> > New arguments set default console for DEBUG_UART on the EVB and allow
> > the kernel to choose init path, root/overlay, and memory size.
> >
> > Change-Id: I919f7e4027968ad000c4283f8bd80cca1b17e2eb
>
> I normally ask submissions to not include links to internal bugzillas
> when submitting patches. Do you think that's a reasonable rule for
> gerrit tags as well?
>
> > Signed-off-by: Xo Wang <xow at google.com>
> > Signed-off-by: Rick Altherr <raltherr at google.com>
> > ---
> >  include/configs/ast-g5.h | 2 +-
> >  1 file changed, 1 insertion(+), 1 deletion(-)
> >
> > diff --git a/include/configs/ast-g5.h b/include/configs/ast-g5.h
> > index 6c05e6d..149cac8 100644
> > --- a/include/configs/ast-g5.h
> > +++ b/include/configs/ast-g5.h
> > @@ -139,7 +139,7 @@
> >
> >  #define CONFIG_SYS_LOAD_ADDR           0x83000000      /* default load
> address */
> >
> > -#define CONFIG_BOOTARGS                "console=ttyS0,115200n8
> ramdisk_size=16384 root=/dev/ram rw init=/linuxrc mem=80M"
> > +#define CONFIG_BOOTARGS                "console=ttyS4,115200
> earlyprintk"
>
> The change looks good.
>
> As you mentioned on IRC, we should create an ast2500-evb configuration
> in the future.
>
> Cheers,
>
> Joel
>
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://lists.ozlabs.org/pipermail/openbmc/attachments/20160623/3a6ad90f/attachment-0001.html>