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

[Marek Vasut]

On 8/26/22 13:52, Linus Walleij wrote:
> On Mon, Aug 8, 2022 at 3:57 AM Marek Vasut <marex at denx.de> wrote:
>> On 8/4/22 00:41, Rob Herring wrote:
>>> On Tue, Aug 2, 2022 at 5:33 AM Marek Vasut <marex at denx.de> 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@gmail.com
>>>>
>>>> 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.