diff mbox series

[1/2] There are several eeprom drivers

Message ID 20200827152523.6cc67392@endymion
State Awaiting Upstream
Headers show
Series [1/2] There are several eeprom drivers | expand

Commit Message

Jean Delvare Aug. 27, 2020, 1:25 p.m. UTC 
There used to be only 1 eeprom driver (named "eeprom") but now
there are 3 and the legacy "eeprom" driver is not the preferred
option. So list all 3 drivers in our documentation to prevent
confusion.

Signed-off-by: Jean Delvare <jdelvare@suse.de>
---
 README           |    3 ++-
 eeprom/Module.mk |    4 ++--
 2 files changed, 4 insertions(+), 3 deletions(-)

Comments

Wolfram Sang Aug. 28, 2020, 8:43 a.m. UTC | #1 
On Thu, Aug 27, 2020 at 03:25:23PM +0200, Jean Delvare wrote:
> There used to be only 1 eeprom driver (named "eeprom") but now
> there are 3 and the legacy "eeprom" driver is not the preferred
> option. So list all 3 drivers in our documentation to prevent
> confusion.
> 
> Signed-off-by: Jean Delvare <jdelvare@suse.de>

Oh yes, that is needed!

Reviewed-by: Wolfram Sang <wsa@kernel.org>
diff mbox series

Patch

--- i2c-tools.orig/README	2020-08-27 10:28:54.972629860 +0200
+++ i2c-tools/README	2020-08-27 10:41:42.169866472 +0200
@@ -20,7 +20,8 @@  The various tools included in this packa
 
 * eeprom
   Perl scripts for decoding different types of EEPROMs (SPD, EDID...) These
-  scripts rely on the "eeprom" kernel driver. They are installed by default.
+  scripts rely on the eeprom kernel drivers ("at24" and "ee1004", "eeprom" on
+  older kernels). They are installed by default.
 
 * eeprog, eepromer
   Tools for writing to EEPROMs. These tools rely on the "i2c-dev" kernel
--- i2c-tools.orig/eeprom/Module.mk	2020-08-27 10:28:54.972629860 +0200
+++ i2c-tools/eeprom/Module.mk	2020-08-27 10:56:11.760402361 +0200
@@ -1,6 +1,6 @@ 
-# EEPROM decoding scripts for the Linux eeprom driver
+# EEPROM decoding scripts for the Linux eeprom drivers
 #
-# Copyright (C) 2007-2013  Jean Delvare <jdelvare@suse.de>
+# Copyright (C) 2007-2020  Jean Delvare <jdelvare@suse.de>
 #
 # This program is free software; you can redistribute it and/or modify
 # it under the terms of the GNU General Public License as published by