Re: [BUG] SPI broken for SPI based panel drivers

From: Sven Van Asbroeck
Date: Fri Dec 04 2020 - 14:20:20 EST


On Fri, Dec 4, 2020 at 11:52 AM H. Nikolaus Schaller <hns@xxxxxxxxxxxxx> wrote:
>
> >> Anyways it is debatable if this is a bug at all. It is just a definition.
> >
> > I respectfully disagree. Prior to the fix, your panel's active-low chip select
> > needed to be described in the devicetree with 'spi-cs-high'. That sounds
> > very much like a bug to me.
>
> It could have been described by ACTIVE_LOW without spi-cs-high but that did
> emit a nasty and not helpful warning on each boot.

That will not work, try it out. You will see that without the bugfix, your chip
select is consistently inverted, no matter how you formulate it in the
devicetree.

>
> I'd prefer if you or maybe Linus could submit such a patch and I am happy to review it.

I cannot help you with that, I'm sorry.