Re: [PATCH v2 4/6] drivers: uio: Add X-Gene QMTM UIO driver

2014-10-20 Thread Ankit Jindal
On 30 September 2014 11:35, Varka Bhadram  wrote:
> I think *drivers* is not required in the commit message...
>
>
> On 09/30/2014 09:56 AM, Ankit Jindal wrote:
>>
>> The Applied Micro X-Gene SOC has on-chip QMTM (Queue manager
>> and Traffic manager) which is hardware based Queue or Ring
>> manager. This QMTM device can be used in conjunction with
>> other devices such as DMA Engine, Ethernet, Security Engine,
>> etc to assign work based on queues or rings.
>>
>> This patch allows user space access to X-Gene QMTM device.
>>
>> Signed-off-by: Ankit Jindal 
>> Signed-off-by: Tushar Jagad 
>> ---
>>   drivers/uio/Kconfig  |8 ++
>>   drivers/uio/Makefile |1 +
>>   drivers/uio/uio_xgene_qmtm.c |  278
>> ++
>>   3 files changed, 287 insertions(+)
>>   create mode 100644 drivers/uio/uio_xgene_qmtm.c
>>
>> diff --git a/drivers/uio/Kconfig b/drivers/uio/Kconfig
>> index 5a90914..76b1858 100644
>> --- a/drivers/uio/Kconfig
>> +++ b/drivers/uio/Kconfig
>> @@ -135,4 +135,12 @@ config UIO_MF624
>>   If you compile this as a module, it will be called uio_mf624.
>>   +config UIO_XGENE_QMTM
>> +   tristate "Applied Micro X-Gene QMTM driver"
>> +   depends on OF
>> +   help
>> + Userspace I/O interface for the X-Gene QMTM. The userspace part
>> of
>> + this driver will be available for download from the Applied
>> Micro
>> + web site (http://www.apm.com/).
>> +
>>   endif
>> diff --git a/drivers/uio/Makefile b/drivers/uio/Makefile
>> index d3218bd..633eaa0 100644
>> --- a/drivers/uio/Makefile
>> +++ b/drivers/uio/Makefile
>> @@ -8,3 +8,4 @@ obj-$(CONFIG_UIO_PCI_GENERIC)   += uio_pci_generic.o
>>   obj-$(CONFIG_UIO_NETX)+= uio_netx.o
>>   obj-$(CONFIG_UIO_PRUSS) += uio_pruss.o
>>   obj-$(CONFIG_UIO_MF624) += uio_mf624.o
>> +obj-$(CONFIG_UIO_XGENE_QMTM)   += uio_xgene_qmtm.o
>> diff --git a/drivers/uio/uio_xgene_qmtm.c b/drivers/uio/uio_xgene_qmtm.c
>> new file mode 100644
>> index 000..36d9000
>> --- /dev/null
>> +++ b/drivers/uio/uio_xgene_qmtm.c
>> @@ -0,0 +1,278 @@
>> +/*
>> + * X-Gene Queue Manager Traffic Manager (QMTM) UIO driver
>> (uio_xgene_qmtm)
>> + *
>> + * This driver exports QMTM CSRs, Fabric and memory for queues to
>> user-space
>> + *
>> + * Copyright (C) 2014 Applied Micro - http://www.apm.com/
>> + * Copyright (C) 2014 Linaro Ltd.
>> + *
>> + * This program is free software; you can redistribute it and/or
>> + * modify it under the terms of the GNU General Public License as
>> + * published by the Free Software Foundation version 2.
>> + *
>> + * This program is distributed "as is" WITHOUT ANY WARRANTY of any
>> + * kind, whether express or implied; without even the implied warranty
>> + * of MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE.  See the
>> + * GNU General Public License for more details.
>> + */
>> +
>> +#include 
>> +#include 
>> +#include 
>> +#include 
>> +#include 
>> +#include 
>> +#include 
>> +#include 
>> +#include 
>> +#include 
>> +#include 
>> +
>
>
> Headers in alphabetical order...?
>
> #include 
> #include 
>
> ...
>
>
>> +#define DRV_NAME "qmtm_uio"
>> +#define DRV_VERSION "1.0"
>> +
>> +#define QMTM_CFG_MEM_RAM_SHUTDOWN  0xd070
>> +
>> +#define QMTM_DEFAULT_QSIZE 65536
>> +
>> +struct uio_qmtm_dev {
>> +   struct uio_info *info;
>> +   struct clk *qmtm_clk;
>> +};
>> +
>> +/* QMTM CSR read/write routine */
>> +static inline void qmtm_csr_write(struct uio_qmtm_dev *qmtm_dev, u32
>> offset,
>> +   u32 data)
>> +{
>> +   void __iomem *addr = qmtm_dev->info->mem[0].internal_addr;
>> +
>> +   writel(data, addr + offset);
>> +}
>> +
>> +static inline u32 qmtm_csr_read(struct uio_qmtm_dev *qmtm_dev, u32
>> offset)
>> +{
>> +   void __iomem *addr = qmtm_dev->info->mem[0].internal_addr;
>> +
>> +   return readl(addr + offset);
>> +}
>> +
>> +static int qmtm_reset(struct uio_qmtm_dev *qmtm_dev)
>> +{
>> +   u32 val;
>> +   int wait = 1000;
>> +
>> +   /* reset the internal memory of the device */
>> +   qmtm_csr_write(qmtm_dev, QMTM_CFG_MEM_RAM_SHUTDOWN, 0);
>> +
>> +   /* check whether device internal memory is out of reset or not */
>> +   while (1) {
>> +   val = qmtm_csr_read(qmtm_dev, QMTM_CFG_MEM_RAM_SHUTDOWN);
>> +
>> +   if (val != 0x)
>> +   return 0;
>> +
>> +   if (!wait--)
>> +   return -EBUSY;
>> +
>> +   udelay(1);
>> +   }
>> +}
>> +
>> +static void qmtm_cleanup(struct platform_device *pdev,
>> +   struct uio_qmtm_dev *qmtm_dev)
>> +{
>> +   struct uio_info *info = qmtm_dev->info;
>> +
>> +   uio_unregister_device(info);
>> +
>> +   clk_disable_unprepare(qmtm_dev->qmtm_clk);
>> +}
>> +
>> +static int qmtm_probe(struct platform_device *pdev)
>> +{
>> +   struct uio_info *info;
>> +   struct uio_qmtm_dev *qmtm_dev;
>> +   

Re: [PATCH v2 4/6] drivers: uio: Add X-Gene QMTM UIO driver

2014-10-20 Thread Ankit Jindal
On 30 September 2014 11:05, Guenter Roeck  wrote:
> On Tue, Sep 30, 2014 at 09:56:07AM +0530, Ankit Jindal wrote:
>> The Applied Micro X-Gene SOC has on-chip QMTM (Queue manager
>> and Traffic manager) which is hardware based Queue or Ring
>> manager. This QMTM device can be used in conjunction with
>> other devices such as DMA Engine, Ethernet, Security Engine,
>> etc to assign work based on queues or rings.
>>
>> This patch allows user space access to X-Gene QMTM device.
>>
>> Signed-off-by: Ankit Jindal 
>> Signed-off-by: Tushar Jagad 
>> ---
>>  drivers/uio/Kconfig  |8 ++
>>  drivers/uio/Makefile |1 +
>>  drivers/uio/uio_xgene_qmtm.c |  278 
>> ++
>>  3 files changed, 287 insertions(+)
>>  create mode 100644 drivers/uio/uio_xgene_qmtm.c
>>
>> diff --git a/drivers/uio/Kconfig b/drivers/uio/Kconfig
>> index 5a90914..76b1858 100644
>> --- a/drivers/uio/Kconfig
>> +++ b/drivers/uio/Kconfig
>> @@ -135,4 +135,12 @@ config UIO_MF624
>>
>> If you compile this as a module, it will be called uio_mf624.
>>
>> +config UIO_XGENE_QMTM
>> + tristate "Applied Micro X-Gene QMTM driver"
>> + depends on OF
>> + help
>> +   Userspace I/O interface for the X-Gene QMTM. The userspace part of
>> +   this driver will be available for download from the Applied Micro
>> +   web site (http://www.apm.com/).
>> +
>>  endif
>> diff --git a/drivers/uio/Makefile b/drivers/uio/Makefile
>> index d3218bd..633eaa0 100644
>> --- a/drivers/uio/Makefile
>> +++ b/drivers/uio/Makefile
>> @@ -8,3 +8,4 @@ obj-$(CONFIG_UIO_PCI_GENERIC) += uio_pci_generic.o
>>  obj-$(CONFIG_UIO_NETX)   += uio_netx.o
>>  obj-$(CONFIG_UIO_PRUSS) += uio_pruss.o
>>  obj-$(CONFIG_UIO_MF624) += uio_mf624.o
>> +obj-$(CONFIG_UIO_XGENE_QMTM) += uio_xgene_qmtm.o
>> diff --git a/drivers/uio/uio_xgene_qmtm.c b/drivers/uio/uio_xgene_qmtm.c
>> new file mode 100644
>> index 000..36d9000
>> --- /dev/null
>> +++ b/drivers/uio/uio_xgene_qmtm.c
>> @@ -0,0 +1,278 @@
>> +/*
>> + * X-Gene Queue Manager Traffic Manager (QMTM) UIO driver (uio_xgene_qmtm)
>> + *
>> + * This driver exports QMTM CSRs, Fabric and memory for queues to user-space
>> + *
>> + * Copyright (C) 2014 Applied Micro - http://www.apm.com/
>> + * Copyright (C) 2014 Linaro Ltd.
>> + *
>> + * This program is free software; you can redistribute it and/or
>> + * modify it under the terms of the GNU General Public License as
>> + * published by the Free Software Foundation version 2.
>> + *
>> + * This program is distributed "as is" WITHOUT ANY WARRANTY of any
>> + * kind, whether express or implied; without even the implied warranty
>> + * of MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE.  See the
>> + * GNU General Public License for more details.
>> + */
>> +
>> +#include 
>> +#include 
>> +#include 
>> +#include 
>> +#include 
>> +#include 
>> +#include 
>> +#include 
>> +#include 
>> +#include 
>> +#include 
>> +
>> +#define DRV_NAME "qmtm_uio"
>> +#define DRV_VERSION "1.0"
>> +
>> +#define QMTM_CFG_MEM_RAM_SHUTDOWN0xd070
>> +
>> +#define QMTM_DEFAULT_QSIZE   65536
>> +
>> +struct uio_qmtm_dev {
>> + struct uio_info *info;
>> + struct clk *qmtm_clk;
>> +};
>> +
>> +/* QMTM CSR read/write routine */
>> +static inline void qmtm_csr_write(struct uio_qmtm_dev *qmtm_dev, u32 offset,
>> + u32 data)
>> +{
>> + void __iomem *addr = qmtm_dev->info->mem[0].internal_addr;
>> +
>> + writel(data, addr + offset);
>> +}
>> +
>> +static inline u32 qmtm_csr_read(struct uio_qmtm_dev *qmtm_dev, u32 offset)
>> +{
>> + void __iomem *addr = qmtm_dev->info->mem[0].internal_addr;
>> +
>> + return readl(addr + offset);
>> +}
>> +
>> +static int qmtm_reset(struct uio_qmtm_dev *qmtm_dev)
>> +{
>> + u32 val;
>> + int wait = 1000;
>> +
>> + /* reset the internal memory of the device */
>> + qmtm_csr_write(qmtm_dev, QMTM_CFG_MEM_RAM_SHUTDOWN, 0);
>> +
>> + /* check whether device internal memory is out of reset or not */
>> + while (1) {
>
> Seems to me that
> while (wait--) {
> ...
> }
> return -EBUSY;
>
> would be much easier to understand.
>
> Also, not sure if EBUSY is really appropriate here.
> ETIMEDOUT, maybe ?
>
>> + val = qmtm_csr_read(qmtm_dev, QMTM_CFG_MEM_RAM_SHUTDOWN);
>> +
>> + if (val != 0x)
>> + return 0;
>> +
>> + if (!wait--)
>> + return -EBUSY;
>> +
>> + udelay(1);
>> + }
>> +}
>> +
>> +static void qmtm_cleanup(struct platform_device *pdev,
>> + struct uio_qmtm_dev *qmtm_dev)
>> +{
>> + struct uio_info *info = qmtm_dev->info;
>> +
>> + uio_unregister_device(info);
>> +
>> + clk_disable_unprepare(qmtm_dev->qmtm_clk);
>> +}
>> +
>> +static int qmtm_probe(struct platform_device *pdev)
>> +{
>> + struct uio_info *info;
>> + struct uio_qmtm_dev *qmtm_dev;
>> + 

Re: [PATCH v2 4/6] drivers: uio: Add X-Gene QMTM UIO driver

2014-10-20 Thread Ankit Jindal
On 30 September 2014 11:05, Guenter Roeck li...@roeck-us.net wrote:
 On Tue, Sep 30, 2014 at 09:56:07AM +0530, Ankit Jindal wrote:
 The Applied Micro X-Gene SOC has on-chip QMTM (Queue manager
 and Traffic manager) which is hardware based Queue or Ring
 manager. This QMTM device can be used in conjunction with
 other devices such as DMA Engine, Ethernet, Security Engine,
 etc to assign work based on queues or rings.

 This patch allows user space access to X-Gene QMTM device.

 Signed-off-by: Ankit Jindal ankit.jin...@linaro.org
 Signed-off-by: Tushar Jagad tushar.ja...@linaro.org
 ---
  drivers/uio/Kconfig  |8 ++
  drivers/uio/Makefile |1 +
  drivers/uio/uio_xgene_qmtm.c |  278 
 ++
  3 files changed, 287 insertions(+)
  create mode 100644 drivers/uio/uio_xgene_qmtm.c

 diff --git a/drivers/uio/Kconfig b/drivers/uio/Kconfig
 index 5a90914..76b1858 100644
 --- a/drivers/uio/Kconfig
 +++ b/drivers/uio/Kconfig
 @@ -135,4 +135,12 @@ config UIO_MF624

 If you compile this as a module, it will be called uio_mf624.

 +config UIO_XGENE_QMTM
 + tristate Applied Micro X-Gene QMTM driver
 + depends on OF
 + help
 +   Userspace I/O interface for the X-Gene QMTM. The userspace part of
 +   this driver will be available for download from the Applied Micro
 +   web site (http://www.apm.com/).
 +
  endif
 diff --git a/drivers/uio/Makefile b/drivers/uio/Makefile
 index d3218bd..633eaa0 100644
 --- a/drivers/uio/Makefile
 +++ b/drivers/uio/Makefile
 @@ -8,3 +8,4 @@ obj-$(CONFIG_UIO_PCI_GENERIC) += uio_pci_generic.o
  obj-$(CONFIG_UIO_NETX)   += uio_netx.o
  obj-$(CONFIG_UIO_PRUSS) += uio_pruss.o
  obj-$(CONFIG_UIO_MF624) += uio_mf624.o
 +obj-$(CONFIG_UIO_XGENE_QMTM) += uio_xgene_qmtm.o
 diff --git a/drivers/uio/uio_xgene_qmtm.c b/drivers/uio/uio_xgene_qmtm.c
 new file mode 100644
 index 000..36d9000
 --- /dev/null
 +++ b/drivers/uio/uio_xgene_qmtm.c
 @@ -0,0 +1,278 @@
 +/*
 + * X-Gene Queue Manager Traffic Manager (QMTM) UIO driver (uio_xgene_qmtm)
 + *
 + * This driver exports QMTM CSRs, Fabric and memory for queues to user-space
 + *
 + * Copyright (C) 2014 Applied Micro - http://www.apm.com/
 + * Copyright (C) 2014 Linaro Ltd.
 + *
 + * This program is free software; you can redistribute it and/or
 + * modify it under the terms of the GNU General Public License as
 + * published by the Free Software Foundation version 2.
 + *
 + * This program is distributed as is WITHOUT ANY WARRANTY of any
 + * kind, whether express or implied; without even the implied warranty
 + * of MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE.  See the
 + * GNU General Public License for more details.
 + */
 +
 +#include linux/device.h
 +#include linux/delay.h
 +#include linux/module.h
 +#include linux/moduleparam.h
 +#include linux/platform_device.h
 +#include linux/uio_driver.h
 +#include linux/io.h
 +#include linux/clk.h
 +#include linux/slab.h
 +#include linux/of_platform.h
 +#include linux/of_address.h
 +
 +#define DRV_NAME qmtm_uio
 +#define DRV_VERSION 1.0
 +
 +#define QMTM_CFG_MEM_RAM_SHUTDOWN0xd070
 +
 +#define QMTM_DEFAULT_QSIZE   65536
 +
 +struct uio_qmtm_dev {
 + struct uio_info *info;
 + struct clk *qmtm_clk;
 +};
 +
 +/* QMTM CSR read/write routine */
 +static inline void qmtm_csr_write(struct uio_qmtm_dev *qmtm_dev, u32 offset,
 + u32 data)
 +{
 + void __iomem *addr = qmtm_dev-info-mem[0].internal_addr;
 +
 + writel(data, addr + offset);
 +}
 +
 +static inline u32 qmtm_csr_read(struct uio_qmtm_dev *qmtm_dev, u32 offset)
 +{
 + void __iomem *addr = qmtm_dev-info-mem[0].internal_addr;
 +
 + return readl(addr + offset);
 +}
 +
 +static int qmtm_reset(struct uio_qmtm_dev *qmtm_dev)
 +{
 + u32 val;
 + int wait = 1000;
 +
 + /* reset the internal memory of the device */
 + qmtm_csr_write(qmtm_dev, QMTM_CFG_MEM_RAM_SHUTDOWN, 0);
 +
 + /* check whether device internal memory is out of reset or not */
 + while (1) {

 Seems to me that
 while (wait--) {
 ...
 }
 return -EBUSY;

 would be much easier to understand.

 Also, not sure if EBUSY is really appropriate here.
 ETIMEDOUT, maybe ?

 + val = qmtm_csr_read(qmtm_dev, QMTM_CFG_MEM_RAM_SHUTDOWN);
 +
 + if (val != 0x)
 + return 0;
 +
 + if (!wait--)
 + return -EBUSY;
 +
 + udelay(1);
 + }
 +}
 +
 +static void qmtm_cleanup(struct platform_device *pdev,
 + struct uio_qmtm_dev *qmtm_dev)
 +{
 + struct uio_info *info = qmtm_dev-info;
 +
 + uio_unregister_device(info);
 +
 + clk_disable_unprepare(qmtm_dev-qmtm_clk);
 +}
 +
 +static int qmtm_probe(struct platform_device *pdev)
 +{
 + struct uio_info *info;
 + struct uio_qmtm_dev *qmtm_dev;
 + struct resource *csr;
 + struct resource *fabric;
 + struct resource qpool;

Re: [PATCH v2 4/6] drivers: uio: Add X-Gene QMTM UIO driver

2014-10-20 Thread Ankit Jindal
On 30 September 2014 11:35, Varka Bhadram varkabhad...@gmail.com wrote:
 I think *drivers* is not required in the commit message...


 On 09/30/2014 09:56 AM, Ankit Jindal wrote:

 The Applied Micro X-Gene SOC has on-chip QMTM (Queue manager
 and Traffic manager) which is hardware based Queue or Ring
 manager. This QMTM device can be used in conjunction with
 other devices such as DMA Engine, Ethernet, Security Engine,
 etc to assign work based on queues or rings.

 This patch allows user space access to X-Gene QMTM device.

 Signed-off-by: Ankit Jindal ankit.jin...@linaro.org
 Signed-off-by: Tushar Jagad tushar.ja...@linaro.org
 ---
   drivers/uio/Kconfig  |8 ++
   drivers/uio/Makefile |1 +
   drivers/uio/uio_xgene_qmtm.c |  278
 ++
   3 files changed, 287 insertions(+)
   create mode 100644 drivers/uio/uio_xgene_qmtm.c

 diff --git a/drivers/uio/Kconfig b/drivers/uio/Kconfig
 index 5a90914..76b1858 100644
 --- a/drivers/uio/Kconfig
 +++ b/drivers/uio/Kconfig
 @@ -135,4 +135,12 @@ config UIO_MF624
   If you compile this as a module, it will be called uio_mf624.
   +config UIO_XGENE_QMTM
 +   tristate Applied Micro X-Gene QMTM driver
 +   depends on OF
 +   help
 + Userspace I/O interface for the X-Gene QMTM. The userspace part
 of
 + this driver will be available for download from the Applied
 Micro
 + web site (http://www.apm.com/).
 +
   endif
 diff --git a/drivers/uio/Makefile b/drivers/uio/Makefile
 index d3218bd..633eaa0 100644
 --- a/drivers/uio/Makefile
 +++ b/drivers/uio/Makefile
 @@ -8,3 +8,4 @@ obj-$(CONFIG_UIO_PCI_GENERIC)   += uio_pci_generic.o
   obj-$(CONFIG_UIO_NETX)+= uio_netx.o
   obj-$(CONFIG_UIO_PRUSS) += uio_pruss.o
   obj-$(CONFIG_UIO_MF624) += uio_mf624.o
 +obj-$(CONFIG_UIO_XGENE_QMTM)   += uio_xgene_qmtm.o
 diff --git a/drivers/uio/uio_xgene_qmtm.c b/drivers/uio/uio_xgene_qmtm.c
 new file mode 100644
 index 000..36d9000
 --- /dev/null
 +++ b/drivers/uio/uio_xgene_qmtm.c
 @@ -0,0 +1,278 @@
 +/*
 + * X-Gene Queue Manager Traffic Manager (QMTM) UIO driver
 (uio_xgene_qmtm)
 + *
 + * This driver exports QMTM CSRs, Fabric and memory for queues to
 user-space
 + *
 + * Copyright (C) 2014 Applied Micro - http://www.apm.com/
 + * Copyright (C) 2014 Linaro Ltd.
 + *
 + * This program is free software; you can redistribute it and/or
 + * modify it under the terms of the GNU General Public License as
 + * published by the Free Software Foundation version 2.
 + *
 + * This program is distributed as is WITHOUT ANY WARRANTY of any
 + * kind, whether express or implied; without even the implied warranty
 + * of MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE.  See the
 + * GNU General Public License for more details.
 + */
 +
 +#include linux/device.h
 +#include linux/delay.h
 +#include linux/module.h
 +#include linux/moduleparam.h
 +#include linux/platform_device.h
 +#include linux/uio_driver.h
 +#include linux/io.h
 +#include linux/clk.h
 +#include linux/slab.h
 +#include linux/of_platform.h
 +#include linux/of_address.h
 +


 Headers in alphabetical order...?

 #include linux/clk.h
 #include linux/device.h

 ...


 +#define DRV_NAME qmtm_uio
 +#define DRV_VERSION 1.0
 +
 +#define QMTM_CFG_MEM_RAM_SHUTDOWN  0xd070
 +
 +#define QMTM_DEFAULT_QSIZE 65536
 +
 +struct uio_qmtm_dev {
 +   struct uio_info *info;
 +   struct clk *qmtm_clk;
 +};
 +
 +/* QMTM CSR read/write routine */
 +static inline void qmtm_csr_write(struct uio_qmtm_dev *qmtm_dev, u32
 offset,
 +   u32 data)
 +{
 +   void __iomem *addr = qmtm_dev-info-mem[0].internal_addr;
 +
 +   writel(data, addr + offset);
 +}
 +
 +static inline u32 qmtm_csr_read(struct uio_qmtm_dev *qmtm_dev, u32
 offset)
 +{
 +   void __iomem *addr = qmtm_dev-info-mem[0].internal_addr;
 +
 +   return readl(addr + offset);
 +}
 +
 +static int qmtm_reset(struct uio_qmtm_dev *qmtm_dev)
 +{
 +   u32 val;
 +   int wait = 1000;
 +
 +   /* reset the internal memory of the device */
 +   qmtm_csr_write(qmtm_dev, QMTM_CFG_MEM_RAM_SHUTDOWN, 0);
 +
 +   /* check whether device internal memory is out of reset or not */
 +   while (1) {
 +   val = qmtm_csr_read(qmtm_dev, QMTM_CFG_MEM_RAM_SHUTDOWN);
 +
 +   if (val != 0x)
 +   return 0;
 +
 +   if (!wait--)
 +   return -EBUSY;
 +
 +   udelay(1);
 +   }
 +}
 +
 +static void qmtm_cleanup(struct platform_device *pdev,
 +   struct uio_qmtm_dev *qmtm_dev)
 +{
 +   struct uio_info *info = qmtm_dev-info;
 +
 +   uio_unregister_device(info);
 +
 +   clk_disable_unprepare(qmtm_dev-qmtm_clk);
 +}
 +
 +static int qmtm_probe(struct platform_device *pdev)
 +{
 +   struct uio_info *info;
 +   struct uio_qmtm_dev *qmtm_dev;
 +   struct resource *csr;
 +   struct resource *fabric;
 +   

Re: [PATCH v2 4/6] drivers: uio: Add X-Gene QMTM UIO driver

2014-09-30 Thread Varka Bhadram

I think *drivers* is not required in the commit message...

On 09/30/2014 09:56 AM, Ankit Jindal wrote:

The Applied Micro X-Gene SOC has on-chip QMTM (Queue manager
and Traffic manager) which is hardware based Queue or Ring
manager. This QMTM device can be used in conjunction with
other devices such as DMA Engine, Ethernet, Security Engine,
etc to assign work based on queues or rings.

This patch allows user space access to X-Gene QMTM device.

Signed-off-by: Ankit Jindal 
Signed-off-by: Tushar Jagad 
---
  drivers/uio/Kconfig  |8 ++
  drivers/uio/Makefile |1 +
  drivers/uio/uio_xgene_qmtm.c |  278 ++
  3 files changed, 287 insertions(+)
  create mode 100644 drivers/uio/uio_xgene_qmtm.c

diff --git a/drivers/uio/Kconfig b/drivers/uio/Kconfig
index 5a90914..76b1858 100644
--- a/drivers/uio/Kconfig
+++ b/drivers/uio/Kconfig
@@ -135,4 +135,12 @@ config UIO_MF624
  
  	  If you compile this as a module, it will be called uio_mf624.
  
+config UIO_XGENE_QMTM

+   tristate "Applied Micro X-Gene QMTM driver"
+   depends on OF
+   help
+ Userspace I/O interface for the X-Gene QMTM. The userspace part of
+ this driver will be available for download from the Applied Micro
+ web site (http://www.apm.com/).
+
  endif
diff --git a/drivers/uio/Makefile b/drivers/uio/Makefile
index d3218bd..633eaa0 100644
--- a/drivers/uio/Makefile
+++ b/drivers/uio/Makefile
@@ -8,3 +8,4 @@ obj-$(CONFIG_UIO_PCI_GENERIC)   += uio_pci_generic.o
  obj-$(CONFIG_UIO_NETX)+= uio_netx.o
  obj-$(CONFIG_UIO_PRUSS) += uio_pruss.o
  obj-$(CONFIG_UIO_MF624) += uio_mf624.o
+obj-$(CONFIG_UIO_XGENE_QMTM)   += uio_xgene_qmtm.o
diff --git a/drivers/uio/uio_xgene_qmtm.c b/drivers/uio/uio_xgene_qmtm.c
new file mode 100644
index 000..36d9000
--- /dev/null
+++ b/drivers/uio/uio_xgene_qmtm.c
@@ -0,0 +1,278 @@
+/*
+ * X-Gene Queue Manager Traffic Manager (QMTM) UIO driver (uio_xgene_qmtm)
+ *
+ * This driver exports QMTM CSRs, Fabric and memory for queues to user-space
+ *
+ * Copyright (C) 2014 Applied Micro - http://www.apm.com/
+ * Copyright (C) 2014 Linaro Ltd.
+ *
+ * This program is free software; you can redistribute it and/or
+ * modify it under the terms of the GNU General Public License as
+ * published by the Free Software Foundation version 2.
+ *
+ * This program is distributed "as is" WITHOUT ANY WARRANTY of any
+ * kind, whether express or implied; without even the implied warranty
+ * of MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE.  See the
+ * GNU General Public License for more details.
+ */
+
+#include 
+#include 
+#include 
+#include 
+#include 
+#include 
+#include 
+#include 
+#include 
+#include 
+#include 
+


Headers in alphabetical order...?

#include 
#include 

...


+#define DRV_NAME "qmtm_uio"
+#define DRV_VERSION "1.0"
+
+#define QMTM_CFG_MEM_RAM_SHUTDOWN  0xd070
+
+#define QMTM_DEFAULT_QSIZE 65536
+
+struct uio_qmtm_dev {
+   struct uio_info *info;
+   struct clk *qmtm_clk;
+};
+
+/* QMTM CSR read/write routine */
+static inline void qmtm_csr_write(struct uio_qmtm_dev *qmtm_dev, u32 offset,
+   u32 data)
+{
+   void __iomem *addr = qmtm_dev->info->mem[0].internal_addr;
+
+   writel(data, addr + offset);
+}
+
+static inline u32 qmtm_csr_read(struct uio_qmtm_dev *qmtm_dev, u32 offset)
+{
+   void __iomem *addr = qmtm_dev->info->mem[0].internal_addr;
+
+   return readl(addr + offset);
+}
+
+static int qmtm_reset(struct uio_qmtm_dev *qmtm_dev)
+{
+   u32 val;
+   int wait = 1000;
+
+   /* reset the internal memory of the device */
+   qmtm_csr_write(qmtm_dev, QMTM_CFG_MEM_RAM_SHUTDOWN, 0);
+
+   /* check whether device internal memory is out of reset or not */
+   while (1) {
+   val = qmtm_csr_read(qmtm_dev, QMTM_CFG_MEM_RAM_SHUTDOWN);
+
+   if (val != 0x)
+   return 0;
+
+   if (!wait--)
+   return -EBUSY;
+
+   udelay(1);
+   }
+}
+
+static void qmtm_cleanup(struct platform_device *pdev,
+   struct uio_qmtm_dev *qmtm_dev)
+{
+   struct uio_info *info = qmtm_dev->info;
+
+   uio_unregister_device(info);
+
+   clk_disable_unprepare(qmtm_dev->qmtm_clk);
+}
+
+static int qmtm_probe(struct platform_device *pdev)
+{
+   struct uio_info *info;
+   struct uio_qmtm_dev *qmtm_dev;
+   struct resource *csr;
+   struct resource *fabric;
+   struct resource qpool;
+   unsigned int num_queues;
+   unsigned int devid;
+   phandle qpool_phandle;
+   struct device_node *qpool_node;
+   int ret;
+
+   qmtm_dev = devm_kzalloc(>dev, sizeof(struct uio_qmtm_dev),
+   GFP_KERNEL);
+   if (!qmtm_dev)
+   return -ENOMEM;
+
+   qmtm_dev->info = devm_kzalloc(>dev, sizeof(*info), GFP_KERNEL);
+   if (!qmtm_dev->info)
+ 

Re: [PATCH v2 4/6] drivers: uio: Add X-Gene QMTM UIO driver

2014-09-30 Thread Varka Bhadram

I think *drivers* is not required in the commit message...

On 09/30/2014 09:56 AM, Ankit Jindal wrote:

The Applied Micro X-Gene SOC has on-chip QMTM (Queue manager
and Traffic manager) which is hardware based Queue or Ring
manager. This QMTM device can be used in conjunction with
other devices such as DMA Engine, Ethernet, Security Engine,
etc to assign work based on queues or rings.

This patch allows user space access to X-Gene QMTM device.

Signed-off-by: Ankit Jindal ankit.jin...@linaro.org
Signed-off-by: Tushar Jagad tushar.ja...@linaro.org
---
  drivers/uio/Kconfig  |8 ++
  drivers/uio/Makefile |1 +
  drivers/uio/uio_xgene_qmtm.c |  278 ++
  3 files changed, 287 insertions(+)
  create mode 100644 drivers/uio/uio_xgene_qmtm.c

diff --git a/drivers/uio/Kconfig b/drivers/uio/Kconfig
index 5a90914..76b1858 100644
--- a/drivers/uio/Kconfig
+++ b/drivers/uio/Kconfig
@@ -135,4 +135,12 @@ config UIO_MF624
  
  	  If you compile this as a module, it will be called uio_mf624.
  
+config UIO_XGENE_QMTM

+   tristate Applied Micro X-Gene QMTM driver
+   depends on OF
+   help
+ Userspace I/O interface for the X-Gene QMTM. The userspace part of
+ this driver will be available for download from the Applied Micro
+ web site (http://www.apm.com/).
+
  endif
diff --git a/drivers/uio/Makefile b/drivers/uio/Makefile
index d3218bd..633eaa0 100644
--- a/drivers/uio/Makefile
+++ b/drivers/uio/Makefile
@@ -8,3 +8,4 @@ obj-$(CONFIG_UIO_PCI_GENERIC)   += uio_pci_generic.o
  obj-$(CONFIG_UIO_NETX)+= uio_netx.o
  obj-$(CONFIG_UIO_PRUSS) += uio_pruss.o
  obj-$(CONFIG_UIO_MF624) += uio_mf624.o
+obj-$(CONFIG_UIO_XGENE_QMTM)   += uio_xgene_qmtm.o
diff --git a/drivers/uio/uio_xgene_qmtm.c b/drivers/uio/uio_xgene_qmtm.c
new file mode 100644
index 000..36d9000
--- /dev/null
+++ b/drivers/uio/uio_xgene_qmtm.c
@@ -0,0 +1,278 @@
+/*
+ * X-Gene Queue Manager Traffic Manager (QMTM) UIO driver (uio_xgene_qmtm)
+ *
+ * This driver exports QMTM CSRs, Fabric and memory for queues to user-space
+ *
+ * Copyright (C) 2014 Applied Micro - http://www.apm.com/
+ * Copyright (C) 2014 Linaro Ltd.
+ *
+ * This program is free software; you can redistribute it and/or
+ * modify it under the terms of the GNU General Public License as
+ * published by the Free Software Foundation version 2.
+ *
+ * This program is distributed as is WITHOUT ANY WARRANTY of any
+ * kind, whether express or implied; without even the implied warranty
+ * of MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE.  See the
+ * GNU General Public License for more details.
+ */
+
+#include linux/device.h
+#include linux/delay.h
+#include linux/module.h
+#include linux/moduleparam.h
+#include linux/platform_device.h
+#include linux/uio_driver.h
+#include linux/io.h
+#include linux/clk.h
+#include linux/slab.h
+#include linux/of_platform.h
+#include linux/of_address.h
+


Headers in alphabetical order...?

#include linux/clk.h
#include linux/device.h

...


+#define DRV_NAME qmtm_uio
+#define DRV_VERSION 1.0
+
+#define QMTM_CFG_MEM_RAM_SHUTDOWN  0xd070
+
+#define QMTM_DEFAULT_QSIZE 65536
+
+struct uio_qmtm_dev {
+   struct uio_info *info;
+   struct clk *qmtm_clk;
+};
+
+/* QMTM CSR read/write routine */
+static inline void qmtm_csr_write(struct uio_qmtm_dev *qmtm_dev, u32 offset,
+   u32 data)
+{
+   void __iomem *addr = qmtm_dev-info-mem[0].internal_addr;
+
+   writel(data, addr + offset);
+}
+
+static inline u32 qmtm_csr_read(struct uio_qmtm_dev *qmtm_dev, u32 offset)
+{
+   void __iomem *addr = qmtm_dev-info-mem[0].internal_addr;
+
+   return readl(addr + offset);
+}
+
+static int qmtm_reset(struct uio_qmtm_dev *qmtm_dev)
+{
+   u32 val;
+   int wait = 1000;
+
+   /* reset the internal memory of the device */
+   qmtm_csr_write(qmtm_dev, QMTM_CFG_MEM_RAM_SHUTDOWN, 0);
+
+   /* check whether device internal memory is out of reset or not */
+   while (1) {
+   val = qmtm_csr_read(qmtm_dev, QMTM_CFG_MEM_RAM_SHUTDOWN);
+
+   if (val != 0x)
+   return 0;
+
+   if (!wait--)
+   return -EBUSY;
+
+   udelay(1);
+   }
+}
+
+static void qmtm_cleanup(struct platform_device *pdev,
+   struct uio_qmtm_dev *qmtm_dev)
+{
+   struct uio_info *info = qmtm_dev-info;
+
+   uio_unregister_device(info);
+
+   clk_disable_unprepare(qmtm_dev-qmtm_clk);
+}
+
+static int qmtm_probe(struct platform_device *pdev)
+{
+   struct uio_info *info;
+   struct uio_qmtm_dev *qmtm_dev;
+   struct resource *csr;
+   struct resource *fabric;
+   struct resource qpool;
+   unsigned int num_queues;
+   unsigned int devid;
+   phandle qpool_phandle;
+   struct device_node *qpool_node;
+   int ret;
+
+   qmtm_dev = devm_kzalloc(pdev-dev, 

Re: [PATCH v2 4/6] drivers: uio: Add X-Gene QMTM UIO driver

2014-09-29 Thread Guenter Roeck
On Tue, Sep 30, 2014 at 09:56:07AM +0530, Ankit Jindal wrote:
> The Applied Micro X-Gene SOC has on-chip QMTM (Queue manager
> and Traffic manager) which is hardware based Queue or Ring
> manager. This QMTM device can be used in conjunction with
> other devices such as DMA Engine, Ethernet, Security Engine,
> etc to assign work based on queues or rings.
> 
> This patch allows user space access to X-Gene QMTM device.
> 
> Signed-off-by: Ankit Jindal 
> Signed-off-by: Tushar Jagad 
> ---
>  drivers/uio/Kconfig  |8 ++
>  drivers/uio/Makefile |1 +
>  drivers/uio/uio_xgene_qmtm.c |  278 
> ++
>  3 files changed, 287 insertions(+)
>  create mode 100644 drivers/uio/uio_xgene_qmtm.c
> 
> diff --git a/drivers/uio/Kconfig b/drivers/uio/Kconfig
> index 5a90914..76b1858 100644
> --- a/drivers/uio/Kconfig
> +++ b/drivers/uio/Kconfig
> @@ -135,4 +135,12 @@ config UIO_MF624
>  
> If you compile this as a module, it will be called uio_mf624.
>  
> +config UIO_XGENE_QMTM
> + tristate "Applied Micro X-Gene QMTM driver"
> + depends on OF
> + help
> +   Userspace I/O interface for the X-Gene QMTM. The userspace part of
> +   this driver will be available for download from the Applied Micro
> +   web site (http://www.apm.com/).
> +
>  endif
> diff --git a/drivers/uio/Makefile b/drivers/uio/Makefile
> index d3218bd..633eaa0 100644
> --- a/drivers/uio/Makefile
> +++ b/drivers/uio/Makefile
> @@ -8,3 +8,4 @@ obj-$(CONFIG_UIO_PCI_GENERIC) += uio_pci_generic.o
>  obj-$(CONFIG_UIO_NETX)   += uio_netx.o
>  obj-$(CONFIG_UIO_PRUSS) += uio_pruss.o
>  obj-$(CONFIG_UIO_MF624) += uio_mf624.o
> +obj-$(CONFIG_UIO_XGENE_QMTM) += uio_xgene_qmtm.o
> diff --git a/drivers/uio/uio_xgene_qmtm.c b/drivers/uio/uio_xgene_qmtm.c
> new file mode 100644
> index 000..36d9000
> --- /dev/null
> +++ b/drivers/uio/uio_xgene_qmtm.c
> @@ -0,0 +1,278 @@
> +/*
> + * X-Gene Queue Manager Traffic Manager (QMTM) UIO driver (uio_xgene_qmtm)
> + *
> + * This driver exports QMTM CSRs, Fabric and memory for queues to user-space
> + *
> + * Copyright (C) 2014 Applied Micro - http://www.apm.com/
> + * Copyright (C) 2014 Linaro Ltd.
> + *
> + * This program is free software; you can redistribute it and/or
> + * modify it under the terms of the GNU General Public License as
> + * published by the Free Software Foundation version 2.
> + *
> + * This program is distributed "as is" WITHOUT ANY WARRANTY of any
> + * kind, whether express or implied; without even the implied warranty
> + * of MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE.  See the
> + * GNU General Public License for more details.
> + */
> +
> +#include 
> +#include 
> +#include 
> +#include 
> +#include 
> +#include 
> +#include 
> +#include 
> +#include 
> +#include 
> +#include 
> +
> +#define DRV_NAME "qmtm_uio"
> +#define DRV_VERSION "1.0"
> +
> +#define QMTM_CFG_MEM_RAM_SHUTDOWN0xd070
> +
> +#define QMTM_DEFAULT_QSIZE   65536
> +
> +struct uio_qmtm_dev {
> + struct uio_info *info;
> + struct clk *qmtm_clk;
> +};
> +
> +/* QMTM CSR read/write routine */
> +static inline void qmtm_csr_write(struct uio_qmtm_dev *qmtm_dev, u32 offset,
> + u32 data)
> +{
> + void __iomem *addr = qmtm_dev->info->mem[0].internal_addr;
> +
> + writel(data, addr + offset);
> +}
> +
> +static inline u32 qmtm_csr_read(struct uio_qmtm_dev *qmtm_dev, u32 offset)
> +{
> + void __iomem *addr = qmtm_dev->info->mem[0].internal_addr;
> +
> + return readl(addr + offset);
> +}
> +
> +static int qmtm_reset(struct uio_qmtm_dev *qmtm_dev)
> +{
> + u32 val;
> + int wait = 1000;
> +
> + /* reset the internal memory of the device */
> + qmtm_csr_write(qmtm_dev, QMTM_CFG_MEM_RAM_SHUTDOWN, 0);
> +
> + /* check whether device internal memory is out of reset or not */
> + while (1) {

Seems to me that
while (wait--) {
...
}
return -EBUSY;

would be much easier to understand.

Also, not sure if EBUSY is really appropriate here.
ETIMEDOUT, maybe ?

> + val = qmtm_csr_read(qmtm_dev, QMTM_CFG_MEM_RAM_SHUTDOWN);
> +
> + if (val != 0x)
> + return 0;
> +
> + if (!wait--)
> + return -EBUSY;
> +
> + udelay(1);
> + }
> +}
> +
> +static void qmtm_cleanup(struct platform_device *pdev,
> + struct uio_qmtm_dev *qmtm_dev)
> +{
> + struct uio_info *info = qmtm_dev->info;
> +
> + uio_unregister_device(info);
> +
> + clk_disable_unprepare(qmtm_dev->qmtm_clk);
> +}
> +
> +static int qmtm_probe(struct platform_device *pdev)
> +{
> + struct uio_info *info;
> + struct uio_qmtm_dev *qmtm_dev;
> + struct resource *csr;
> + struct resource *fabric;
> + struct resource qpool;
> + unsigned int num_queues;
> + unsigned int devid;
> + phandle qpool_phandle;
> + struct device_node 

[PATCH v2 4/6] drivers: uio: Add X-Gene QMTM UIO driver

2014-09-29 Thread Ankit Jindal
The Applied Micro X-Gene SOC has on-chip QMTM (Queue manager
and Traffic manager) which is hardware based Queue or Ring
manager. This QMTM device can be used in conjunction with
other devices such as DMA Engine, Ethernet, Security Engine,
etc to assign work based on queues or rings.

This patch allows user space access to X-Gene QMTM device.

Signed-off-by: Ankit Jindal 
Signed-off-by: Tushar Jagad 
---
 drivers/uio/Kconfig  |8 ++
 drivers/uio/Makefile |1 +
 drivers/uio/uio_xgene_qmtm.c |  278 ++
 3 files changed, 287 insertions(+)
 create mode 100644 drivers/uio/uio_xgene_qmtm.c

diff --git a/drivers/uio/Kconfig b/drivers/uio/Kconfig
index 5a90914..76b1858 100644
--- a/drivers/uio/Kconfig
+++ b/drivers/uio/Kconfig
@@ -135,4 +135,12 @@ config UIO_MF624
 
  If you compile this as a module, it will be called uio_mf624.
 
+config UIO_XGENE_QMTM
+   tristate "Applied Micro X-Gene QMTM driver"
+   depends on OF
+   help
+ Userspace I/O interface for the X-Gene QMTM. The userspace part of
+ this driver will be available for download from the Applied Micro
+ web site (http://www.apm.com/).
+
 endif
diff --git a/drivers/uio/Makefile b/drivers/uio/Makefile
index d3218bd..633eaa0 100644
--- a/drivers/uio/Makefile
+++ b/drivers/uio/Makefile
@@ -8,3 +8,4 @@ obj-$(CONFIG_UIO_PCI_GENERIC)   += uio_pci_generic.o
 obj-$(CONFIG_UIO_NETX) += uio_netx.o
 obj-$(CONFIG_UIO_PRUSS) += uio_pruss.o
 obj-$(CONFIG_UIO_MF624) += uio_mf624.o
+obj-$(CONFIG_UIO_XGENE_QMTM)   += uio_xgene_qmtm.o
diff --git a/drivers/uio/uio_xgene_qmtm.c b/drivers/uio/uio_xgene_qmtm.c
new file mode 100644
index 000..36d9000
--- /dev/null
+++ b/drivers/uio/uio_xgene_qmtm.c
@@ -0,0 +1,278 @@
+/*
+ * X-Gene Queue Manager Traffic Manager (QMTM) UIO driver (uio_xgene_qmtm)
+ *
+ * This driver exports QMTM CSRs, Fabric and memory for queues to user-space
+ *
+ * Copyright (C) 2014 Applied Micro - http://www.apm.com/
+ * Copyright (C) 2014 Linaro Ltd.
+ *
+ * This program is free software; you can redistribute it and/or
+ * modify it under the terms of the GNU General Public License as
+ * published by the Free Software Foundation version 2.
+ *
+ * This program is distributed "as is" WITHOUT ANY WARRANTY of any
+ * kind, whether express or implied; without even the implied warranty
+ * of MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE.  See the
+ * GNU General Public License for more details.
+ */
+
+#include 
+#include 
+#include 
+#include 
+#include 
+#include 
+#include 
+#include 
+#include 
+#include 
+#include 
+
+#define DRV_NAME "qmtm_uio"
+#define DRV_VERSION "1.0"
+
+#define QMTM_CFG_MEM_RAM_SHUTDOWN  0xd070
+
+#define QMTM_DEFAULT_QSIZE 65536
+
+struct uio_qmtm_dev {
+   struct uio_info *info;
+   struct clk *qmtm_clk;
+};
+
+/* QMTM CSR read/write routine */
+static inline void qmtm_csr_write(struct uio_qmtm_dev *qmtm_dev, u32 offset,
+   u32 data)
+{
+   void __iomem *addr = qmtm_dev->info->mem[0].internal_addr;
+
+   writel(data, addr + offset);
+}
+
+static inline u32 qmtm_csr_read(struct uio_qmtm_dev *qmtm_dev, u32 offset)
+{
+   void __iomem *addr = qmtm_dev->info->mem[0].internal_addr;
+
+   return readl(addr + offset);
+}
+
+static int qmtm_reset(struct uio_qmtm_dev *qmtm_dev)
+{
+   u32 val;
+   int wait = 1000;
+
+   /* reset the internal memory of the device */
+   qmtm_csr_write(qmtm_dev, QMTM_CFG_MEM_RAM_SHUTDOWN, 0);
+
+   /* check whether device internal memory is out of reset or not */
+   while (1) {
+   val = qmtm_csr_read(qmtm_dev, QMTM_CFG_MEM_RAM_SHUTDOWN);
+
+   if (val != 0x)
+   return 0;
+
+   if (!wait--)
+   return -EBUSY;
+
+   udelay(1);
+   }
+}
+
+static void qmtm_cleanup(struct platform_device *pdev,
+   struct uio_qmtm_dev *qmtm_dev)
+{
+   struct uio_info *info = qmtm_dev->info;
+
+   uio_unregister_device(info);
+
+   clk_disable_unprepare(qmtm_dev->qmtm_clk);
+}
+
+static int qmtm_probe(struct platform_device *pdev)
+{
+   struct uio_info *info;
+   struct uio_qmtm_dev *qmtm_dev;
+   struct resource *csr;
+   struct resource *fabric;
+   struct resource qpool;
+   unsigned int num_queues;
+   unsigned int devid;
+   phandle qpool_phandle;
+   struct device_node *qpool_node;
+   int ret;
+
+   qmtm_dev = devm_kzalloc(>dev, sizeof(struct uio_qmtm_dev),
+   GFP_KERNEL);
+   if (!qmtm_dev)
+   return -ENOMEM;
+
+   qmtm_dev->info = devm_kzalloc(>dev, sizeof(*info), GFP_KERNEL);
+   if (!qmtm_dev->info)
+   return -ENOMEM;
+
+   /* Power on qmtm in case its not done as part of boot-loader */
+   qmtm_dev->qmtm_clk = devm_clk_get(>dev, NULL);
+   if 

Re: [PATCH v2 4/6] drivers: uio: Add X-Gene QMTM UIO driver

2014-09-29 Thread Guenter Roeck
On Tue, Sep 30, 2014 at 09:56:07AM +0530, Ankit Jindal wrote:
 The Applied Micro X-Gene SOC has on-chip QMTM (Queue manager
 and Traffic manager) which is hardware based Queue or Ring
 manager. This QMTM device can be used in conjunction with
 other devices such as DMA Engine, Ethernet, Security Engine,
 etc to assign work based on queues or rings.
 
 This patch allows user space access to X-Gene QMTM device.
 
 Signed-off-by: Ankit Jindal ankit.jin...@linaro.org
 Signed-off-by: Tushar Jagad tushar.ja...@linaro.org
 ---
  drivers/uio/Kconfig  |8 ++
  drivers/uio/Makefile |1 +
  drivers/uio/uio_xgene_qmtm.c |  278 
 ++
  3 files changed, 287 insertions(+)
  create mode 100644 drivers/uio/uio_xgene_qmtm.c
 
 diff --git a/drivers/uio/Kconfig b/drivers/uio/Kconfig
 index 5a90914..76b1858 100644
 --- a/drivers/uio/Kconfig
 +++ b/drivers/uio/Kconfig
 @@ -135,4 +135,12 @@ config UIO_MF624
  
 If you compile this as a module, it will be called uio_mf624.
  
 +config UIO_XGENE_QMTM
 + tristate Applied Micro X-Gene QMTM driver
 + depends on OF
 + help
 +   Userspace I/O interface for the X-Gene QMTM. The userspace part of
 +   this driver will be available for download from the Applied Micro
 +   web site (http://www.apm.com/).
 +
  endif
 diff --git a/drivers/uio/Makefile b/drivers/uio/Makefile
 index d3218bd..633eaa0 100644
 --- a/drivers/uio/Makefile
 +++ b/drivers/uio/Makefile
 @@ -8,3 +8,4 @@ obj-$(CONFIG_UIO_PCI_GENERIC) += uio_pci_generic.o
  obj-$(CONFIG_UIO_NETX)   += uio_netx.o
  obj-$(CONFIG_UIO_PRUSS) += uio_pruss.o
  obj-$(CONFIG_UIO_MF624) += uio_mf624.o
 +obj-$(CONFIG_UIO_XGENE_QMTM) += uio_xgene_qmtm.o
 diff --git a/drivers/uio/uio_xgene_qmtm.c b/drivers/uio/uio_xgene_qmtm.c
 new file mode 100644
 index 000..36d9000
 --- /dev/null
 +++ b/drivers/uio/uio_xgene_qmtm.c
 @@ -0,0 +1,278 @@
 +/*
 + * X-Gene Queue Manager Traffic Manager (QMTM) UIO driver (uio_xgene_qmtm)
 + *
 + * This driver exports QMTM CSRs, Fabric and memory for queues to user-space
 + *
 + * Copyright (C) 2014 Applied Micro - http://www.apm.com/
 + * Copyright (C) 2014 Linaro Ltd.
 + *
 + * This program is free software; you can redistribute it and/or
 + * modify it under the terms of the GNU General Public License as
 + * published by the Free Software Foundation version 2.
 + *
 + * This program is distributed as is WITHOUT ANY WARRANTY of any
 + * kind, whether express or implied; without even the implied warranty
 + * of MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE.  See the
 + * GNU General Public License for more details.
 + */
 +
 +#include linux/device.h
 +#include linux/delay.h
 +#include linux/module.h
 +#include linux/moduleparam.h
 +#include linux/platform_device.h
 +#include linux/uio_driver.h
 +#include linux/io.h
 +#include linux/clk.h
 +#include linux/slab.h
 +#include linux/of_platform.h
 +#include linux/of_address.h
 +
 +#define DRV_NAME qmtm_uio
 +#define DRV_VERSION 1.0
 +
 +#define QMTM_CFG_MEM_RAM_SHUTDOWN0xd070
 +
 +#define QMTM_DEFAULT_QSIZE   65536
 +
 +struct uio_qmtm_dev {
 + struct uio_info *info;
 + struct clk *qmtm_clk;
 +};
 +
 +/* QMTM CSR read/write routine */
 +static inline void qmtm_csr_write(struct uio_qmtm_dev *qmtm_dev, u32 offset,
 + u32 data)
 +{
 + void __iomem *addr = qmtm_dev-info-mem[0].internal_addr;
 +
 + writel(data, addr + offset);
 +}
 +
 +static inline u32 qmtm_csr_read(struct uio_qmtm_dev *qmtm_dev, u32 offset)
 +{
 + void __iomem *addr = qmtm_dev-info-mem[0].internal_addr;
 +
 + return readl(addr + offset);
 +}
 +
 +static int qmtm_reset(struct uio_qmtm_dev *qmtm_dev)
 +{
 + u32 val;
 + int wait = 1000;
 +
 + /* reset the internal memory of the device */
 + qmtm_csr_write(qmtm_dev, QMTM_CFG_MEM_RAM_SHUTDOWN, 0);
 +
 + /* check whether device internal memory is out of reset or not */
 + while (1) {

Seems to me that
while (wait--) {
...
}
return -EBUSY;

would be much easier to understand.

Also, not sure if EBUSY is really appropriate here.
ETIMEDOUT, maybe ?

 + val = qmtm_csr_read(qmtm_dev, QMTM_CFG_MEM_RAM_SHUTDOWN);
 +
 + if (val != 0x)
 + return 0;
 +
 + if (!wait--)
 + return -EBUSY;
 +
 + udelay(1);
 + }
 +}
 +
 +static void qmtm_cleanup(struct platform_device *pdev,
 + struct uio_qmtm_dev *qmtm_dev)
 +{
 + struct uio_info *info = qmtm_dev-info;
 +
 + uio_unregister_device(info);
 +
 + clk_disable_unprepare(qmtm_dev-qmtm_clk);
 +}
 +
 +static int qmtm_probe(struct platform_device *pdev)
 +{
 + struct uio_info *info;
 + struct uio_qmtm_dev *qmtm_dev;
 + struct resource *csr;
 + struct resource *fabric;
 + struct resource qpool;
 + unsigned int num_queues;
 + unsigned int devid;
 + 

[PATCH v2 4/6] drivers: uio: Add X-Gene QMTM UIO driver

2014-09-29 Thread Ankit Jindal
The Applied Micro X-Gene SOC has on-chip QMTM (Queue manager
and Traffic manager) which is hardware based Queue or Ring
manager. This QMTM device can be used in conjunction with
other devices such as DMA Engine, Ethernet, Security Engine,
etc to assign work based on queues or rings.

This patch allows user space access to X-Gene QMTM device.

Signed-off-by: Ankit Jindal ankit.jin...@linaro.org
Signed-off-by: Tushar Jagad tushar.ja...@linaro.org
---
 drivers/uio/Kconfig  |8 ++
 drivers/uio/Makefile |1 +
 drivers/uio/uio_xgene_qmtm.c |  278 ++
 3 files changed, 287 insertions(+)
 create mode 100644 drivers/uio/uio_xgene_qmtm.c

diff --git a/drivers/uio/Kconfig b/drivers/uio/Kconfig
index 5a90914..76b1858 100644
--- a/drivers/uio/Kconfig
+++ b/drivers/uio/Kconfig
@@ -135,4 +135,12 @@ config UIO_MF624
 
  If you compile this as a module, it will be called uio_mf624.
 
+config UIO_XGENE_QMTM
+   tristate Applied Micro X-Gene QMTM driver
+   depends on OF
+   help
+ Userspace I/O interface for the X-Gene QMTM. The userspace part of
+ this driver will be available for download from the Applied Micro
+ web site (http://www.apm.com/).
+
 endif
diff --git a/drivers/uio/Makefile b/drivers/uio/Makefile
index d3218bd..633eaa0 100644
--- a/drivers/uio/Makefile
+++ b/drivers/uio/Makefile
@@ -8,3 +8,4 @@ obj-$(CONFIG_UIO_PCI_GENERIC)   += uio_pci_generic.o
 obj-$(CONFIG_UIO_NETX) += uio_netx.o
 obj-$(CONFIG_UIO_PRUSS) += uio_pruss.o
 obj-$(CONFIG_UIO_MF624) += uio_mf624.o
+obj-$(CONFIG_UIO_XGENE_QMTM)   += uio_xgene_qmtm.o
diff --git a/drivers/uio/uio_xgene_qmtm.c b/drivers/uio/uio_xgene_qmtm.c
new file mode 100644
index 000..36d9000
--- /dev/null
+++ b/drivers/uio/uio_xgene_qmtm.c
@@ -0,0 +1,278 @@
+/*
+ * X-Gene Queue Manager Traffic Manager (QMTM) UIO driver (uio_xgene_qmtm)
+ *
+ * This driver exports QMTM CSRs, Fabric and memory for queues to user-space
+ *
+ * Copyright (C) 2014 Applied Micro - http://www.apm.com/
+ * Copyright (C) 2014 Linaro Ltd.
+ *
+ * This program is free software; you can redistribute it and/or
+ * modify it under the terms of the GNU General Public License as
+ * published by the Free Software Foundation version 2.
+ *
+ * This program is distributed as is WITHOUT ANY WARRANTY of any
+ * kind, whether express or implied; without even the implied warranty
+ * of MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE.  See the
+ * GNU General Public License for more details.
+ */
+
+#include linux/device.h
+#include linux/delay.h
+#include linux/module.h
+#include linux/moduleparam.h
+#include linux/platform_device.h
+#include linux/uio_driver.h
+#include linux/io.h
+#include linux/clk.h
+#include linux/slab.h
+#include linux/of_platform.h
+#include linux/of_address.h
+
+#define DRV_NAME qmtm_uio
+#define DRV_VERSION 1.0
+
+#define QMTM_CFG_MEM_RAM_SHUTDOWN  0xd070
+
+#define QMTM_DEFAULT_QSIZE 65536
+
+struct uio_qmtm_dev {
+   struct uio_info *info;
+   struct clk *qmtm_clk;
+};
+
+/* QMTM CSR read/write routine */
+static inline void qmtm_csr_write(struct uio_qmtm_dev *qmtm_dev, u32 offset,
+   u32 data)
+{
+   void __iomem *addr = qmtm_dev-info-mem[0].internal_addr;
+
+   writel(data, addr + offset);
+}
+
+static inline u32 qmtm_csr_read(struct uio_qmtm_dev *qmtm_dev, u32 offset)
+{
+   void __iomem *addr = qmtm_dev-info-mem[0].internal_addr;
+
+   return readl(addr + offset);
+}
+
+static int qmtm_reset(struct uio_qmtm_dev *qmtm_dev)
+{
+   u32 val;
+   int wait = 1000;
+
+   /* reset the internal memory of the device */
+   qmtm_csr_write(qmtm_dev, QMTM_CFG_MEM_RAM_SHUTDOWN, 0);
+
+   /* check whether device internal memory is out of reset or not */
+   while (1) {
+   val = qmtm_csr_read(qmtm_dev, QMTM_CFG_MEM_RAM_SHUTDOWN);
+
+   if (val != 0x)
+   return 0;
+
+   if (!wait--)
+   return -EBUSY;
+
+   udelay(1);
+   }
+}
+
+static void qmtm_cleanup(struct platform_device *pdev,
+   struct uio_qmtm_dev *qmtm_dev)
+{
+   struct uio_info *info = qmtm_dev-info;
+
+   uio_unregister_device(info);
+
+   clk_disable_unprepare(qmtm_dev-qmtm_clk);
+}
+
+static int qmtm_probe(struct platform_device *pdev)
+{
+   struct uio_info *info;
+   struct uio_qmtm_dev *qmtm_dev;
+   struct resource *csr;
+   struct resource *fabric;
+   struct resource qpool;
+   unsigned int num_queues;
+   unsigned int devid;
+   phandle qpool_phandle;
+   struct device_node *qpool_node;
+   int ret;
+
+   qmtm_dev = devm_kzalloc(pdev-dev, sizeof(struct uio_qmtm_dev),
+   GFP_KERNEL);
+   if (!qmtm_dev)
+   return -ENOMEM;
+
+   qmtm_dev-info = devm_kzalloc(pdev-dev, sizeof(*info), GFP_KERNEL);
+   if