Patchwork [U-Boot] omap5_common: Add optargs variable for kernel command line args

login
register
mail settings
Submitter Tom Rini
Date April 11, 2013, 3:22 p.m.
Message ID <1365693730-2330-1-git-send-email-trini@ti.com>
Download mbox | patch
Permalink /patch/235794/
State Accepted
Delegated to: Tom Rini
Headers show

Comments

Tom Rini - April 11, 2013, 3:22 p.m.
Add 'optargs' variable to be set to additional kernel arguments, similar
to omap3*/am3* usage.

Cc: Sricharan R <r.sricharan@ti.com>
Signed-off-by: Tom Rini <trini@ti.com>
---
 include/configs/omap5_common.h |    2 ++
 1 file changed, 2 insertions(+)
SRICHARAN R - April 15, 2013, 7:10 a.m.
On Thursday 11 April 2013 08:52 PM, Tom Rini wrote:
> Add 'optargs' variable to be set to additional kernel arguments, similar
> to omap3*/am3* usage.
> 
> Cc: Sricharan R <r.sricharan@ti.com>
> Signed-off-by: Tom Rini <trini@ti.com>
> ---
>  include/configs/omap5_common.h |    2 ++
>  1 file changed, 2 insertions(+)
> 
> diff --git a/include/configs/omap5_common.h b/include/configs/omap5_common.h
> index 2751627..5384a55 100644
> --- a/include/configs/omap5_common.h
> +++ b/include/configs/omap5_common.h
> @@ -150,10 +150,12 @@
>  	"usbtty=cdc_acm\0" \
>  	"vram=16M\0" \
>  	"partitions=" PARTS_DEFAULT "\0" \
> +	"optargs=\0" \
>  	"mmcdev=0\0" \
>  	"mmcroot=/dev/mmcblk0p2 rw\0" \
>  	"mmcrootfstype=ext3 rootwait\0" \
>  	"mmcargs=setenv bootargs console=${console} " \
> +		"${optargs} " \
 Sorry, what is this used for ?

Regards,
 Sricharan
Tom Rini - April 15, 2013, 12:14 p.m.
On Mon, Apr 15, 2013 at 12:40:19PM +0530, Sricharan R wrote:

> On Thursday 11 April 2013 08:52 PM, Tom Rini wrote:
> > Add 'optargs' variable to be set to additional kernel arguments, similar
> > to omap3*/am3* usage.
> > 
> > Cc: Sricharan R <r.sricharan@ti.com>
> > Signed-off-by: Tom Rini <trini@ti.com>
> > ---
> >  include/configs/omap5_common.h |    2 ++
> >  1 file changed, 2 insertions(+)
> > 
> > diff --git a/include/configs/omap5_common.h b/include/configs/omap5_common.h
> > index 2751627..5384a55 100644
> > --- a/include/configs/omap5_common.h
> > +++ b/include/configs/omap5_common.h
> > @@ -150,10 +150,12 @@
> >  	"usbtty=cdc_acm\0" \
> >  	"vram=16M\0" \
> >  	"partitions=" PARTS_DEFAULT "\0" \
> > +	"optargs=\0" \
> >  	"mmcdev=0\0" \
> >  	"mmcroot=/dev/mmcblk0p2 rw\0" \
> >  	"mmcrootfstype=ext3 rootwait\0" \
> >  	"mmcargs=setenv bootargs console=${console} " \
> > +		"${optargs} " \
>  Sorry, what is this used for ?

The common form is you set this in uEnv.txt to whatever you happen to
need (say force the display to be some resolution) and not have to
re-write the whole of mmcargs.  Or other times you want to pass in
additional args, without modifying the rest of the boot logic.
Tom Rini - April 29, 2013, 8:21 p.m.
On Thu, Apr 11, 2013 at 05:22:10AM -0000, Tom Rini wrote:

> Add 'optargs' variable to be set to additional kernel arguments, similar
> to omap3*/am3* usage.
> 
> Cc: Sricharan R <r.sricharan@ti.com>
> Signed-off-by: Tom Rini <trini@ti.com>

Applied to u-boot-ti/master, thanks!

Patch

diff --git a/include/configs/omap5_common.h b/include/configs/omap5_common.h
index 2751627..5384a55 100644
--- a/include/configs/omap5_common.h
+++ b/include/configs/omap5_common.h
@@ -150,10 +150,12 @@ 
 	"usbtty=cdc_acm\0" \
 	"vram=16M\0" \
 	"partitions=" PARTS_DEFAULT "\0" \
+	"optargs=\0" \
 	"mmcdev=0\0" \
 	"mmcroot=/dev/mmcblk0p2 rw\0" \
 	"mmcrootfstype=ext3 rootwait\0" \
 	"mmcargs=setenv bootargs console=${console} " \
+		"${optargs} " \
 		"vram=${vram} " \
 		"root=${mmcroot} " \
 		"rootfstype=${mmcrootfstype}\0" \