Re: [PATCH 0/2] Fix few omap gpmc regressions when booted with device tree

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 



Hi Tony,

On Tue, Apr 22, 2014 at 2:54 AM, Tony Lindgren <tony@xxxxxxxxxxx> wrote:
> Hi all,
>
> Here are two fixes to GPMC issues I've seen. It seems that we have
> few more issues left to solve:
>
> 1. The remap of a device with gpmc_cs_remap seems to fail for
>    a device if it's address specified in the .dts file is
>    different from the address used by the bootloader
>
> 2. There seems to be some timing issues with smc911x where
>    rsync of larger files and apt-get dist-upgrade can produce
>    strange errors. This seems to work reliably when booted in
>    legacy mode.
>

In what board are you having this issue? The smsc911x driver supports
both SMSC's LAN911x and LAN921x families and I see that we have two
.dtsi files with different timings
(arm/boot/dts/omap-gpmc-smsc{911x,9221}.dtsi).

This is only a wild guess, but maybe your board has a smsc LAN921x
chip but is including omap-gpmc-smsc911x.dtsi on its DTS?

> 3. The DT mappings of GPMC devices are wrong for most devices
>    where the ranges property should contain the GPMC partition
>    size (16, 32, 128 or 256 MB), and the reg property for the
>    device should only contain the device IO range. So only NOR
>    should use large values for ranges and IO range, the rest
>    should use the minimum 16 MB range for GPMC partition, and
>    0x4 - 0x20000 for the IO range. And while patching these
>    it makes sense to also unify the GPMC ranges being used for
>    devices.
>
> Pekon, can you take a look at 1 and 2 above?
>
> Then for 3 above, it seems that we cannot safely change
> ranges before 1 and 2 are working reliably. Or else we have
> patch things twice, once to fix the range sizes, then to
> unify the mappings for the range address...
>
> Regards,
>
> Tony
>
> Tony Lindgren (2):
>   ARM: OMAP2+: Fix oops for GPMC free
>   ARM: OMAP2+: Fix GPMC remap for devices using an offset
>

These fixes look good to me.

Reviewed-by: Javier Martinez Canillas <javier@xxxxxxxxxxxx>

>  arch/arm/mach-omap2/gpmc.c | 15 +++++++++++++--
>  1 file changed, 13 insertions(+), 2 deletions(-)
>
> --
> 1.8.1.1
>
--
To unsubscribe from this list: send the line "unsubscribe linux-omap" in
the body of a message to majordomo@xxxxxxxxxxxxxxx
More majordomo info at  http://vger.kernel.org/majordomo-info.html




[Index of Archives]     [Linux Arm (vger)]     [ARM Kernel]     [ARM MSM]     [Linux Tegra]     [Linux WPAN Networking]     [Linux Wireless Networking]     [Maemo Users]     [Linux USB Devel]     [Video for Linux]     [Linux Audio Users]     [Yosemite Trails]     [Linux Kernel]     [Linux SCSI]

  Powered by Linux