Re: [PATCH 4/4] [RFC] UIO: generic platform driver

From: Hans J. Koch
Date: Thu Apr 10 2008 - 17:17:35 EST


On Thu, Apr 10, 2008 at 10:08:19PM +0200, Uwe Kleine-KÃnig wrote:
> Hello Hans,
>
> Hans J. Koch wrote:
> > On Thu, Apr 10, 2008 at 02:37:03PM +0200, Uwe Kleine-KÃnig wrote:
> > > Signed-off-by: Uwe Kleine-KÃnig <Uwe.Kleine-Koenig@xxxxxxxx>
> > > ---
> > > drivers/uio/Kconfig | 7 ++
> > > drivers/uio/Makefile | 1 +
> > > drivers/uio/uio_pdrv.c | 165 ++++++++++++++++++++++++++++++++++++++++++++++++
> > > 3 files changed, 173 insertions(+), 0 deletions(-)
> > > create mode 100644 drivers/uio/uio_pdrv.c
> >
> > I'm a bit slow today, I don't really understand what this is good for.
> > It's to complicated to serve as a template, and it doesn't support
> > interrupts, so it's not good for any real device, too. So the only
> > usecase would be an irq-less platform_device that just needs its memory
> > mapped. Is this what you intended? What do _you_ use it for?
> In my use case I don't use an irq, but you're free to provide a handler
> when you create the platform device.

Hmm, I get the idea. You could have a UIO driver for a platform device
just by setting up a struct in the board support file. Nice thought.

Hmm. It's late, let me sleep over it... Tomorrow I'll look at it again.

Thanks,
Hans


> Attached is the file that provides
> an uio platform device. Current vanilla's support for ns9xxx isn't
> (yet) complete enough to compile that, but you should be able to see how
> it works.
>
> Best regards
> Uwe
>
> --
> Uwe Kleine-KÃnig, Software Engineer
> Digi International GmbH Branch Breisach, KÃferstrasse 8, 79206 Breisach, Germany
> Tax: 315/5781/0242 / VAT: DE153662976 / Reg. Amtsgericht Dortmund HRB 13962

> /*
> * arch/arm/mach-ns9xxx/plat-uio.c
> *
> * Copyright (C) 2008 by Digi International Inc.
> * All rights reserved.
> *
> * This program is free software; you can redistribute it and/or modify it
> * under the terms of the GNU General Public License version 2 as published by
> * the Free Software Foundation.
> */
> #include <linux/err.h>
> #include <linux/platform_device.h>
> #include <linux/uio_driver.h>
>
> #include <asm/mach-types.h>
>
> #include "clock.h"
>
> #define DRIVER_NAME "uio"
>
> struct ns9xxx_uio_platdata {
> struct uio_info uioinfo;
> struct clk clk;
> };
>
> static struct platform_device *__init ns9xxx_plat_uio_alloc(
> struct ns9xxx_uio_platdata **pdata)
> {
> struct platform_device *pdev;
> static int id;
>
> BUG_ON(!pdata);
>
> pdev = platform_device_alloc(DRIVER_NAME, id);
> if (!pdev)
> goto err;
>
> *pdata = kzalloc(sizeof(**pdata), GFP_KERNEL);
> if (!*pdata) {
> err:
> platform_device_put(pdev);
> return ERR_PTR(-ENOMEM);
> }
>
> (*pdata)->clk.owner = THIS_MODULE;
> (*pdata)->clk.name = DRIVER_NAME;
> (*pdata)->clk.id = id;
>
> pdev->dev.platform_data = *pdata;
>
> ++id;
>
> return pdev;
> }
>
> static int __init ns9xxx_plat_uio_inc20otter_init(void)
> {
> struct ns9xxx_uio_platdata *uninitialized_var(pdata);
> struct platform_device *pdev = ns9xxx_plat_uio_alloc(&pdata);
> struct resource res[] = {
> {
> .start = 0x70000000,
> .end = 0x70001fff,
> .flags = IORESOURCE_MEM,
> }, {
> .start = 0xa0700000,
> .end = 0xa070027b,
> .flags = IORESOURCE_MEM,
> }
> };
> int ret;
>
> if (IS_ERR(pdev)) {
> ret = PTR_ERR(pdev);
> pr_debug("%s: err_alloc_pdev -> %d\n", __func__, ret);
> goto err_alloc_pdev;
> }
>
> ret = platform_device_add_resources(pdev, res, ARRAY_SIZE(res));
> if (ret) {
> pr_debug("%s: err_add_resources -> %d\n", __func__, ret);
> goto err_add_resources;
> }
>
> ret = clk_register(&pdata->clk);
> if (ret) {
> pr_debug("%s: err_clk_register -> %d", __func__, ret);
> goto err_clk_register;
> }
>
> pdata->uioinfo.name = "inc20otter CS3";
> pdata->uioinfo.version = "0.0a";
> pdata->uioinfo.irq = UIO_IRQ_NONE;
>
> ret = platform_device_add(pdev);
> if (ret) {
>
> clk_unregister(&pdata->clk);
> err_clk_register:
> err_add_resources:
>
> /* this kfrees pdata, too */
> platform_device_put(pdev);
> err_alloc_pdev:
> return ret;
> }
>
> return 0;
> }
>
>
> static int __init ns9xxx_plat_uio_init(void)
> {
> if (machine_is_inc20otter())
> ns9xxx_plat_uio_inc20otter_init();
>
> return 0;
> }
>
> arch_initcall(ns9xxx_plat_uio_init);

--
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/