Re: [meta-fsl-arm][PATCH] u-boot-fslc: mx6sabresd: Enable HDMI video in bootargs

[Nikolay Dimitrov <picmaster@mail.bg>]

Hi guys,

On 06/01/2015 10:46 PM, Nikolay Dimitrov wrote:
> Hi Fabio,
>
> On 06/01/2015 05:01 PM, Fabio Estevam wrote:
>> On Sun, May 31, 2015 at 6:01 PM, Nikolay Dimitrov <picmaster@mail.bg>
>> wrote:
>>
>>> ++#define VIDEO_ARGS    "video=mxcfb0:dev=hdmi,1280x720M@60,if=RGB24 " \
>>
>> What if the user wants to use LVDS as fb0?
>>
>> Not sure if we should really hardcode this kernel command line.
>
> What do you think about this:
>
>
> video_args_hdmi=setenv video_args $video_args
> video=mxcfb${fbid}:dev=hdmi,1280x720M@60,if=RGB24\0
> video_args_lvds=setenv video_args $video_args
> video=mxcfb${fbid}:dev=ldb,LDB-XGA,if=RGB666\0
> video_args_lcd=setenv video_args $video_args
> video=mxcfb${fbid}:dev=lcd,CLAA-WVGA,if=RGB666\0
> fb=0\0
> video_interfaces=hdmi lvds lcd\0
>
> config_video=for v in ${video_interfaces}; do \
>      run video_args_${v}; \
>      setexpr fb $fb + 1; \
> done\0
>
> mmcargs=setenv bootargs console=${console},${baudrate} \
>      root=${mmcroot} \
>      ${video_args}\0
> ...
> mmcboot=echo Booting from mmc ...; \
>      run config_video; \
>      run mmcargs; \
>      ...
>
>
> Now video interfaces can be selected and reordered at will just by
> editing video_interfaces env-var.

I've cleaned-up the code and sent a pull-request for you to review:

https://github.com/Freescale/u-boot-fslc/pull/8

Regards,
Nikolay