On Thu, Apr 24, 2014 at 11:11 PM, Ezequiel Garcia
<ezequiel@xxxxxxxxxxxxxxxxxxxx> wrote:
> The DMA controller is needed for the USB controller to be correctly
> registered. Therefore, if the DMA node is located at the end an unecessary
> probe deferral is produced systematically.
>
> This is easily fixed by moving the node at the beggining of the child list,
> so it's probed first.
>
> Signed-off-by: Ezequiel Garcia <ezequiel@xxxxxxxxxxxxxxxxxxxx>
Tested-by: Yegor Yefremov <yegorslists@xxxxxxxxxxxxxx>
> --
> v1->v2:
> * Added a comment to prevent a future clean-up based on the memory offset.
>
> arch/arm/boot/dts/am33xx.dtsi | 34 ++++++++++++++++++++--------------
> 1 file changed, 20 insertions(+), 14 deletions(-)
>
> diff --git a/arch/arm/boot/dts/am33xx.dtsi b/arch/arm/boot/dts/am33xx.dtsi
> index 9770e35..02e1eb6 100644
> --- a/arch/arm/boot/dts/am33xx.dtsi
> +++ b/arch/arm/boot/dts/am33xx.dtsi
> @@ -453,6 +453,26 @@
> ti,hwmods = "usb_otg_hs";
> status = "disabled";
>
> + /*
> + * The probe order matches the child ordering so the
> + * dma-controller node must be the first one to prevent
> + * spurious probe deferrals.
> + */
> + cppi41dma: dma-controller@47402000 {
> + compatible = "ti,am3359-cppi41";
> + reg = <0x47400000 0x1000
> + 0x47402000 0x1000
> + 0x47403000 0x1000
> + 0x47404000 0x4000>;
> + reg-names = "glue", "controller", "scheduler", "queuemgr";
> + interrupts = <17>;
> + interrupt-names = "glue";
> + #dma-cells = <2>;
> + #dma-channels = <30>;
> + #dma-requests = <256>;
> + status = "disabled";
> + };
> +
> usb_ctrl_mod: control@44e10620 {
> compatible = "ti,am335x-usb-ctrl-module";
> reg = <0x44e10620 0x10
> @@ -556,20 +576,6 @@
> "tx14", "tx15";
> };
>
> - cppi41dma: dma-controller@47402000 {
> - compatible = "ti,am3359-cppi41";
> - reg = <0x47400000 0x1000
> - 0x47402000 0x1000
> - 0x47403000 0x1000
> - 0x47404000 0x4000>;
> - reg-names = "glue", "controller", "scheduler", "queuemgr";
> - interrupts = <17>;
> - interrupt-names = "glue";
> - #dma-cells = <2>;
> - #dma-channels = <30>;
> - #dma-requests = <256>;
> - status = "disabled";
> - };
> };
>
> epwmss0: epwmss@48300000 {
> --
> 1.9.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
--
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
