Re: [PATCH] Custom IORESOURCE Class

[Kumar Gala]

On Aug 8, 2005, at 6:17 PM, Adam Belay wrote:

> 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@xxxxxxxxxx>
> >
> > 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?

I would also like to understand more about what the need is here.  We  
have clock data and such but use platform_data for it.

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