'bmp display' convert 24bit bmp to 16bit lcd format

[Simon Glass]

Hi Qianfan,

On Tue, 28 Dec 2021 at 20:05, qianfan <qianfanguijin at qq.com> wrote:
>
>
> 在 2021/12/28 16:52, Simon Glass 写道:
> > Hi Qianfan,
> >
> > +Anatolij Gustschin
> >
> > On Mon, 20 Dec 2021 at 18:22, qianfan <qianfanguijin at qq.com> wrote:
> >> Hi:
> >>
> >> I had tested 'bmp display' command with 24bits bmp picture,
> >> **video_bmp_display** function always convert 24bits picture to RGB555
> >> format if the lcd is 16bits. But most of all lcd controller is RGB565
> >> format. Should we improve this?
> > See also:
> >
> > https://patchwork.ozlabs.org/project/uboot/patch/20211119202408.1815506-16-sjg@chromium.org/
>
> Hi, I read this patch and it is resolved this question. But I don't know
> if it is a good idea that just replace RGB555 to RGB565.
>
> Maybe somedays after this patch merged, someone will complain about
> video_bmp_display doesn't support RGB555 mode.

Maybe, at which point it should really become an option. But you did
say that 565 is more common. The existing BPP enum is not really
descriptive enough.

Regards,
Simon