Re: [PATCH 1/2] dt-bindings: display: bridge: icn6211: Add support for RGB/BGR swap

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 



On Mon, Aug 8, 2022 at 3:57 AM Marek Vasut <marex@xxxxxxx> wrote:
> On 8/4/22 00:41, Rob Herring wrote:
> > On Tue, Aug 2, 2022 at 5:33 AM Marek Vasut <marex@xxxxxxx> wrote:
> >>
> >> On 8/1/22 18:32, Rob Herring wrote:
> >>> On Mon, Aug 01, 2022 at 03:19:00PM +0200, Marek Vasut wrote:
> >>>> The ICN6211 is capable of swapping the output DPI RGB/BGR color channels,
> >>>> document a DT property to select this swap in DT. This can be useful on
> >>>> hardware where such swap happens.
> >>>
> >>> We should ensure this series[1] works for you instead.
> >>
> >> [...]
> >>
> >>> Rob
> >>>
> >>> [1] https://lore.kernel.org/r/20220628181838.2031-3-max.oss.09@xxxxxxxxx
> >>
> >> I'm still not convinced that we should encode this pixel format value
> >> directly into the DT instead of trying to describe the DPI bus color
> >> channel width/order/shift in the DT instead. I think I mentioned that
> >> before in one of the previous versions of that series.
> >
> > I worry that it gets pretty verbose, but worth having the discussion.
> > In any case, let's have that discussion and not add yet another one
> > off property.
>
> Done, I replied

So what is the verdict? Is this patch set dropped or shall it be
applied?

Yours,
Linus Walleij



[Index of Archives]     [Linux DRI Users]     [Linux Intel Graphics]     [Linux USB Devel]     [Video for Linux]     [Linux Audio Users]     [Yosemite News]     [Linux Kernel]     [Linux SCSI]     [XFree86]     [Linux USB Devel]     [Video for Linux]     [Linux Audio Users]     [Linux Kernel]     [Linux SCSI]     [XFree86]
  Powered by Linux