Jeff: The original patch was attempting to load the u-code on the 10/100 controllers that are embedded in the ICH (I/O Control Hub). The u-code patch is not designed to be loaded for these embedded controllers. Hence the failure. The patch that Malli sent out earlier explicitly checks for the controller to be part of the ICH device before attempting to load the u-code.
Ganesh. >-----Original Message----- >From: Jeff Garzik [mailto:[EMAIL PROTECTED] >Sent: Thursday, August 25, 2005 1:20 PM >To: Chilakala, Mallikarjuna >Cc: netdev; Venkatesan, Ganesh; Ronciak, John >Subject: Re: [PATCH net-drivers-2.6 5/6] e100: CPU cycle saver microcode > >Malli Chilakala wrote: >> Add cpu cycle saver microcode to 8086:{1209/1229} other than ICH devices. >> >> Signed-off-by: Mallikarjuna R Chilakala ><[EMAIL PROTECTED]> >> Signed-off-by: Ganesh Venkatesan <[EMAIL PROTECTED]> >> Signed-off-by: John Ronciak <[EMAIL PROTECTED]> > >Can you please review for me the reported problems in the CPU cycle >saver code, and how this patch fixes those problems? > > Jeff > > - To unsubscribe from this list: send the line "unsubscribe netdev" in the body of a message to [EMAIL PROTECTED] More majordomo info at http://vger.kernel.org/majordomo-info.html
