[PATCH 01/10] ARM: dts: Fix bootloader version dependencies by muxing n900 smc91x pins
rogerq at ti.com
Mon Nov 3 01:30:27 PST 2014
On 10/30/2014 05:49 PM, Tony Lindgren wrote:
> * Roger Quadros <rogerq at ti.com> [141030 05:01]:
>> On 10/30/2014 02:28 AM, Tony Lindgren wrote:
>>> + /*
>>> + * gpmc_ncs0, gpmc_nadv_ale, gpmc_noe, gpmc_nwe, gpmc_wait0 not muxable
>>> + * according to TRM. REVISIT: why does nolo set input for gpmc_clk?
>>> + */
>>> + OMAP3_CORE1_IOPAD(0x20b0, PIN_OUTPUT | MUX_MODE0) /* gpmc_ncs1.gpmc_ncs1 */
>>> + OMAP3_CORE1_IOPAD(0x20be, PIN_INPUT | MUX_MODE0) /* gpmc_clk.gpmc_clk */
>> Right about the comment. GPMC_CLK should feed the CLK input of the OneNAND.
>> This needs to be an OUTPUT pin.
>> Does OneNAND work when this pin is configured as output?
> Does not seem to work, it produces onenand_wait: ECC error = 0xffff.
> It seems the clock needs to be copied to GPMC too in some cases.
> For MMC, there's the MMCSDIO2ADPCLKISEL option to copy the clock to
> account for level shifter latencies . But in the OneNAND case I
> don't think there are any level shifters, and I don't think we have
> "copy clock" option for GPMC either in SCM so it somehow is automatic
> in GPMC.
> Anyways, updated patch below with wrong guessing removed.
>  http://processors.wiki.ti.com/index.php/SD-MMC_Usage_Notes_on_OMAP35x_and_AM37x
> 8< --------------------
> From: Tony Lindgren <tony at atomide.com>
> Date: Wed, 29 Oct 2014 17:16:47 -0700
> Subject: [PATCH] ARM: dts: Fix bootloader version dependencies by muxing n900 smc91x pins
> Apparently some versions of nolo don't mux the all the necessary GPMC
> pins for the smc91x probe to work properly. Let's fix this issue
> by adding mux support for GPMC to the kernel.
> Note that GPMC clk needs input enabled for OnenNAND to work.
> Cc: Kevin Hilman <khilman at kernel.org>
> Cc: Roger Quadros <rogerq at ti.com>
> Signed-off-by: Tony Lindgren <tony at atomide.com>
Acked-by: Roger Quadros <rogerq at ti.com>
More information about the linux-arm-kernel