Hello.
On 6/10/2015 11:05 AM, Ludovic Desroches wrote:
The probe() function now prints the hardware version of the I2C
controller.
Signed-off-by: Cyrille Pitchen <cyrille.pitchen@xxxxxxxxx>
---
drivers/i2c/busses/i2c-at91.c | 5 ++++-
1 file changed, 4 insertions(+), 1 deletion(-)
diff --git a/drivers/i2c/busses/i2c-at91.c b/drivers/i2c/busses/i2c-at91.c
index 817ae69..6e88b30 100644
--- a/drivers/i2c/busses/i2c-at91.c
+++ b/drivers/i2c/busses/i2c-at91.c
[...]
@@ -908,7 +910,8 @@ static int at91_twi_probe(struct platform_device *pdev)
return rc;
}
- dev_info(dev->dev, "AT91 i2c bus driver.\n");
+ dev_info(dev->dev, "AT91 i2c bus driver (version: %#x).\n",
It looks as if you rather print the driver's version. :-)
From my point of view, having a version number for a Linux driver would
be strange
Not everybody shares your opinion.
so it's not confusing.
Oh, it is, from the purely grammatical PoV. Addiung "hardware " to
"version" (or not mentioning the driver at all) would clear up that confusion.
+ at91_twi_read(dev, AT91_TWI_VER));
return 0;
}
WBR, Sergei
Regards
Ludovic
WBR, Sergei
--
To unsubscribe from this list: send the line "unsubscribe linux-i2c" in
the body of a message to majordomo@xxxxxxxxxxxxxxx
More majordomo info at http://vger.kernel.org/majordomo-info.html
