lkml.org 
[lkml]   [2008]   [May]   [20]   [last100]   RSS Feed
Views: [wrap][no wrap]   [headers]  [forward] 
 
Messages in this thread
    /
    Date
    From
    SubjectRe: [PATCH 00/03][RFC] Reusable UIO Platform Driver
    Hi Hans!

    On Wed, May 21, 2008 at 6:07 AM, Hans J. Koch <hjk@linutronix.de> wrote:
    > On Tue, May 20, 2008 at 07:51:32PM +0900, Magnus Damm wrote:
    >> These patches implement a reusable UIO platform driver.
    >
    > Uwe Kleine-Koenig already submitted such a framework:
    >
    > http://lkml.org/lkml/2008/5/20/94
    >
    > It's his third version, and it looks good. I presume you didn't know
    > about his work. The main difference is that he leaves interrupt handling
    > to platform code. That might look strange (it did to me first), but it
    > has the advantage that you can put hardware dependent stuff in your
    > board support (which depends on hardware anyway).

    I was not aware of this driver, thanks for the pointer!

    > Could you have a look at his patch and tell me if that does what you
    > need?

    The uio_pdrv driver doesn't do what I need at this point, though I may
    be able to extend it with the following:
    - Interrupt enable/disable code
    - Physically contiguous memory support

    The interrupt code may be placed in the board/cpu code, but I need to
    share that code between multiple UIO driver instances. We want to use
    the same UIO driver for many different processor models and hardware
    blocks. Extending uio_pdrv driver with a chunk of physically
    contiguous memory isn't a big deal though.

    To be frank, I have my doubts in adding an extra forwarding-only
    platform layer on top of UIO compared to using uio_register_device()
    directly from the board code. I like that the platform layer is using
    struct resource and handles resource ranges for us automatically, but
    wouldn't it make more sense to extend the UIO core to always use
    struct resource instead of struct uio_mem? I'd be happy to help out -
    just point me in the right direction.

    >> The interrupt handling code in uio_platform assumes the device is the
    >> only user of the assigned interrupt.
    >
    > Uwe's approach doesn't have this limitation.

    True, but the uio_pdrv driver is choosing to not deal with interrupts
    at all. I'd like to have shared interrupt handling code. With my
    driver, you just feed it io memory window parameters and an interrupt
    number and off you go. No need for any callbacks.

    >> If anyone is interested then I have a proof of concept vidix driver
    >> for mplayer that is interfacing using UIO to the VEU on a sh7722 to
    >> provide accelerated color space conversion and stretching.
    >
    > This sounds quite interesting. Unfortunately, I'm not familiar with the
    > SuperH architecture. Could you also do this with Uwe's approach?

    I could handle things by extending Uwe's uio_pdrv driver, but I still
    need the enable_irq() callback. I wonder if it makes sense to let the
    two drivers coexist side by side, since they are solving different
    problems. I can rename my driver to uio_pdrv_unique_irq or something,
    or maybe uio_superh.c. I dislike the latter since my driver doesn't do
    anything SuperH specific and that I suspect it can be useful for other
    architectures as well.

    > I'm about to sign-off Uwe's patch, and we'll possibly have that in
    > mainline soon. I don't mind having a second "generic platform" driver,
    > but you'll need to have good technical arguments.

    I'd like to have this driver upstream as well, and sharing the code
    with the uio_pdrv driver is one way, but I suspect that adding another
    reusable layer on top of that driver will just complicate things. The
    uio-specific part of my patches is less than 200 lines of code. From
    the top of my head I can think of at least 10 different SuperH
    hardware devices that can reuse this driver.

    Please let me know what you prefer and I'll update the code and repost.

    Thanks for your help!

    / magnus


    \
     
     \ /
      Last update: 2008-05-21 05:35    [W:3.617 / U:0.744 seconds]
    ©2003-2020 Jasper Spaans|hosted at Digital Ocean and TransIP|Read the blog|Advertise on this site