Re: [Letux-kernel] [PATCH 2/2] DTS: ARM: gta04: introduce legacy spi-cs-high to make display work again

From: H. Nikolaus Schaller
Date: Fri Sep 20 2019 - 11:55:40 EST



> Am 20.09.2019 um 17:50 schrieb H. Nikolaus Schaller <hns@xxxxxxxxxxxxx>:
>
>
>> Am 20.09.2019 um 17:29 schrieb Andreas Kemnade <andreas@xxxxxxxxxxxx>:
>>
>> On Fri, 20 Sep 2019 16:54:18 +0200
>> "H. Nikolaus Schaller" <hns@xxxxxxxxxxxxx> wrote:
>>
>>>> Am 20.09.2019 um 16:20 schrieb Tony Lindgren <tony@xxxxxxxxxxx>:
>>>>
>>>> * H. Nikolaus Schaller <hns@xxxxxxxxxxxxx> [190920 09:19]:
>>>>>> Am 20.09.2019 um 10:55 schrieb Linus Walleij <linus.walleij@xxxxxxxxxx>:
>>>>>> I suggest to go both way:
>>>>>> apply this oneliner and tag for stable so that GTA04 works
>>>>>> again.
>>>>>>
>>>>>> Then for the next kernel think about a possible more abitious
>>>>>> whitelist solution and after adding that remove *all* "spi-cs-high"
>>>>>> flags from all device trees in the kernel after fixing them
>>>>>> all up.
>>>>>
>>>>> Ok, that looks like a viable path.
>>>>
>>>> Please repost the oneline so people can ack easily. At least
>>>> I've already lost track of this thread.
>>>
>>> It is all here:
>>>
>>> https://patchwork.kernel.org/patch/11035253/
>>>
>> It is the full one (incl. documentation), not the oneline and does not
>> apply.
>
> Looks as if it was sitting too long in the queue and linux-next has changed
> the basis in the meantime, while v5.3 has not yet.
>
> Documentation/devicetree/bindings/spi/spi-bus.txt -> spi-controller.yaml
>
> So it should still apply for v5.3.1 and earlier and we need both versions.
> One for stable and one for linux-next. I don't know how to handle such cases.

Ok, here is a correction of this statement:

It applies fine to v5.2 and v5.3 already uses the spi-controller.yaml