Hi Masahiro,

On 22 December 2015 at 03:04, Masahiro Yamada
<yamada.masah...@socionext.com> wrote:
> The most basic thing for clock is to enable it, but it is missing
> in this uclass.
>
> Signed-off-by: Masahiro Yamada <yamada.masah...@socionext.com>
> ---
>
>  include/clk.h | 9 +++++++++
>  1 file changed, 9 insertions(+)

Acked-by: Simon Glass <s...@chromium.org>

Thinking ahead, should we have disable() also, or maybe replacing both
with set_enable(bool enable) would be better?

>
> diff --git a/include/clk.h b/include/clk.h
> index 518cb47..ce2db41 100644
> --- a/include/clk.h
> +++ b/include/clk.h
> @@ -33,6 +33,15 @@ struct clk_ops {
>         ulong (*set_rate)(struct udevice *dev, ulong rate);
>
>         /**
> +        * enable() - Enable the clock for a peripheral
> +        *
> +        * @dev:        clock provider
> +        * @periph:     Peripheral ID to enable
> +        * @return zero on success, or -ve error code
> +        */
> +       int (*enable)(struct udevice *dev, int periph);
> +
> +       /**
>          * get_periph_rate() - Get clock rate for a peripheral
>          *
>          * @dev:        Device to check (UCLASS_CLK)
> --
> 1.9.1
>

Regards,
Simon
_______________________________________________
U-Boot mailing list
U-Boot@lists.denx.de
http://lists.denx.de/mailman/listinfo/u-boot

Reply via email to