Hi Ezequiel,
Thanks for the patch
On Fri, Dec 21, 2012 at 5:02 PM, Enric Balletbo Serra
<eballetbo@xxxxxxxxx> wrote:
> Hi Ezequiel,
>
> Thanks for the patch.
>
> 2012/12/20 Ezequiel Garcia <ezequiel.garcia@xxxxxxxxxxxxxxxxxx>:
>> IGEP v2 boards has console as ttyO2,115200n8.
>> It's better so simply set this as the default kernel argument
>> directly in the device tree board file.
>>
>> Cc: Enric Balletbo i Serra <eballetbo@xxxxxxxxx>
>> Cc: Javier Martinez Canillas <javier@xxxxxxxxxxxx>
>> Signed-off-by: Ezequiel Garcia <ezequiel.garcia@xxxxxxxxxxxxxxxxxx>
>> ---
>> arch/arm/boot/dts/omap3-igep0020.dts | 4 ++++
>> 1 files changed, 4 insertions(+), 0 deletions(-)
>>
>> diff --git a/arch/arm/boot/dts/omap3-igep0020.dts b/arch/arm/boot/dts/omap3-igep0020.dts
>> index e2b9849..45ffe21 100644
>> --- a/arch/arm/boot/dts/omap3-igep0020.dts
>> +++ b/arch/arm/boot/dts/omap3-igep0020.dts
>> @@ -15,6 +15,10 @@
>> model = "IGEPv2";
>> compatible = "isee,omap3-igep0020", "ti,omap3";
>>
>> + chosen {
>> + bootargs = "console=ttyO2,115200n8 earlyprintk";
>> + };
>> +
>
> The u-boot 'bootargs' env overwrites the one from chosen ?
>
I just tested and U-Boot 'bootargs' has precedence over DT bootargs
but I didn't find a way to append boot parameters from U-Boot. You can
either overwrite it completely or not.
That been said I think that is a good idea to have some default
bootargs on the DT but since you can't append any boot option from
U-Boot I would also add root=/dev/mmcblk0p2 rw rootwait so at least it
will be able to mount the rootfs from the uSD/MMC.
Also, many IGEP COM Module base boards also have serial consoles that
are connected to the OMAP3 UART3 (ttyO2), so maybe is better to add
this to omap3-igep.dtsi instead of omap3-igep0020.dts to avoid future
duplication.
Best regards,
Javier
--
To unsubscribe from this list: send the line "unsubscribe linux-omap" in
the body of a message to majordomo@xxxxxxxxxxxxxxx
More majordomo info at http://vger.kernel.org/majordomo-info.html
