Re: [PATCH] dmaengine: pl300: enable the clock to PL330 dma

From: Krzysztof Kozlowski
Date: Mon May 04 2015 - 01:31:12 EST

Next message: linux cbon: "how to have the kernel do udev's job and autoload the right modules ?"
Previous message: linux cbon: "is kernel slower at loading for some hardware ?"
In reply to: dinguyen: "[PATCH] dmaengine: pl300: enable the clock to PL330 dma"
Next in thread: Dinh Nguyen: "Re: [PATCH] dmaengine: pl300: enable the clock to PL330 dma"
Messages sorted by: [ date ] [ thread ] [ subject ] [ author ]

2015-05-04 13:28 GMT+09:00 <dinguyen@xxxxxxxxxxxxxxxxxxxxx>:
> From: Dinh Nguyen <dinguyen@xxxxxxxxxxxxxxxxxxxxx>
>
> Turn on the clock to the PL330 DMA if there is a clock node provided.

Why? There is no explanation in the patch for this important question - why?

Amba bus already does this and provide a wrapper function.
Additionally that would mess up with runtime PM and clock
enable/disable.

Best regards,
Krzysztof
--
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majordomo@xxxxxxxxxxxxxxx
More majordomo info at http://vger.kernel.org/majordomo-info.html
Please read the FAQ at http://www.tux.org/lkml/

Next message: linux cbon: "how to have the kernel do udev's job and autoload the right modules ?"
Previous message: linux cbon: "is kernel slower at loading for some hardware ?"
In reply to: dinguyen: "[PATCH] dmaengine: pl300: enable the clock to PL330 dma"
Next in thread: Dinh Nguyen: "Re: [PATCH] dmaengine: pl300: enable the clock to PL330 dma"
Messages sorted by: [ date ] [ thread ] [ subject ] [ author ]