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

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

Hash Suite: Windows password security audit tool. GUI, reports in PDF.

[<prev] [next>] [<thread-prev] [thread-next>] [day] [month] [year] [list]

Message-ID: <20080410211717.GG3193@local>
Date:	Thu, 10 Apr 2008 23:17:17 +0200
From:	"Hans J. Koch" <hjk@...utronix.de>
To:	Uwe Kleine-König <Uwe.Kleine-Koenig@...i.com>
Cc:	"Hans J. Koch" <hjk@...utronix.de>,
	Greg Kroah-Hartman <gregkh@...e.de>,
	linux-kernel@...r.kernel.org
Subject: Re: [PATCH 4/4] [RFC] UIO: generic platform driver

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@...i.com>
> > > ---
> > >  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@...r.kernel.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html
Please read the FAQ at  http://www.tux.org/lkml/