Re: [PATCH][I2C] ST M41T00 I2C RTC chip driver

2005-02-17 Thread Greg KH
On Fri, Feb 04, 2005 at 05:04:09PM -0700, Mark A. Greer wrote: > Greg KH wrote: > > >Can you resend it with a proper Changelog description in the top of the > >email and the signed-off-by line? thanks, > > > >greg k-h > > > > > > > Certainly. > -- > > This patch adds support for the ST M41T00

Re: [PATCH][I2C] ST M41T00 I2C RTC chip driver

2005-02-17 Thread Greg KH
On Fri, Feb 04, 2005 at 05:04:09PM -0700, Mark A. Greer wrote: Greg KH wrote: Can you resend it with a proper Changelog description in the top of the email and the signed-off-by line? thanks, greg k-h Certainly. -- This patch adds support for the ST M41T00 I2C RTC chip.

Re: [PATCH][I2C] ST M41T00 I2C RTC chip driver

2005-02-04 Thread Mark A. Greer
Greg KH wrote: Can you resend it with a proper Changelog description in the top of the email and the signed-off-by line? thanks, greg k-h Certainly. -- This patch adds support for the ST M41T00 I2C RTC chip. This rtc chip has no mechanism to freeze it's registers while being read; however, it

Re: [PATCH][I2C] ST M41T00 I2C RTC chip driver

2005-02-04 Thread Greg KH
On Fri, Feb 04, 2005 at 04:14:51PM -0700, Mark A. Greer wrote: > From http://archives.andrew.net.au/lm-sensors/msg29319.html: > > Mark A. Greer wrote: > > > > > > >Here is a replacement patch that should address Jean Delvare and Dick > >Johnson's issues. Please let me know if there are any

Re: [PATCH][I2C] ST M41T00 I2C RTC chip driver

2005-02-04 Thread Mark A. Greer
From http://archives.andrew.net.au/lm-sensors/msg29319.html: Mark A. Greer wrote: Here is a replacement patch that should address Jean Delvare and Dick Johnson's issues. Please let me know if there are any more issues. Thanks, Mark Signed-off-by: Mark A. Greer <[EMAIL PROTECTED]> -- I haven't

Re: [PATCH][I2C] ST M41T00 I2C RTC chip driver

2005-02-04 Thread Mark A. Greer
From http://archives.andrew.net.au/lm-sensors/msg29319.html: Mark A. Greer wrote: snip Here is a replacement patch that should address Jean Delvare and Dick Johnson's issues. Please let me know if there are any more issues. Thanks, Mark Signed-off-by: Mark A. Greer [EMAIL PROTECTED] -- I

Re: [PATCH][I2C] ST M41T00 I2C RTC chip driver

2005-02-04 Thread Greg KH
On Fri, Feb 04, 2005 at 04:14:51PM -0700, Mark A. Greer wrote: From http://archives.andrew.net.au/lm-sensors/msg29319.html: Mark A. Greer wrote: snip Here is a replacement patch that should address Jean Delvare and Dick Johnson's issues. Please let me know if there are any more

Re: [PATCH][I2C] ST M41T00 I2C RTC chip driver

2005-02-04 Thread Mark A. Greer
Greg KH wrote: Can you resend it with a proper Changelog description in the top of the email and the signed-off-by line? thanks, greg k-h Certainly. -- This patch adds support for the ST M41T00 I2C RTC chip. This rtc chip has no mechanism to freeze it's registers while being read; however, it

Re: [PATCH][I2C] ST M41T00 I2C RTC chip driver

2005-02-01 Thread Mark A. Greer
Thank you for your comments, Jean. Jean Delvare wrote: Hi Mark, + This driver can also be built as a module. If so, the module + will be called i2c-m41t00. It'll actually be called m41t00, according to the Makefile. Good catch. +struct m41t00_data { + struct i2c_client client; +};

Re: [PATCH][I2C] ST M41T00 I2C RTC chip driver

2005-02-01 Thread Mark A. Greer
Thank you for your comments, Jean. Jean Delvare wrote: Hi Mark, + This driver can also be built as a module. If so, the module + will be called i2c-m41t00. It'll actually be called m41t00, according to the Makefile. Good catch. +struct m41t00_data { + struct i2c_client client; +};

Re: [PATCH][I2C] ST M41T00 I2C RTC chip driver

2005-01-31 Thread Mark A. Greer
linux-os wrote: On ix86 machines, it is appropriate to read the RTC clock several times in a row until nothing changes. This protects against getting bad readings when some values wrap (like seconds). You can't stop the clock when you read it or you will lose time. I don't see anything like this

Re: [PATCH][I2C] ST M41T00 I2C RTC chip driver

2005-01-31 Thread Jean Delvare
Hi Mark, > This patch adds support for the ST M41T00 RTC chip. As for your other driver, I lack the device-specific knowledge to comment on the functionality, but still have some technical comments on the code itself: > + This driver can also be built as a module. If so, the module > +

Re: [PATCH][I2C] ST M41T00 I2C RTC chip driver

2005-01-31 Thread linux-os
On ix86 machines, it is appropriate to read the RTC clock several times in a row until nothing changes. This protects against getting bad readings when some values wrap (like seconds). You can't stop the clock when you read it or you will lose time. I don't see anything like this in your code.

[PATCH][I2C] ST M41T00 I2C RTC chip driver

2005-01-31 Thread Mark A. Greer
This patch adds support for the ST M41T00 RTC chip. You will likely notice that it implements a PPC-specific interface (/dev/rtc->drivers/char/genrtc.h->include/asm-ppc/rtc.h->this file). This was necessary to support a subset of ppc platforms that need to hook up the rtc support at runtime.

[PATCH][I2C] ST M41T00 I2C RTC chip driver

2005-01-31 Thread Mark A. Greer
This patch adds support for the ST M41T00 RTC chip. You will likely notice that it implements a PPC-specific interface (/dev/rtc-drivers/char/genrtc.h-include/asm-ppc/rtc.h-this file). This was necessary to support a subset of ppc platforms that need to hook up the rtc support at runtime. If

Re: [PATCH][I2C] ST M41T00 I2C RTC chip driver

2005-01-31 Thread linux-os
On ix86 machines, it is appropriate to read the RTC clock several times in a row until nothing changes. This protects against getting bad readings when some values wrap (like seconds). You can't stop the clock when you read it or you will lose time. I don't see anything like this in your code.

Re: [PATCH][I2C] ST M41T00 I2C RTC chip driver

2005-01-31 Thread Jean Delvare
Hi Mark, This patch adds support for the ST M41T00 RTC chip. As for your other driver, I lack the device-specific knowledge to comment on the functionality, but still have some technical comments on the code itself: + This driver can also be built as a module. If so, the module +

Re: [PATCH][I2C] ST M41T00 I2C RTC chip driver

2005-01-31 Thread Mark A. Greer
linux-os wrote: On ix86 machines, it is appropriate to read the RTC clock several times in a row until nothing changes. This protects against getting bad readings when some values wrap (like seconds). You can't stop the clock when you read it or you will lose time. I don't see anything like this