[media] media: davinci: vpss: enable vpss clocks
authorLad, Prabhakar <prabhakar.csengg@gmail.com>
Fri, 22 Mar 2013 07:53:12 +0000 (04:53 -0300)
committerMauro Carvalho Chehab <mchehab@redhat.com>
Sun, 14 Apr 2013 23:08:09 +0000 (20:08 -0300)
By default the VPSS clocks were enabled in capture driver
for davinci family which creates duplicates for dm355/dm365/dm644x.
This patch adds support to enable the VPSS clocks in VPSS driver,
which avoids duplication of code and also adding clock aliases.
This patch uses PM runtime API to enable/disable clock, instead
of DaVinci clock framework. con_ids for master and slave clocks of
vpss is added in pm_domain.
This patch cleanups the VPSS clock enabling in the capture driver,
and also removes the clock alias in machine file. Along side adds
a vpss slave clock for DM365 as mentioned by Sekhar
(https://patchwork.kernel.org/patch/1221261/).
The Suspend/Resume in dm644x_ccdc.c which enabled/disabled the VPSS clock
is now implemented as part of the VPSS driver.

Signed-off-by: Lad, Prabhakar <prabhakar.csengg@gmail.com>
Acked-by: Sekhar Nori <nsekhar@ti.com>
Signed-off-by: Hans Verkuil <hans.verkuil@cisco.com>
Signed-off-by: Mauro Carvalho Chehab <mchehab@redhat.com>
arch/arm/mach-davinci/dm355.c
arch/arm/mach-davinci/dm365.c
arch/arm/mach-davinci/dm644x.c
arch/arm/mach-davinci/pm_domain.c
drivers/media/platform/davinci/dm355_ccdc.c
drivers/media/platform/davinci/dm644x_ccdc.c
drivers/media/platform/davinci/isif.c
drivers/media/platform/davinci/vpss.c

index b49c3b77d55e5ac3e9884762a2fcdab28c03453b..8e98bb06b1c1e4056f9daba6cbc418a0f54be606 100644 (file)
@@ -345,8 +345,8 @@ static struct clk_lookup dm355_clks[] = {
        CLK(NULL, "pll1_aux", &pll1_aux_clk),
        CLK(NULL, "pll1_sysclkbp", &pll1_sysclkbp),
        CLK(NULL, "vpss_dac", &vpss_dac_clk),
-       CLK(NULL, "vpss_master", &vpss_master_clk),
-       CLK(NULL, "vpss_slave", &vpss_slave_clk),
+       CLK("vpss", "master", &vpss_master_clk),
+       CLK("vpss", "slave", &vpss_slave_clk),
        CLK(NULL, "clkout1", &clkout1_clk),
        CLK(NULL, "clkout2", &clkout2_clk),
        CLK(NULL, "pll2", &pll2_clk),
@@ -873,9 +873,6 @@ static int __init dm355_init_devices(void)
        if (!cpu_is_davinci_dm355())
                return 0;
 
-       /* Add ccdc clock aliases */
-       clk_add_alias("master", dm355_ccdc_dev.name, "vpss_master", NULL);
-       clk_add_alias("slave", dm355_ccdc_dev.name, "vpss_master", NULL);
        davinci_cfg_reg(DM355_INT_EDMA_CC);
        platform_device_register(&dm355_edma_device);
        platform_device_register(&dm355_vpss_device);
index 6c3980540be0525a9e3d7a5c699ed2221d378a35..c61dd94daadcbee1483f454b8b0ae6e67c82ee97 100644 (file)
@@ -257,6 +257,12 @@ static struct clk vpss_master_clk = {
        .flags          = CLK_PSC,
 };
 
+static struct clk vpss_slave_clk = {
+       .name           = "vpss_slave",
+       .parent         = &pll1_sysclk5,
+       .lpsc           = DAVINCI_LPSC_VPSSSLV,
+};
+
 static struct clk arm_clk = {
        .name           = "arm_clk",
        .parent         = &pll2_sysclk2,
@@ -449,7 +455,8 @@ static struct clk_lookup dm365_clks[] = {
        CLK(NULL, "pll2_sysclk8", &pll2_sysclk8),
        CLK(NULL, "pll2_sysclk9", &pll2_sysclk9),
        CLK(NULL, "vpss_dac", &vpss_dac_clk),
-       CLK(NULL, "vpss_master", &vpss_master_clk),
+       CLK("vpss", "master", &vpss_master_clk),
+       CLK("vpss", "slave", &vpss_slave_clk),
        CLK(NULL, "arm", &arm_clk),
        CLK(NULL, "uart0", &uart0_clk),
        CLK(NULL, "uart1", &uart1_clk),
@@ -1239,8 +1246,6 @@ static int __init dm365_init_devices(void)
        clk_add_alias(NULL, dev_name(&dm365_mdio_device.dev),
                      NULL, &dm365_emac_device.dev);
 
-       /* Add isif clock alias */
-       clk_add_alias("master", dm365_isif_dev.name, "vpss_master", NULL);
        platform_device_register(&dm365_vpss_device);
        platform_device_register(&dm365_isif_dev);
        platform_device_register(&vpfe_capture_dev);
index ee0e994748e0b65a0c3c40493025cd858584ff7d..c2a9273330bfbd221752fb0857a2445025c13035 100644 (file)
@@ -300,8 +300,8 @@ static struct clk_lookup dm644x_clks[] = {
        CLK(NULL, "dsp", &dsp_clk),
        CLK(NULL, "arm", &arm_clk),
        CLK(NULL, "vicp", &vicp_clk),
-       CLK(NULL, "vpss_master", &vpss_master_clk),
-       CLK(NULL, "vpss_slave", &vpss_slave_clk),
+       CLK("vpss", "master", &vpss_master_clk),
+       CLK("vpss", "slave", &vpss_slave_clk),
        CLK(NULL, "arm", &arm_clk),
        CLK(NULL, "uart0", &uart0_clk),
        CLK(NULL, "uart1", &uart1_clk),
@@ -901,11 +901,6 @@ int __init dm644x_init_video(struct vpfe_config *vpfe_cfg,
                dm644x_vpfe_dev.dev.platform_data = vpfe_cfg;
                platform_device_register(&dm644x_ccdc_dev);
                platform_device_register(&dm644x_vpfe_dev);
-               /* Add ccdc clock aliases */
-               clk_add_alias("master", dm644x_ccdc_dev.name,
-                             "vpss_master", NULL);
-               clk_add_alias("slave", dm644x_ccdc_dev.name,
-                             "vpss_slave", NULL);
        }
 
        if (vpbe_cfg) {
index c90250e3bef8068f0333c2387fb9ae96c84cd7a3..6b98413cebd61179f100b2e77ac7237f97e6c102 100644 (file)
@@ -53,7 +53,7 @@ static struct dev_pm_domain davinci_pm_domain = {
 
 static struct pm_clk_notifier_block platform_bus_notifier = {
        .pm_domain = &davinci_pm_domain,
-       .con_ids = { "fck", NULL, },
+       .con_ids = { "fck", "master", "slave", NULL },
 };
 
 static int __init davinci_pm_runtime_init(void)
index 2364dbab804678b42ff7b1768d17a56136a9d3d5..05f8fb7f7b70a335c89eade3925fca309a79ab5d 100644 (file)
@@ -37,7 +37,6 @@
 #include <linux/platform_device.h>
 #include <linux/uaccess.h>
 #include <linux/videodev2.h>
-#include <linux/clk.h>
 #include <linux/err.h>
 #include <linux/module.h>
 
@@ -59,10 +58,6 @@ static struct ccdc_oper_config {
        struct ccdc_params_raw bayer;
        /* YCbCr configuration */
        struct ccdc_params_ycbcr ycbcr;
-       /* Master clock */
-       struct clk *mclk;
-       /* slave clock */
-       struct clk *sclk;
        /* ccdc base address */
        void __iomem *base_addr;
 } ccdc_cfg = {
@@ -997,32 +992,10 @@ static int dm355_ccdc_probe(struct platform_device *pdev)
                goto fail_nomem;
        }
 
-       /* Get and enable Master clock */
-       ccdc_cfg.mclk = clk_get(&pdev->dev, "master");
-       if (IS_ERR(ccdc_cfg.mclk)) {
-               status = PTR_ERR(ccdc_cfg.mclk);
-               goto fail_nomap;
-       }
-       if (clk_prepare_enable(ccdc_cfg.mclk)) {
-               status = -ENODEV;
-               goto fail_mclk;
-       }
-
-       /* Get and enable Slave clock */
-       ccdc_cfg.sclk = clk_get(&pdev->dev, "slave");
-       if (IS_ERR(ccdc_cfg.sclk)) {
-               status = PTR_ERR(ccdc_cfg.sclk);
-               goto fail_mclk;
-       }
-       if (clk_prepare_enable(ccdc_cfg.sclk)) {
-               status = -ENODEV;
-               goto fail_sclk;
-       }
-
        /* Platform data holds setup_pinmux function ptr */
        if (NULL == pdev->dev.platform_data) {
                status = -ENODEV;
-               goto fail_sclk;
+               goto fail_nomap;
        }
        setup_pinmux = pdev->dev.platform_data;
        /*
@@ -1033,12 +1006,6 @@ static int dm355_ccdc_probe(struct platform_device *pdev)
        ccdc_cfg.dev = &pdev->dev;
        printk(KERN_NOTICE "%s is registered with vpfe.\n", ccdc_hw_dev.name);
        return 0;
-fail_sclk:
-       clk_disable_unprepare(ccdc_cfg.sclk);
-       clk_put(ccdc_cfg.sclk);
-fail_mclk:
-       clk_disable_unprepare(ccdc_cfg.mclk);
-       clk_put(ccdc_cfg.mclk);
 fail_nomap:
        iounmap(ccdc_cfg.base_addr);
 fail_nomem:
@@ -1052,10 +1019,6 @@ static int dm355_ccdc_remove(struct platform_device *pdev)
 {
        struct resource *res;
 
-       clk_disable_unprepare(ccdc_cfg.sclk);
-       clk_disable_unprepare(ccdc_cfg.mclk);
-       clk_put(ccdc_cfg.mclk);
-       clk_put(ccdc_cfg.sclk);
        iounmap(ccdc_cfg.base_addr);
        res = platform_get_resource(pdev, IORESOURCE_MEM, 0);
        if (res)
index 971d639b6674cccf37748fd366c563f363006be7..30fa08405d6111e07ccadd4488e077b49664eec5 100644 (file)
@@ -38,7 +38,6 @@
 #include <linux/uaccess.h>
 #include <linux/videodev2.h>
 #include <linux/gfp.h>
-#include <linux/clk.h>
 #include <linux/err.h>
 #include <linux/module.h>
 
@@ -60,10 +59,6 @@ static struct ccdc_oper_config {
        struct ccdc_params_raw bayer;
        /* YCbCr configuration */
        struct ccdc_params_ycbcr ycbcr;
-       /* Master clock */
-       struct clk *mclk;
-       /* slave clock */
-       struct clk *sclk;
        /* ccdc base address */
        void __iomem *base_addr;
 } ccdc_cfg = {
@@ -991,38 +986,9 @@ static int dm644x_ccdc_probe(struct platform_device *pdev)
                goto fail_nomem;
        }
 
-       /* Get and enable Master clock */
-       ccdc_cfg.mclk = clk_get(&pdev->dev, "master");
-       if (IS_ERR(ccdc_cfg.mclk)) {
-               status = PTR_ERR(ccdc_cfg.mclk);
-               goto fail_nomap;
-       }
-       if (clk_prepare_enable(ccdc_cfg.mclk)) {
-               status = -ENODEV;
-               goto fail_mclk;
-       }
-
-       /* Get and enable Slave clock */
-       ccdc_cfg.sclk = clk_get(&pdev->dev, "slave");
-       if (IS_ERR(ccdc_cfg.sclk)) {
-               status = PTR_ERR(ccdc_cfg.sclk);
-               goto fail_mclk;
-       }
-       if (clk_prepare_enable(ccdc_cfg.sclk)) {
-               status = -ENODEV;
-               goto fail_sclk;
-       }
        ccdc_cfg.dev = &pdev->dev;
        printk(KERN_NOTICE "%s is registered with vpfe.\n", ccdc_hw_dev.name);
        return 0;
-fail_sclk:
-       clk_disable_unprepare(ccdc_cfg.sclk);
-       clk_put(ccdc_cfg.sclk);
-fail_mclk:
-       clk_disable_unprepare(ccdc_cfg.mclk);
-       clk_put(ccdc_cfg.mclk);
-fail_nomap:
-       iounmap(ccdc_cfg.base_addr);
 fail_nomem:
        release_mem_region(res->start, resource_size(res));
 fail_nores:
@@ -1034,10 +1000,6 @@ static int dm644x_ccdc_remove(struct platform_device *pdev)
 {
        struct resource *res;
 
-       clk_disable_unprepare(ccdc_cfg.mclk);
-       clk_disable_unprepare(ccdc_cfg.sclk);
-       clk_put(ccdc_cfg.mclk);
-       clk_put(ccdc_cfg.sclk);
        iounmap(ccdc_cfg.base_addr);
        res = platform_get_resource(pdev, IORESOURCE_MEM, 0);
        if (res)
@@ -1052,18 +1014,12 @@ static int dm644x_ccdc_suspend(struct device *dev)
        ccdc_save_context();
        /* Disable CCDC */
        ccdc_enable(0);
-       /* Disable both master and slave clock */
-       clk_disable_unprepare(ccdc_cfg.mclk);
-       clk_disable_unprepare(ccdc_cfg.sclk);
 
        return 0;
 }
 
 static int dm644x_ccdc_resume(struct device *dev)
 {
-       /* Enable both master and slave clock */
-       clk_prepare_enable(ccdc_cfg.mclk);
-       clk_prepare_enable(ccdc_cfg.sclk);
        /* Restore CCDC context */
        ccdc_restore_context();
 
index abc3ae36645c59da16a36c2b7a127ab68d555c9c..3332cca632e53e37c22d8684593f2c4624ea31b4 100644 (file)
@@ -32,7 +32,6 @@
 #include <linux/uaccess.h>
 #include <linux/io.h>
 #include <linux/videodev2.h>
-#include <linux/clk.h>
 #include <linux/err.h>
 #include <linux/module.h>
 
@@ -88,8 +87,6 @@ static struct isif_oper_config {
        struct isif_ycbcr_config ycbcr;
        struct isif_params_raw bayer;
        enum isif_data_pack data_pack;
-       /* Master clock */
-       struct clk *mclk;
        /* ISIF base address */
        void __iomem *base_addr;
        /* ISIF Linear Table 0 */
@@ -1039,6 +1036,10 @@ static int isif_probe(struct platform_device *pdev)
        void *__iomem addr;
        int status = 0, i;
 
+       /* Platform data holds setup_pinmux function ptr */
+       if (!pdev->dev.platform_data)
+               return -ENODEV;
+
        /*
         * first try to register with vpfe. If not correct platform, then we
         * don't have to iomap
@@ -1047,22 +1048,6 @@ static int isif_probe(struct platform_device *pdev)
        if (status < 0)
                return status;
 
-       /* Get and enable Master clock */
-       isif_cfg.mclk = clk_get(&pdev->dev, "master");
-       if (IS_ERR(isif_cfg.mclk)) {
-               status = PTR_ERR(isif_cfg.mclk);
-               goto fail_mclk;
-       }
-       if (clk_prepare_enable(isif_cfg.mclk)) {
-               status = -ENODEV;
-               goto fail_mclk;
-       }
-
-       /* Platform data holds setup_pinmux function ptr */
-       if (NULL == pdev->dev.platform_data) {
-               status = -ENODEV;
-               goto fail_mclk;
-       }
        setup_pinmux = pdev->dev.platform_data;
        /*
         * setup Mux configuration for ccdc which may be different for
@@ -1124,9 +1109,6 @@ fail_nobase_res:
                release_mem_region(res->start, resource_size(res));
                i--;
        }
-fail_mclk:
-       clk_disable_unprepare(isif_cfg.mclk);
-       clk_put(isif_cfg.mclk);
        vpfe_unregister_ccdc_device(&isif_hw_dev);
        return status;
 }
@@ -1146,8 +1128,6 @@ static int isif_remove(struct platform_device *pdev)
                i++;
        }
        vpfe_unregister_ccdc_device(&isif_hw_dev);
-       clk_disable_unprepare(isif_cfg.mclk);
-       clk_put(isif_cfg.mclk);
        return 0;
 }
 
index a19c552232d1548afd4c142e19f0b040f0356b61..d36429d1bd69709693fdfa1be65e810502ff0c3c 100644 (file)
@@ -25,6 +25,8 @@
 #include <linux/spinlock.h>
 #include <linux/compiler.h>
 #include <linux/io.h>
+#include <linux/pm_runtime.h>
+
 #include <media/davinci/vpss.h>
 
 MODULE_LICENSE("GPL");
@@ -490,6 +492,10 @@ static int vpss_probe(struct platform_device *pdev)
        } else
                oper_cfg.hw_ops.clear_wbl_overflow = dm644x_clear_wbl_overflow;
 
+       pm_runtime_enable(&pdev->dev);
+
+       pm_runtime_get(&pdev->dev);
+
        spin_lock_init(&oper_cfg.vpss_lock);
        dev_info(&pdev->dev, "%s vpss probe success\n", platform_name);
        return 0;
@@ -507,6 +513,7 @@ static int vpss_remove(struct platform_device *pdev)
 {
        struct resource         *res;
 
+       pm_runtime_disable(&pdev->dev);
        iounmap(oper_cfg.vpss_regs_base0);
        res = platform_get_resource(pdev, IORESOURCE_MEM, 0);
        release_mem_region(res->start, resource_size(res));
@@ -518,10 +525,28 @@ static int vpss_remove(struct platform_device *pdev)
        return 0;
 }
 
+static int vpss_suspend(struct device *dev)
+{
+       pm_runtime_put(dev);
+       return 0;
+}
+
+static int vpss_resume(struct device *dev)
+{
+       pm_runtime_get(dev);
+       return 0;
+}
+
+static const struct dev_pm_ops vpss_pm_ops = {
+       .suspend = vpss_suspend,
+       .resume = vpss_resume,
+};
+
 static struct platform_driver vpss_driver = {
        .driver = {
                .name   = "vpss",
                .owner = THIS_MODULE,
+               .pm = &vpss_pm_ops,
        },
        .remove = vpss_remove,
        .probe = vpss_probe,