| lists.openwall.net | lists / announce owl-users owl-dev john-users john-dev passwdqc-users yescrypt popa3d-users / oss-security kernel-hardening musl sabotage tlsify passwords / crypt-dev xvendor / Bugtraq Full-Disclosure linux-kernel linux-netdev linux-ext4 linux-hardening linux-cve-announce PHC | |
|
Open Source and information security mailing list archives
| ||
|
Date: Fri, 28 Sep 2012 14:53:34 -0400
From: Matt Porter <mporter@...com>
To: "Hans J. Koch" <hjk@...sjkoch.de>
Cc: Greg Kroah-Hartman <gregkh@...uxfoundation.org>,
Benoit Cousson <b-cousson@...com>,
Paul Walmsley <paul@...an.com>,
Tony Lindgren <tony@...mide.com>,
Linux OMAP List <linux-omap@...r.kernel.org>,
Linux Kernel Mailing List <linux-kernel@...r.kernel.org>
Subject: Re: [RFC PATCH 1/3] uio: uio_pruss: port to AM33xx
On Thu, Sep 27, 2012 at 01:51:18AM +0200, Hans J. Koch wrote:
> 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.
Will separate out on v2.
-Matt
> >
> > Adds DT, pinctrl, and runtime PM support for use on
> > AM33xx.
> >
> > Signed-off-by: Matt Porter <mporter@...com>
> > ---
> > 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@...00000 {
> > + 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@...00000 {
> > + 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@...r.kernel.org
> More majordomo info at http://vger.kernel.org/majordomo-info.html
> Please read the FAQ at http://www.tux.org/lkml/
--
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majordomo@...r.kernel.org
More majordomo info at http://vger.kernel.org/majordomo-info.html
Please read the FAQ at http://www.tux.org/lkml/
Powered by blists - more mailing lists