On 04/09/2013 07:11 AM, Javier Martinez Canillas wrote:
> The GPMC timing properties for device-tree have been updated
> by adding a "-ns" or "-ps" suffix to indicate the units of
> time the property represents. Therefore, update the timing
> property names for TI GPMC ethernet binding.
> 
> Signed-off-by: Javier Martinez Canillas <[email protected]>
> ---
> 
> Jon, Benoit:
> 
> Sorry that I didn't send this patch before but I just realized
> that the GPMC timing properties changed after
> 
> commit 5330dc16 ("ARM: OMAP2+: Add GPMC DT support for Ethernet child nodes")
> 
> got queued.

Thanks, I missed this one too. Looks like I need to update the gpmc-nand
documentation as well :-(

> Tony,
> 
> Is still possible to queue this patch on your omap-for-v3.10/gpmc branch
> or it is too late?

If it is I think that this could be queued as a fix. Tony?

> Thanks a lot and best regards,
> Javier
> 
>  Documentation/devicetree/bindings/net/gpmc-eth.txt |   56 
> ++++++++++----------
>  1 files changed, 28 insertions(+), 28 deletions(-)
> 
> diff --git a/Documentation/devicetree/bindings/net/gpmc-eth.txt 
> b/Documentation/devicetree/bindings/net/gpmc-eth.txt
> index 24cb4e4..ace4a64 100644
> --- a/Documentation/devicetree/bindings/net/gpmc-eth.txt
> +++ b/Documentation/devicetree/bindings/net/gpmc-eth.txt
> @@ -26,16 +26,16 @@ Required properties:
>  - bank-width:                Address width of the device in bytes. GPMC 
> supports 8-bit
>                       and 16-bit devices and so must be either 1 or 2 bytes.
>  - compatible:                Compatible string property for the ethernet 
> child device.
> -- gpmc,cs-on:                Chip-select assertion time
> -- gpmc,cs-rd-off:    Chip-select de-assertion time for reads
> -- gpmc,cs-wr-off:    Chip-select de-assertion time for writes
> -- gpmc,oe-on:                Output-enable assertion time
> -- gpmc,oe-off                Output-enable de-assertion time
> -- gpmc,we-on:                Write-enable assertion time
> -- gpmc,we-off:               Write-enable de-assertion time
> -- gpmc,access:               Start cycle to first data capture (read access)
> -- gpmc,rd-cycle:     Total read cycle time
> -- gpmc,wr-cycle:     Total write cycle time
> +- gpmc,cs-on-ns:     Chip-select assertion time
> +- gpmc,cs-rd-off-ns: Chip-select de-assertion time for reads
> +- gpmc,cs-wr-off-ns: Chip-select de-assertion time for writes
> +- gpmc,oe-on-ns:     Output-enable assertion time
> +- gpmc,oe-off-ns:    Output-enable de-assertion time
> +- gpmc,we-on-ns:     Write-enable assertion time
> +- gpmc,we-off-ns:    Write-enable de-assertion time
> +- gpmc,access-ns:    Start cycle to first data capture (read access)
> +- gpmc,rd-cycle-ns:  Total read cycle time
> +- gpmc,wr-cycle-ns:  Total write cycle time
>  - reg:                       Chip-select, base address (relative to 
> chip-select)
>                       and size of the memory mapped for the device.
>                       Note that base address will be typically 0 as this
> @@ -65,24 +65,24 @@ gpmc: gpmc@6e000000 {
>               bank-width = <2>;
>  
>               gpmc,mux-add-data;
> -             gpmc,cs-on = <0>;
> -             gpmc,cs-rd-off = <186>;
> -             gpmc,cs-wr-off = <186>;
> -             gpmc,adv-on = <12>;
> -             gpmc,adv-rd-off = <48>;
> -             gpmc,adv-wr-off = <48>;
> -             gpmc,oe-on = <54>;
> -             gpmc,oe-off = <168>;
> -             gpmc,we-on = <54>;
> -             gpmc,we-off = <168>;
> -             gpmc,rd-cycle = <186>;
> -             gpmc,wr-cycle = <186>;
> -             gpmc,access = <114>;
> -             gpmc,page-burst-access = <6>;
> -             gpmc,bus-turnaround = <12>;
> -             gpmc,cycle2cycle-delay = <18>;
> -             gpmc,wr-data-mux-bus = <90>;
> -             gpmc,wr-access = <186>;
> +             gpmc,cs-on-ns = <0>;
> +             gpmc,cs-rd-off-ns = <186>;
> +             gpmc,cs-wr-off-ns = <186>;
> +             gpmc,adv-on-ns = <12>;
> +             gpmc,adv-rd-off-ns = <48>;
> +             gpmc,adv-wr-off-ns = <48>;
> +             gpmc,oe-on-ns = <54>;
> +             gpmc,oe-off-ns = <168>;
> +             gpmc,we-on-ns = <54>;
> +             gpmc,we-off-ns = <168>;
> +             gpmc,rd-cycle-ns = <186>;
> +             gpmc,wr-cycle-ns = <186>;
> +             gpmc,access-ns = <114>;
> +             gpmc,page-burst-access-ns = <6>;
> +             gpmc,bus-turnaround-ns = <12>;
> +             gpmc,cycle2cycle-delay-ns = <18>;
> +             gpmc,wr-data-mux-bus-ns = <90>;
> +             gpmc,wr-access-ns = <186>;
>               gpmc,cycle2cycle-samecsen;
>               gpmc,cycle2cycle-diffcsen;

Acked-by: Jon Hunter <[email protected]>

Cheers
Jon
--
To unsubscribe from this list: send the line "unsubscribe linux-omap" in
the body of a message to [email protected]
More majordomo info at  http://vger.kernel.org/majordomo-info.html

Reply via email to