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.
