[PATCH] eeprom_93cx6: Add comment for 1us delay after pulse
authorIvo van Doorn <ivdoorn@gmail.com>
Mon, 14 May 2007 21:06:01 +0000 (23:06 +0200)
committerJeff Garzik <jeff@garzik.org>
Mon, 9 Jul 2007 02:16:36 +0000 (22:16 -0400)
This will add a comment for the 1us delay which is taken
after the pulse has been switched. The 1us delay is based
on the specifications so that should be made clear.

Signed-off-by: Ivo van Doorn <IvDoorn@gmail.com>
Signed-off-by: John W. Linville <linville@tuxdriver.com>
drivers/misc/eeprom_93cx6.c

index bfcb43424dcd66c64d6c568e82c665f3f19a42fb..0d6d742d42c351b61f30b908298c7c1dac365b57 100644 (file)
@@ -39,6 +39,12 @@ static inline void eeprom_93cx6_pulse_high(struct eeprom_93cx6 *eeprom)
 {
        eeprom->reg_data_clock = 1;
        eeprom->register_write(eeprom);
+
+       /*
+        * Add a short delay for the pulse to work.
+        * According to the specifications the minimal time
+        * should be 450ns so a 1us delay is sufficient.
+        */
        udelay(1);
 }
 
@@ -46,6 +52,12 @@ static inline void eeprom_93cx6_pulse_low(struct eeprom_93cx6 *eeprom)
 {
        eeprom->reg_data_clock = 0;
        eeprom->register_write(eeprom);
+
+       /*
+        * Add a short delay for the pulse to work.
+        * According to the specifications the minimal time
+        * should be 450ns so a 1us delay is sufficient.
+        */
        udelay(1);
 }