lkml.org 
[lkml]   [2013]   [Jul]   [31]   [last100]   RSS Feed
Views: [wrap][no wrap]   [headers]  [forward] 
 
Messages in this thread
    /
    Date
    From
    SubjectRe: [PATCH 2/2] chipidea: Use devm_request_irq()
    On Wed, Jul 31, 2013 at 10:07:58AM -0400, Tejun Heo wrote:
    > On Wed, Jul 31, 2013 at 02:57:51PM +0100, Mark Brown wrote:

    > > That's the only API I've ever heard of doing that. Everything else is
    > > just using it to do deallocation.

    > I'm not sure why or what you're trying to argue here but take a look
    > at devm_pwm_release() for example. It calls back into low level
    > driver free routine. Are you arguing that it'd be a good idea to

    That the callback is into the driver providing the PWM, not into the
    driver that's using the PWM and is releasing it.

    > release pci regions before this is complete? It's just stupid to do
    > any differently. There's nothing to argue about.

    What I'm saying is that in essentially all the users I've seen devm is
    only being used for things like kfree() or clk_put() which aren't really
    connected in any way and can happen in any order. This (coupled with
    the lack of documentation that this is supported) is why people are
    nervous about anything that relies on ordering with this stuff - aside
    from ATA everything is just using this for straight frees.
    [unhandled content-type:application/pgp-signature]
    \
     
     \ /
      Last update: 2013-08-01 05:21    [W:2.530 / U:0.564 seconds]
    ©2003-2020 Jasper Spaans|hosted at Digital Ocean and TransIP|Read the blog|Advertise on this site