> if(eeprom->len == 0) return; > > > The return value should probably be zero or an error code. > Definately not undefined as it is now. Thanks. I sent a patch for this to Jeff Garzik last Friday, so it should get picked up soon. -scott - : send the line "unsubscribe linux-net" in the body of a message to majordomo@vger.kernel.org More majordomo info at http://vger.kernel.org/majordomo-info.html