Re: [RFC PATCH 1/3] uio: uio_pruss: port to AM33xx
From: Hans J. Koch
Date: Wed Sep 26 2012 - 19:51:19 EST
On Wed, Sep 26, 2012 at 09:44:29AM -0400, Matt Porter wrote:
> Add ifdefery hacks to only use SRAM on Davinci. This
> needs to be cleaned up with a sane generic SRAM allocator
> (like the DT based driver available that can't be used on
> Davinci which is just starting DT conversion) before it
> can go upstream.
I agree with that ;-)
Needing lots of ifdefs in *.c files is usually a bad sign...
Thanks anyway for posting, it's good to have things like that in the archives.
And please make driver code and documentation updates separate patches in the
future.
Hans
>
> Adds DT, pinctrl, and runtime PM support for use on
> AM33xx.
>
> Signed-off-by: Matt Porter <mporter@xxxxxx>
> ---
> Documentation/devicetree/bindings/uio/pruss.txt | 17 ++++++
> .../devicetree/bindings/uio/uio_pruss.txt | 17 ++++++
> drivers/uio/Kconfig | 4 +-
> drivers/uio/uio_pruss.c | 63 +++++++++++++++++++-
> 4 files changed, 98 insertions(+), 3 deletions(-)
> create mode 100644 Documentation/devicetree/bindings/uio/pruss.txt
> create mode 100644 Documentation/devicetree/bindings/uio/uio_pruss.txt
>
> diff --git a/Documentation/devicetree/bindings/uio/pruss.txt b/Documentation/devicetree/bindings/uio/pruss.txt
> new file mode 100644
> index 0000000..2ac45c5
> --- /dev/null
> +++ b/Documentation/devicetree/bindings/uio/pruss.txt
> @@ -0,0 +1,17 @@
> +TI PRUSS device
> +
> +Required properties:
> +- compatible :
> + - "ti,pruss-v1" for AM18xx/OMAP-L138/DA850
> + - "ti,pruss-v2" for AM33xx.
> +- ti,pintc-offset : Offset of the PINTC from the PRUSS address base
> +- ti,hwmods: Name of the hwmod associated to the PRUSS
> +
> +Example:
> +
> +pruss: pruss@4a300000 {
> + compatible = "ti,pruss-v2";
> + ti,hwmods = "pruss";
> + reg = <0x4a300000 0x080000>;
> + ti,pintc-offset = <0x20000>;
> +};
> diff --git a/Documentation/devicetree/bindings/uio/uio_pruss.txt b/Documentation/devicetree/bindings/uio/uio_pruss.txt
> new file mode 100644
> index 0000000..2ac45c5
> --- /dev/null
> +++ b/Documentation/devicetree/bindings/uio/uio_pruss.txt
> @@ -0,0 +1,17 @@
> +TI PRUSS device
> +
> +Required properties:
> +- compatible :
> + - "ti,pruss-v1" for AM18xx/OMAP-L138/DA850
> + - "ti,pruss-v2" for AM33xx.
> +- ti,pintc-offset : Offset of the PINTC from the PRUSS address base
> +- ti,hwmods: Name of the hwmod associated to the PRUSS
> +
> +Example:
> +
> +pruss: pruss@4a300000 {
> + compatible = "ti,pruss-v2";
> + ti,hwmods = "pruss";
> + reg = <0x4a300000 0x080000>;
> + ti,pintc-offset = <0x20000>;
> +};
> diff --git a/drivers/uio/Kconfig b/drivers/uio/Kconfig
> index 6f3ea9b..8da7d9b 100644
> --- a/drivers/uio/Kconfig
> +++ b/drivers/uio/Kconfig
> @@ -96,9 +96,9 @@ config UIO_NETX
>
> config UIO_PRUSS
> tristate "Texas Instruments PRUSS driver"
> - depends on ARCH_DAVINCI_DA850
> + depends on ARCH_DAVINCI_DA850 || SOC_AM33XX
> help
> - PRUSS driver for OMAPL138/DA850/AM18XX devices
> + PRUSS driver for OMAPL138/DA850/AM18XX and AM33XX devices
> PRUSS driver requires user space components, examples and user space
> driver is available from below SVN repo - you may use anonymous login
>
> diff --git a/drivers/uio/uio_pruss.c b/drivers/uio/uio_pruss.c
> index 33a7a27..326ce40 100644
> --- a/drivers/uio/uio_pruss.c
> +++ b/drivers/uio/uio_pruss.c
> @@ -25,7 +25,15 @@
> #include <linux/clk.h>
> #include <linux/dma-mapping.h>
> #include <linux/slab.h>
> +#include <linux/of_address.h>
> +#include <linux/of_device.h>
> +#include <linux/pinctrl/consumer.h>
> +#include <linux/err.h>
> +#include <linux/pm_runtime.h>
> +
> +#ifdef CONFIG_ARCH_DAVINCI_DA850
> #include <mach/sram.h>
> +#endif
>
> #define DRV_NAME "pruss_uio"
> #define DRV_VERSION "1.0"
> @@ -105,8 +113,10 @@ static void pruss_cleanup(struct platform_device *dev,
> dma_free_coherent(&dev->dev, extram_pool_sz, gdev->ddr_vaddr,
> gdev->ddr_paddr);
> }
> +#ifdef CONFIG_ARCH_DAVINCI_DA850
> if (gdev->sram_vaddr)
> sram_free(gdev->sram_vaddr, sram_pool_sz);
> +#endif
> kfree(gdev->info);
> clk_put(gdev->pruss_clk);
> kfree(gdev);
> @@ -117,8 +127,10 @@ static int __devinit pruss_probe(struct platform_device *dev)
> struct uio_info *p;
> struct uio_pruss_dev *gdev;
> struct resource *regs_prussio;
> + struct resource res;
> int ret = -ENODEV, cnt = 0, len;
> struct uio_pruss_pdata *pdata = dev->dev.platform_data;
> + struct pinctrl *pinctrl;
>
> gdev = kzalloc(sizeof(struct uio_pruss_dev), GFP_KERNEL);
> if (!gdev)
> @@ -129,6 +141,7 @@ static int __devinit pruss_probe(struct platform_device *dev)
> kfree(gdev);
> return -ENOMEM;
> }
> +#ifdef CONFIG_ARCH_DAVINCI_DA850
> /* Power on PRU in case its not done as part of boot-loader */
> gdev->pruss_clk = clk_get(&dev->dev, "pruss");
> if (IS_ERR(gdev->pruss_clk)) {
> @@ -140,6 +153,28 @@ static int __devinit pruss_probe(struct platform_device *dev)
> } else {
> clk_enable(gdev->pruss_clk);
> }
> +#endif
> +
> + if (dev->dev.of_node) {
> + pm_runtime_enable(&dev->dev);
> + ret = pm_runtime_get_sync(&dev->dev);
> + if (IS_ERR_VALUE(ret)) {
> + dev_err(&dev->dev, "pm_runtime_get_sync() failed\n");
> + return ret;
> + }
> +
> + ret = of_address_to_resource(dev->dev.of_node, 0, &res);
> + if (IS_ERR_VALUE(ret)) {
> + dev_err(&dev->dev, "failed to parse DT reg\n");
> + return ret;
> + }
> + regs_prussio = &res;
> + }
> +
> + pinctrl = devm_pinctrl_get_select_default(&dev->dev);
> + if (IS_ERR(pinctrl))
> + dev_warn(&dev->dev,
> + "pins are not configured from the driver\n");
>
> regs_prussio = platform_get_resource(dev, IORESOURCE_MEM, 0);
> if (!regs_prussio) {
> @@ -152,11 +187,13 @@ static int __devinit pruss_probe(struct platform_device *dev)
> goto out_free;
> }
>
> +#ifdef CONFIG_ARCH_DAVINCI_DA850
> gdev->sram_vaddr = sram_alloc(sram_pool_sz, &(gdev->sram_paddr));
> if (!gdev->sram_vaddr) {
> dev_err(&dev->dev, "Could not allocate SRAM pool\n");
> goto out_free;
> }
> +#endif
>
> gdev->ddr_vaddr = dma_alloc_coherent(&dev->dev, extram_pool_sz,
> &(gdev->ddr_paddr), GFP_KERNEL | GFP_DMA);
> @@ -172,7 +209,17 @@ static int __devinit pruss_probe(struct platform_device *dev)
> goto out_free;
> }
>
> - gdev->pintc_base = pdata->pintc_base;
> + if (dev->dev.of_node) {
> + ret = of_property_read_u32(dev->dev.of_node,
> + "ti,pintc-offset",
> + &gdev->pintc_base);
> + if (ret < 0) {
> + dev_err(&dev->dev,
> + "Can't parse ti,pintc-offset property\n");
> + goto out_free;
> + }
> + } else
> + gdev->pintc_base = pdata->pintc_base;
> gdev->hostirq_start = platform_get_irq(dev, 0);
>
> for (cnt = 0, p = gdev->info; cnt < MAX_PRUSS_EVT; cnt++, p++) {
> @@ -180,6 +227,7 @@ static int __devinit pruss_probe(struct platform_device *dev)
> p->mem[0].size = resource_size(regs_prussio);
> p->mem[0].memtype = UIO_MEM_PHYS;
>
> +#ifdef CONFIG_ARCH_DAVINCI_DA850
> p->mem[1].addr = gdev->sram_paddr;
> p->mem[1].size = sram_pool_sz;
> p->mem[1].memtype = UIO_MEM_PHYS;
> @@ -187,6 +235,11 @@ static int __devinit pruss_probe(struct platform_device *dev)
> p->mem[2].addr = gdev->ddr_paddr;
> p->mem[2].size = extram_pool_sz;
> p->mem[2].memtype = UIO_MEM_PHYS;
> +#else
> + p->mem[1].addr = gdev->ddr_paddr;
> + p->mem[1].size = extram_pool_sz;
> + p->mem[1].memtype = UIO_MEM_PHYS;
> +#endif
>
> p->name = kasprintf(GFP_KERNEL, "pruss_evt%d", cnt);
> p->version = DRV_VERSION;
> @@ -218,12 +271,20 @@ static int __devexit pruss_remove(struct platform_device *dev)
> return 0;
> }
>
> +static const struct of_device_id pruss_dt_ids[] = {
> + { .compatible = "ti,pruss-v1", .data = NULL, },
> + { .compatible = "ti,pruss-v2", .data = NULL, },
> + {},
> +};
> +MODULE_DEVICE_TABLE(of, pruss_dt_ids);
> +
> static struct platform_driver pruss_driver = {
> .probe = pruss_probe,
> .remove = __devexit_p(pruss_remove),
> .driver = {
> .name = DRV_NAME,
> .owner = THIS_MODULE,
> + .of_match_table = pruss_dt_ids,
> },
> };
>
> --
> 1.7.9.5
>
>
--
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/