From mboxrd@z Thu Jan 1 00:00:00 1970 From: Tony Lindgren Subject: Re: [Letux-kernel] [PATCH 2/2] DTS: ARM: gta04: introduce legacy spi-cs-high to make display work again Date: Fri, 20 Sep 2019 08:53:06 -0700 Message-ID: <20190920155306.GT5610@atomide.com> References: <20190831084852.5e726cfa@aktux> <1624298A-C51B-418A-96C3-EA09367A010D@goldelico.com> <7DF102BC-C818-4D27-988F-150C7527E6CC@goldelico.com> <20190920142059.GO5610@atomide.com> <633E7AD9-A909-4619-BBD7-8CFD965FDFF7@goldelico.com> <20190920172947.51c1fdec@aktux> <96E62EC2-2A3E-4722-A9DE-3F320B0A98B0@goldelico.com> Mime-Version: 1.0 Content-Type: text/plain; charset=us-ascii Cc: Andreas Kemnade , Discussions about the Letux Kernel , Mark Rutland , Rob Herring , Linux-OMAP , devicetree , Linus Walleij , Linux Kernel Mailing List , stable , linux-spi , Mark Brown , =?utf-8?Q?Beno=C3=AEt?= Cousson To: "H. Nikolaus Schaller" Return-path: Content-Disposition: inline In-Reply-To: <96E62EC2-2A3E-4722-A9DE-3F320B0A98B0@goldelico.com> Sender: linux-kernel-owner@vger.kernel.org List-Id: linux-spi.vger.kernel.org * H. Nikolaus Schaller [190920 15:50]: > > > Am 20.09.2019 um 17:29 schrieb Andreas Kemnade : > > > > On Fri, 20 Sep 2019 16:54:18 +0200 > > "H. Nikolaus Schaller" wrote: > > > >>> Am 20.09.2019 um 16:20 schrieb Tony Lindgren : > >>> > >>> * H. Nikolaus Schaller [190920 09:19]: > >>>>> Am 20.09.2019 um 10:55 schrieb Linus Walleij : > >>>>> 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. Please just repost a minimal dts one line fix. Then a separate patch for the documentation changes. Regards, Tony