Messages in this thread | | | Date | Sat, 5 Nov 2005 08:54:40 +0100 | From | Jean Delvare <> | Subject | Re: [PATCH: 2.6.14] i2c chips: ds1337 1/2 |
| |
Hi James, Michael, all,
> Patch for ds1337 i2c driver: > > Add code to handle case where board firmware does not start the > RTC.
I understand the idea, but I don't like the implementation. Why did you add the initialization code to ds1337_set_datetime, rather than to ds1337_init_client where is seems to belong?
Also, the initialization loop is way less efficient than it could be, given the fact that the DS1337 autoincrements its address register on write. You could at least use i2c_smbus_write_byte (instead of i2c_smbus_write_byte_data), but even more efficient would be a block transfer, like the ds1337_set_datetime function use.
Care to respin a patch?
Thanks, -- Jean Delvare - To unsubscribe from this list: send the line "unsubscribe linux-kernel" in the body of a message to majordomo@vger.kernel.org More majordomo info at http://vger.kernel.org/majordomo-info.html Please read the FAQ at http://www.tux.org/lkml/
| |