Re: [PATCH] bcm43xx: Add error checking in bcm43xx_sprom_write()

2006-11-06 Thread Michael Buesch
On Monday 06 November 2006 16:48, Larry Finger wrote: > From: Adrian Bunk <[EMAIL PROTECTED]> > > The Coverity checker noted that these "if (err)"'s couldn't ever be > true. > > It seems the intention was to check the return values of the > bcm43xx_pci_write_config32()'s? Whoops, I thought I h

[PATCH] bcm43xx: Add error checking in bcm43xx_sprom_write()

2006-11-06 Thread Larry Finger
From: Adrian Bunk <[EMAIL PROTECTED]> The Coverity checker noted that these "if (err)"'s couldn't ever be true. It seems the intention was to check the return values of the bcm43xx_pci_write_config32()'s? Signed-off-by: Adrian Bunk <[EMAIL PROTECTED]> Signed-off-by: Larry Finger <[EMAIL PROTEC