From mboxrd@z Thu Jan 1 00:00:00 1970 From: Lubomir Rintel Subject: Re: [PATCH 10/11] spi: pxa2xx: Add ready signal Date: Thu, 11 Oct 2018 18:13:26 +0200 Message-ID: References: <20181010170936.316862-1-lkundrak@v3.sk> <20181010170936.316862-11-lkundrak@v3.sk> Mime-Version: 1.0 Content-Type: text/plain; charset="UTF-8" Content-Transfer-Encoding: 7bit Cc: Mark Brown , Geert Uytterhoeven , quozl@laptop.org, Rob Herring , Mark Rutland , Eric Miao , Haojian Zhuang , Daniel Mack , Robert Jarzmik , linux-spi , "open list:OPEN FIRMWARE AND FLATTENED DEVICE TREE BINDINGS" , Linux Kernel Mailing List , Linux ARM To: Geert Uytterhoeven Return-path: In-Reply-To: Sender: linux-kernel-owner@vger.kernel.org List-Id: linux-spi.vger.kernel.org On Thu, 2018-10-11 at 09:28 +0200, Geert Uytterhoeven wrote: > Hi Lubomir, > > On Wed, Oct 10, 2018 at 7:10 PM Lubomir Rintel > wrote: > > Strobe a GPIO line when the slave TX FIFO is filled. This is how > > the > > Embedded Controller on an OLPC XO-1.75 machine, that happens to be > > a SPI > > master, learns that it can initiate a transaction. > > > > Signed-off-by: Lubomir Rintel > > Thanks for your patch! > > I'm repeating my comments on the RFC below: Ah, sorry for not addressing those; it somehow slipped through my fingers. I'll wait a while to collect more feedback before sending v2 and then I'll include fixes for those. Thanks for the review. Lubo