From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1755514AbaAVMvA (ORCPT ); Wed, 22 Jan 2014 07:51:00 -0500 Received: from mail-wg0-f42.google.com ([74.125.82.42]:43108 "EHLO mail-wg0-f42.google.com" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1752357AbaAVMu6 (ORCPT ); Wed, 22 Jan 2014 07:50:58 -0500 Date: Wed, 22 Jan 2014 12:50:49 +0000 From: Lee Jones To: linux-arm-kernel@lists.infradead.org, linux-kernel@vger.kernel.org Cc: computersforpeace@gmail.com, angus.clark@st.com, linux-mtd@lists.infradead.org Subject: Re: [PATCH v4 00/36] mtd: st_spi_fsm: Add new driver Message-ID: <20140122125049.GA8586@lee--X1> References: <1389188840-14306-1-git-send-email-lee.jones@linaro.org> <20140108143316.GA14575@lee--X1> MIME-Version: 1.0 Content-Type: text/plain; charset=utf-8 Content-Disposition: inline Content-Transfer-Encoding: 8bit In-Reply-To: <20140108143316.GA14575@lee--X1> User-Agent: Mutt/1.5.21 (2010-09-15) Sender: linux-kernel-owner@vger.kernel.org List-ID: X-Mailing-List: linux-kernel@vger.kernel.org Hi Brian, > Version 4: > Tended to Brian's previous review comments > - Checkpatch acceptance > - MODULE_DEVICE_TABLE() name slip correction > - Timeout issue(s) resolved > - Potential infinite loop mitigated > - Code clarity suggests heeded > - Duplication with MTD core code removed > - Upgraded to using ROUND_UP() helper > - Moved non-shared header code into main driver > - Relocated dynamic msg sequence stores into main struct > - Averted adaption of static (table) data > - Basic whitespace/spelling/data type/dev_err suggestions accepted > > Version 3: > Okay, this thing should be fully functional now. Identify a chip > based on it's JEDEC ID, Read, Write, Erase (all or by sector). > Support for various chip quirks added too. > > Version 2: > The first bunch of these patches have been on the MLs before, but > didn't receive a great deal of attention for the most part. We are > a little more featureful this time however. We can now successfully > setup and configure the N25Q256. We still can't read/write/erase > it though. I'll start work on that next week and will provide it in > the next instalment. > > Version 1: > First stab at getting this thing Mainlined. It doesn't do a great deal > yet, but we are able to initialise the device and dynamically set it up > correctly based on an extracted JEDEC ID. > > Documentation/devicetree/bindings/mtd/st-fsm.txt | 26 ++ > arch/arm/boot/dts/stih416-b2105.dts | 14 + > arch/arm/boot/dts/stih416-pinctrl.dtsi | 12 + > drivers/mtd/devices/Kconfig | 8 + > drivers/mtd/devices/Makefile | 1 + > drivers/mtd/devices/serial_flash_cmds.h | 81 ++++ > drivers/mtd/devices/st_spi_fsm.c | 2124 +++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++ > 7 files changed, 2266 insertions(+) Can you confirm receipt of this set, or would you like me to resend? Kind regards, Lee -- Lee Jones Linaro STMicroelectronics Landing Team Lead Linaro.org │ Open source software for ARM SoCs Follow Linaro: Facebook | Twitter | Blog