On Mon, Jun 25, 2018 at 03:29:42PM +0800, alanx.chi...@intel.com wrote:
> From: "alanx.chiang" <alanx.chi...@intel.com>
>
> Provide a flexible way to determine the addressing bits of eeprom.
> It doesn't need to add acpi or i2c ids for specific modules.
>
> Signed-off-by: Alan Chiang <alanx.chi...@intel.com>
> Signed-off-by: Andy Yeh <andy....@intel.com>
> Reviewed-by: Sakari Ailus <sakari.ai...@linux.intel.com>
> Reviewed-by: Andy Shevchenko <andriy.shevche...@intel.com>
> Reviewed-by: Rajmohan Mani <rajmohan.m...@intel.com>
> ---
> drivers/misc/eeprom/at24.c | 16 ++++++++++++++++
> 1 file changed, 16 insertions(+)
>
> diff --git a/drivers/misc/eeprom/at24.c b/drivers/misc/eeprom/at24.c
> index 0c125f2..a6fbdae 100644
> --- a/drivers/misc/eeprom/at24.c
> +++ b/drivers/misc/eeprom/at24.c
> @@ -478,6 +478,22 @@ static void at24_properties_to_pdata(struct device *dev,
> if (device_property_present(dev, "no-read-rollover"))
> chip->flags |= AT24_FLAG_NO_RDROL;
>
> + err = device_property_read_u32(dev, "address-width", &val);
> + if (!err) {
> + switch (val) {
> + case 8:
> + chip->flags &= ~AT24_FLAG_ADDR16;
I think I'd print a warning here if the bit was set. Perhaps unlikely that
it'd happen but that'd suggest there's a problem nevertheless.
> + break;
> + case 16:
> + chip->flags |= AT24_FLAG_ADDR16;
> + break;
> + default:
> + dev_warn(dev,
> + "Bad \"address-width\" property: %u\n",
Fits on previous line.
> + val);
> + }
> + }
> +
> err = device_property_read_u32(dev, "size", &val);
> if (!err)
> chip->byte_len = val;
--
Sakari Ailus
sakari.ai...@linux.intel.com
