On Tue, 2014-09-02 at 10:55 +0300, Heikki Krogerus wrote:
> It was used to provide the correct con_id for the dma
> driver, but it's not needed. Even if the driver requests a
> clock with the con_id, it still gets the correct clock. The
Maybe with -> without?
> device name is enough to match a single clock.
Since it affects DW DMA driver
Acked-by: Andy Shevchenko <andriy.shevchenko@xxxxxxxxxxxxxxx>
>
> Signed-off-by: Heikki Krogerus <heikki.krogerus@xxxxxxxxxxxxxxx>
> ---
> drivers/acpi/acpi_lpss.c | 9 +--------
> 1 file changed, 1 insertion(+), 8 deletions(-)
>
> diff --git a/drivers/acpi/acpi_lpss.c b/drivers/acpi/acpi_lpss.c
> index 4723a0e..db3498b 100644
> --- a/drivers/acpi/acpi_lpss.c
> +++ b/drivers/acpi/acpi_lpss.c
> @@ -70,7 +70,6 @@ struct lpss_shared_clock {
> struct lpss_private_data;
>
> struct lpss_device_desc {
> - const char *clkdev_name;
> unsigned int flags;
> unsigned int prv_offset;
> size_t prv_size_override;
> @@ -79,7 +78,6 @@ struct lpss_device_desc {
> };
>
> static struct lpss_device_desc lpss_dma_desc = {
> - .clkdev_name = "hclk",
> .flags = LPSS_CLK,
> };
>
> @@ -268,12 +266,7 @@ static int register_device_clock(struct acpi_device *adev,
> clk_data = platform_get_drvdata(lpss_clk_dev);
> if (!clk_data)
> return -ENODEV;
> -
> - if (dev_desc->clkdev_name) {
> - clk_register_clkdev(clk_data->clk, dev_desc->clkdev_name,
> - devname);
> - return 0;
> - }
> + clk = clk_data->clk;
>
> if (!pdata->mmio_base
> || pdata->mmio_size < dev_desc->prv_offset + LPSS_CLK_SIZE)
--
Andy Shevchenko <andriy.shevchenko@xxxxxxxxxxxxxxx>
Intel Finland Oy
--
To unsubscribe from this list: send the line "unsubscribe linux-acpi" in
the body of a message to majordomo@xxxxxxxxxxxxxxx
More majordomo info at http://vger.kernel.org/majordomo-info.html
