From mboxrd@z Thu Jan 1 00:00:00 1970 From: Grant Likely Subject: Re: [PATCH V5 16/17] SPI: MIPS: lantiq: add FALCON spi driver Date: Wed, 30 May 2012 15:20:32 +0800 Message-ID: <20120530072032.247CF3E065C@localhost> References: <1337521579-1597-1-git-send-email-blogic@openwrt.org> <20120525233845.BD93C3E0BD2@localhost> <4FC0DEEC.8050204@openwrt.org> Mime-Version: 1.0 Content-Type: text/plain; charset="us-ascii" Content-Transfer-Encoding: 7bit Cc: linux-mips-6z/3iImG2C8G8FEW9MqTrA@public.gmane.org, Thomas Langer , Ralf Baechle , spi-devel-general-5NWGOfrQmneRv+LV9MX5uipxlwaOVQ5f@public.gmane.org To: John Crispin Return-path: In-Reply-To: <4FC0DEEC.8050204-p3rKhJxN3npAfugRpC6u6w@public.gmane.org> List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , Errors-To: spi-devel-general-bounces-5NWGOfrQmneRv+LV9MX5uipxlwaOVQ5f@public.gmane.org List-Id: linux-spi.vger.kernel.org On Sat, 26 May 2012 15:47:24 +0200, John Crispin wrote: > > > What exactly does this mean? How does it not support any other type > > of SPI peripheral? SPI is a really simple protocol, so what is it > > about this hardware that prevents it being used with other SPI > > hardware? > > > > I see a big state machine that appears to interpret the messages and > > pretend to be an SPI slave instead of telling linux about the real > > device. /me wonders if it should this instead be a block device > > driver? > > > > Thomas will need to comment on this part > > >> +static int falcon_sflash_prepare_xfer(struct spi_master *master) > >> +{ > >> + return 0; > >> +} > >> + > >> +static int falcon_sflash_unprepare_xfer(struct spi_master *master) > >> +{ > >> + return 0; > >> +} > > Don't use empty hooks. Just leave them uninitialized. The core will > > do the right thing. > > > > I was under the impression that the need for these 2 callbacks was > removed in 3.5. As this patch flows via MIPS there would be a merge > order problem making the kernel non bisectable > > I am a bit confused. You keep ack'ing this driver and then commenting on > it a few weeks later.... obsoleting the ACK ... Hahah. I receive a *lot* of email. I can't remember what I reviewed yesterday, let alone last week. If I ack something, then add my ack when you repost. Otherwise I don't have any clues as to what I've said in the past. Also, I reserve the right to review all new versions of patches; that doesn't invalidate the ack, but Ralf can decide whether to pick it up and ask for follow-up changes, or to ask for another respin. g. ------------------------------------------------------------------------------ Live Security Virtual Conference Exclusive live event will cover all the ways today's security and threat landscape has changed and how IT managers can respond. Discussions will include endpoint security, mobile security and the latest in malware threats. http://www.accelacomm.com/jaw/sfrnl04242012/114/50122263/