From mboxrd@z Thu Jan 1 00:00:00 1970 From: Marek Vasut Date: Sun, 3 Aug 2014 15:46:38 +0200 Subject: [U-Boot] [PATCH 03/18] sf: fix sf probe In-Reply-To: <1407051288-17324-4-git-send-email-nikita@compulab.co.il> References: <1407051288-17324-1-git-send-email-nikita@compulab.co.il> <1407051288-17324-4-git-send-email-nikita@compulab.co.il> Message-ID: <201408031546.39080.marex@denx.de> List-Id: MIME-Version: 1.0 Content-Type: text/plain; charset="us-ascii" Content-Transfer-Encoding: 7bit To: u-boot@lists.denx.de On Sunday, August 03, 2014 at 09:34:33 AM, Nikita Kiryanov wrote: > MXC SPI driver has a feature whereas a GPIO line can be used as a CS > signal. This is set up by joining the CS and GPIO values into a single > value using (cs | gpio << 8), and passing it off as a CS value. This > breaks the sf probe command, because it is no longer possible to invoke it > as sf probe . Instead, the user must use sf probe . > > Fix this by introducing a new board function: board_spi_cs_gpio(). > When called, board_spi_cs_gpio() will return the gpio number for the > cs value it is given. > > Cc: Jagannadha Sutradharudu Teki > Cc: Eric Nelson > Cc: Eric Benard > Cc: Fabio Estevam > Cc: Tim Harvey > Cc: Stefano Babic > Cc: Tom Rini > Signed-off-by: Nikita Kiryanov Just curious, but is this fixing generic SF code or MXC SPI driver ? I'd think the later, but it's not obvious from neither the description nor the subject. I don't quite understand the problem that you're trying to fix either, what happened, did the user command interface change ? [...] Best regards, Marek Vasut