lkml.org 
[lkml]   [2010]   [Jul]   [15]   [last100]   RSS Feed
Views: [wrap][no wrap]   [headers]  [forward] 
 
Messages in this thread
    /
    Date
    From
    SubjectRe: tq 2440
    On Thu, Jul 15, 2010 at 06:10:29PM +0100, Russell King - ARM Linux wrote:
    > On Thu, Jul 15, 2010 at 09:51:41AM -0700, Greg KH wrote:
    > > On Thu, Jul 15, 2010 at 05:35:26PM +0100, Russell King - ARM Linux wrote:
    > > > On Thu, Jul 15, 2010 at 11:16:30PM +0800, Conke Hu wrote:
    > > > > ever notice to the following kernel log?
    > > > > "Device 's3c2440-nand' does not have a release() function, it is
    > > > > broken and must."
    > > > > release() function should be implemented in the platform_device.
    > > >
    > > > That's not telling you to provide a release function. The warning is
    > > > telling you that a device is being unregistered which doesn't have a
    > > > release function.
    > > >
    > > > Consider this point - maybe it doesn't have a release function because
    > > > it's not supposed to be unregistered?
    > >
    > > Heh, well, it is being unregistered, and at that point in time, the
    > > kernel complains.
    >
    > The unregistration occurs because platform_register_devices() (which is
    > used by arch code to register a block of platform devices) undoes its
    > work if one of the devices fails to register.
    >
    > I've long since thought, since I created that function, that this probably
    > isn't desirable behaviour, and it should continue to register as many
    > devices as it possibly can.
    >
    > > All kobjects need a release function that actually frees it. If not,
    > > that is a logic bug. Please see the Documentation/kobject.txt file for
    > > details.
    >
    > How do you free a statically declared platform device?

    You never unregister it :)

    Otherwise, you will have problems with the reference counting logic, and
    you should make it a dynamic device. I really want to move to forcing
    all devices to be created dynamically, but the platform devices are the
    biggest static device users, so I've not been pushing it very hard, due
    to other more pressing issues...

    thanks,

    greg k-h


    \
     
     \ /
      Last update: 2010-07-15 19:23    [W:4.018 / U:0.000 seconds]
    ©2003-2020 Jasper Spaans|hosted at Digital Ocean and TransIP|Read the blog|Advertise on this site