lkml.org 
[lkml]   [2005]   [Apr]   [2]   [last100]   RSS Feed
Views: [wrap][no wrap]   [headers]  [forward] 
 
Messages in this thread
/
From
SubjectRe: [PATCH] PCI Hotplug: remove code duplication in drivers/pci/hotplug/ibmphp_pci.c
Date
Greg KH wrote:
> ChangeSet 1.2181.16.9, 2005/03/17 13:54:33-08:00, eike-hotplug@sf-tec.de
>
> [PATCH] PCI Hotplug: remove code duplication in
> drivers/pci/hotplug/ibmphp_pci.c
>
> This patch removes some code duplication where if and else have the
> same code at the beginning and the end of the branch.

Greg, as you correctly pointed out this patch if broken. It could never reach
the if branch and always uses the else branch. Please drop this one and
review the patch I sent on March 21th to pcihp-discuss for inclusion. It
removes much more duplication and handles this case correctly. Sorry, it
looks like I forgot to CC you. I'll bounce this mail to you.

Eike
[unhandled content-type:application/pgp-signature]
\
 
 \ /
  Last update: 2005-04-06 13:31    [W:0.190 / U:0.048 seconds]
©2003-2020 Jasper Spaans|hosted at Digital Ocean and TransIP|Read the blog|Advertise on this site