Re: fixing up DRM device model usage

2007-10-26 Thread Jesse Barnes
On Friday, October 26, 2007 10:10 am Kay Sievers wrote: > On 10/26/07, Jesse Barnes <[EMAIL PROTECTED]> wrote: > > On Thursday, October 25, 2007 9:59 pm Greg KH wrote: > > > On Thu, Oct 25, 2007 at 04:53:18PM -0700, Jesse Barnes wrote: > > > > Ok, here's yet another version that uses the device mod

Re: fixing up DRM device model usage

2007-10-26 Thread Kay Sievers
On Fri, 2007-10-26 at 11:40 -0700, Jesse Barnes wrote: > On Friday, October 26, 2007 10:10 am Kay Sievers wrote: > > On 10/26/07, Jesse Barnes <[EMAIL PROTECTED]> wrote: > > > On Thursday, October 25, 2007 9:59 pm Greg KH wrote: > > > > On Thu, Oct 25, 2007 at 04:53:18PM -0700, Jesse Barnes wrote:

Re: fixing up DRM device model usage

2007-10-26 Thread Jesse Barnes
On Friday, October 26, 2007 12:08 pm Kay Sievers wrote: > > How does this conversion look? > > Seems fine, at a first look. You moved the device structure into the > object where it belongs, instead of allocating one, and saving the > pointer. You should really considering changing the core to do t

Re: fixing up DRM device model usage

2007-10-27 Thread Kay Sievers
On Fri, 2007-10-26 at 14:31 -0700, Jesse Barnes wrote: > On Friday, October 26, 2007 12:08 pm Kay Sievers wrote: > > The open coded: device_create_file(&dev->dev, &device_attrs[i]) > > should probably replaced by passing the array to the class, and the > > core will do that for you. > > You mean