From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1752176AbcAFG1Q (ORCPT ); Wed, 6 Jan 2016 01:27:16 -0500 Received: from mx0a-0016f401.pphosted.com ([67.231.148.174]:27701 "EHLO mx0a-0016f401.pphosted.com" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1751106AbcAFG1O convert rfc822-to-8bit (ORCPT ); Wed, 6 Jan 2016 01:27:14 -0500 Date: Wed, 6 Jan 2016 14:23:40 +0800 From: Jisheng Zhang To: Khoa Dang Pham CC: Mark Brown , , Subject: Re: [RFC] spi: dw: support setting tmode dynamically Message-ID: <20160106142340.34e2d334@xhacker> In-Reply-To: References: <20151223192338.128e95a4@xhacker> <20151223121512.GV16023@sirena.org.uk> <20151223202952.28be5bab@xhacker> <20160105161231.GG6588@sirena.org.uk> X-Mailer: Claws Mail 3.13.1 (GTK+ 2.24.29; x86_64-pc-linux-gnu) MIME-Version: 1.0 Content-Type: text/plain; charset="UTF-8" Content-Transfer-Encoding: 8BIT X-Proofpoint-Virus-Version: vendor=fsecure engine=2.50.10432:,, definitions=2016-01-06_04:,, signatures=0 X-Proofpoint-Details: rule=outbound_notspam policy=outbound score=0 spamscore=0 suspectscore=0 malwarescore=0 phishscore=0 adultscore=0 bulkscore=0 classifier=spam adjust=0 reason=mlx scancount=1 engine=8.0.1-1507310008 definitions=main-1601060123 Sender: linux-kernel-owner@vger.kernel.org List-ID: X-Mailing-List: linux-kernel@vger.kernel.org Dear Khoa, Mark, On Wed, 6 Jan 2016 08:45:38 +0700 Khoa Dang Pham wrote: > Hi Mark, > > May I provide a bit more info about the "EEPROM read" on this controller? > > According to the "DesignWare DW_apb_ssi Databook" (version 3.21b) provided > by Synopsys, the EEPROM read is: > > "When TMOD = 2‘b11, the transmit data is used to transmit an opcode and/or > an address to the EEPROM > device. Typically this takes three data frames (8-bit opcode followed by > 8-bit upper address and 8-bit lower > address). During the transmission of the opcode and address, no data is > captured by the receive logic (as > long as the DW_apb_ssi master is transmitting data on its txd line, data on > the rxd line is ignored). The > DW_apb_ssi master continues to transmit data until the transmit FIFO is > empty. Therefore, you should > ONLY have enough data frames in the transmit FIFO to supply the opcode and > address to the EEPROM. If > more data frames are in the transmit FIFO than are needed, then read data > is lost. > When the transmit FIFO becomes empty (all control information has been > sent), data on the receive line > (rxd) is valid and is stored in the receive FIFO; the txd output is held at > a constant logic level. The serial > transfer continues until the number of data frames received by the > DW_apb_ssi master matches the value of > the NDF field in the CTRLR1 register + 1." I tried the following combinations: 1. "Transmit only" to send opcode and address, "Receive only" to read the response 2. "Transmit and Receive" to send opcode and address, "Receive only" to read the response 3. "Transmit and Receive" to send opcode and address, "Transmit and Receive" to read the response 4. "Transmit and Receive" only to send opcode and address None of the above succeed. I only succeed when using 5. EEPROM Read to send opcode, address. I can get the correct NOR flash content from the response. Thanks, Jisheng > > Regards, > Khoa Pham > > On Tue, Jan 5, 2016 at 11:12 PM, Mark Brown wrote: > > > On Wed, Dec 23, 2015 at 08:29:52PM +0800, Jisheng Zhang wrote: > > > On Wed, 23 Dec 2015 12:15:12 +0000 Mark Brown wrote: > > > > On Wed, Dec 23, 2015 at 07:23:38PM +0800, Jisheng Zhang wrote: > > > > > > > Currently the spi-dw tmode is fixed to SPI_TMOD_TR if cs_control is > > NULL, but we > > > > > need to set it as SPI_TMOD_EPROMREAD to read nor flash, my solution > > is to add and > > > > > export one functions to set the tmode, then the nor flash driver > > call it > > > > > before reading and set back to SPI_TMOD_TR after done. > > > > > > What does this mean - what is TMOD and why do we need to set it to read > > > > NOR flash? I've no information on this controller... > > > > > TMOD is one field of DW_SPI_CTRL0. Its available value could be: > > > > > 0: Transmit and Receive > > > 1: Transmit only > > > 2: Receive only > > > 3: EEPROM Read > > > > > If the one spi nor flash is connected to the SPI host, so far I can only > > succeed > > > to read the nor flash content after setting the TMOD field as 3. > > > > Why? What does this mean in practical terms at the hardware level, what > > is "EEPROM read"? It sounds like there's some bigger issue here. > > >