[lm-sensors] [PATCH] hwmon: (i5k_amb) Replace BUG with WARN

Guenter Roeck guenter.roeck at ericsson.com
Wed Aug 24 06:06:54 CEST 2011


On Tue, Aug 23, 2011 at 06:28:01PM -0400, Darrick J. Wong wrote:
> On Tue, Aug 23, 2011 at 08:05:55PM +0200, Jean Delvare wrote:
> > On Tue, 23 Aug 2011 10:14:41 -0700, Guenter Roeck wrote:
> > > On Tue, 2011-08-23 at 12:55 -0400, Darrick J. Wong wrote:
> > > > On Mon, Aug 22, 2011 at 09:01:07PM -0700, Guenter Roeck wrote:
> > > > > As I don't see a consensus for my proposed patch, I'll drop it.
> > > > 
> > > > Sorry, I'm just slow at testing things sometimes. :(
> > > > 
> > > > In general I like the idea of replacing BUG with WARN when possible,
> > > > particularly with hardware detection.  Your patch seems to load ok on all the
> > > > systems I can still claw together, so,
> > > > 
> > > > Acked-by: Darrick J. Wong <djwong at us.ibm.com>
> > > 
> > > Hi Darrick,
> > > 
> > > thanks for the feedback.
> > > 
> > > Problem is that Jean disagreed with my solution and proposed a different
> > > patch. I don't think my proposed patch is important enough to ignore or
> > > override Jean's disagreement, so I decided to drop it.
> > 
> > Darrick, any chance you could test my alternative patch? I posted it to
> > the list, see:
> > 
> > http://marc.info/?l=lm-sensors&m=131378870513822&w=2
> > 
> > I thought I had Cc'd you but I'm not sure if you received that copy.
> 
> Oops, I actually meant to ack Jean's new patch, not the old one; I think I
> replied to the wrong thread.  Serves me right for writing email before coffee.
> :/
> 

Guess that explains why you sent a second Ack to my patch ... I did wonder about that ;).

Thanks for the Ack - I applied Jean's patch.

Guenter




More information about the lm-sensors mailing list