On Tue, Dec 09, 2014 at 08:21:30PM +0000, Lee Jones wrote:
> This patch adds support for the SPI portion of ST's SSC device.

I've applied both of these, though I had to rename the DT binding
document and...

> +static int spi_st_clk_enable(struct spi_st *spi_st)
> +{
> +     /*
> +      * Current platforms use one of the core clocks for SPI and I2C.
> +      * If we attempt to disable the clock, the system will hang.
> +      *
> +      * TODO: Remove this when platform supports power domains.
> +      */
> +     return 0;
> +
> +     return clk_prepare_enable(spi_st->clk);
> +}

...this is not good at all and a step back from the previous version -
if there are problems with the SoC integration they need to be fixed
there or possibly in the clock driver for the SoC rather than bodging
individual drivers.

Attachment: signature.asc
Description: Digital signature

Reply via email to