Re: [PATCH] Custom IORESOURCE Class

From: Adam Belay (ambx1_at_neo.rr.com)
Date: 08/09/05

  • Next message: Kumar Gala: "[PATCH] cpm_uart: Fix dpram allocation and non-console uarts" 
    Date:	Mon, 8 Aug 2005 23:17:14 +0000
To: Greg KH <greg@kroah.com>, Matthew Gilbert <mgilbert@mvista.com>

    On Mon, Aug 08, 2005 at 09:00:21AM -0700, Greg KH wrote:
    > On Mon, Aug 08, 2005 at 11:11:45AM -0700, Matthew Gilbert wrote:
    > > Below is a patch that adds an additional resource class to the platform
    > > resource types. This is to support additional resources that need to be passed
    > > to drivers without overloading the existing specific types. In my case, I need
    > > to send clock information to the driver to enable power management.
    > >
    > > Signed-off-by: Matthew Gilbert <mgilbert@mvista.com>
    >
    > Hm, you do realize that Pat's no longer the driver core maintainer? :)
    >
    > Anyway, Russell and Adam, any objections to this patch?

    I'm not sure if I agree with this patch. "struct resource" is used primarily for
    I/O resource assignment. Although I agree we may need to add new IORESOURCE types,
    I'm not sure if clock data belongs here. I don't think "start" and "end" would be
    useful for most platform data. Could you provide more information about this
    specific issue and resource type? Maybe we could create a new sysfs attribute?

    Thanks,
    Adam
    -
    To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
    the body of a message to majordomo@vger.kernel.org
    More majordomo info at http://vger.kernel.org/majordomo-info.html
    Please read the FAQ at http://www.tux.org/lkml/

  • Next message: Kumar Gala: "[PATCH] cpm_uart: Fix dpram allocation and non-console uarts"

    Relevant Pages