lkml.org 
[lkml]   [2012]   [Sep]   [24]   [last100]   RSS Feed
Views: [wrap][no wrap]   [headers]  [forward] 
 
Messages in this thread
/
From
SubjectRE: [PATCH] Thermal: Fix bug on generic thermal framework.
Date
Hi Rui,

> > > Hi, Jonghwa,
> > >
> > > I still do not understand what the problem is.
> > > Say if a cooling device fails to bind, the thermal zone device would
> > > still work properly, just like the failure cooling device is not
> > > referenced in this thermal zone.
> > >
> > > thanks,
> > > rui
> > Hi rui,
> > No, it doesn't work properly. If it fails to bind some cool dev to
> > thermal zone device, it frees thermal zone
> > device without canceling delayed work. After freeing thermal zone
> > device, system may call work function
> > pointed NULL as the timer expired. Thus it requires skipping the
> > initialization of polling work or canceling before
> > the unregister.
>
>
> hah, I see what the problem is.
> ideally, if we fail to bind one cooling device, we should just ignore it
> and continue to bind other, what do you think?

Yes, this is what we should do.

>
> does the patch below fix your problem?
> If yes, I'll try to rebase it on top of my next tree.

This is already fixed in your -next tree, since you applied the
fair share patches 10/15. The function bind_tz(tz) does the
exact same thing, and continues.

Thanks,
Durga
\
 
 \ /
  Last update: 2012-09-25 05:41    [W:0.061 / U:1.756 seconds]
©2003-2020 Jasper Spaans|hosted at Digital Ocean and TransIP|Read the blog|Advertise on this site