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 8/26/22 13:52, Linus Walleij wrote:
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?

Drop

Let's wait for the DPI pixel format description discussion in DT gets sorted out.



[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