<div dir="ltr">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.</div><div class="gmail_extra"><br><div class="gmail_quote">On Tue, Jun 14, 2016 at 8:49 PM, Joel Stanley <span dir="ltr"><<a href="mailto:joel@jms.id.au" target="_blank">joel@jms.id.au</a>></span> wrote:<br><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex"><span class="">On Wed, Jun 15, 2016 at 5:20 AM, Rick Altherr <<a href="mailto:raltherr@google.com">raltherr@google.com</a>> wrote:<br>
> From: Xo Wang <<a href="mailto:xow@google.com">xow@google.com</a>><br>
><br>
> New arguments set default console for DEBUG_UART on the EVB and allow<br>
> the kernel to choose init path, root/overlay, and memory size.<br>
><br>
> Change-Id: I919f7e4027968ad000c4283f8bd80cca1b17e2eb<br>
<br>
</span>I normally ask submissions to not include links to internal bugzillas<br>
when submitting patches. Do you think that's a reasonable rule for<br>
gerrit tags as well?<br>
<span class=""><br>
> Signed-off-by: Xo Wang <<a href="mailto:xow@google.com">xow@google.com</a>><br>
> Signed-off-by: Rick Altherr <<a href="mailto:raltherr@google.com">raltherr@google.com</a>><br>
> ---<br>
>  include/configs/ast-g5.h | 2 +-<br>
>  1 file changed, 1 insertion(+), 1 deletion(-)<br>
><br>
> diff --git a/include/configs/ast-g5.h b/include/configs/ast-g5.h<br>
> index 6c05e6d..149cac8 100644<br>
> --- a/include/configs/ast-g5.h<br>
> +++ b/include/configs/ast-g5.h<br>
> @@ -139,7 +139,7 @@<br>
><br>
>  #define CONFIG_SYS_LOAD_ADDR           0x83000000      /* default load address */<br>
><br>
> -#define CONFIG_BOOTARGS                "console=ttyS0,115200n8 ramdisk_size=16384 root=/dev/ram rw init=/linuxrc mem=80M"<br>
> +#define CONFIG_BOOTARGS                "console=ttyS4,115200 earlyprintk"<br>
<br>
</span>The change looks good.<br>
<br>
As you mentioned on IRC, we should create an ast2500-evb configuration<br>
in the future.<br>
<br>
Cheers,<br>
<br>
Joel<br>
</blockquote></div><br></div>