The function cmos_do_probe contains the code: if (is_hpet_enabled()) { int err;
rtc_cmos_int_handler = hpet_rtc_interrupt; err = hpet_register_irq_handler(cmos_interrupt); if (err != 0) { dev_warn(dev, "hpet_register_irq_handler " " failed in rtc_init()."); goto cleanup1; } } Is it intentional that the error code returned by hpet_register_irq_handler is put ina local variable that will not be seen at label cleanup1? The return value is retval, which is 0 at this point. thanks, julia -- To unsubscribe from this list: send the line "unsubscribe linux-kernel" in the body of a message to majord...@vger.kernel.org More majordomo info at http://vger.kernel.org/majordomo-info.html Please read the FAQ at http://www.tux.org/lkml/