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
