Groups | Search | Server Info | Keyboard shortcuts | Login | Register [http] [https] [nntp] [nntps]
Groups > linux.kernel > #1735427 > unrolled thread
| Started by | "Mani, Rajmohan" <rajmohan.mani@intel.com> |
|---|---|
| First post | 2017-09-20 06:00 +0200 |
| Last post | 2017-09-26 07:30 +0200 |
| Articles | 3 — 3 participants |
Back to article view | Back to linux.kernel
This discussion starts older than the indexed window; earlier articles aren't shown. The article labeled Started by
below is the oldest one visible, not the original post.
RE: [PATCH v6 2/3] eeprom: at24: add support to fetch eeprom device property "size" "Mani, Rajmohan" <rajmohan.mani@intel.com> - 2017-09-20 06:00 +0200
Re: [PATCH v6 2/3] eeprom: at24: add support to fetch eeprom device property "size" Andy Shevchenko <andy.shevchenko@gmail.com> - 2017-09-22 18:20 +0200
RE: [PATCH v6 2/3] eeprom: at24: add support to fetch eeprom device property "size" "Mohandass, Divagar" <divagar.mohandass@intel.com> - 2017-09-26 07:30 +0200
| From | "Mani, Rajmohan" <rajmohan.mani@intel.com> |
|---|---|
| Date | 2017-09-20 06:00 +0200 |
| Subject | RE: [PATCH v6 2/3] eeprom: at24: add support to fetch eeprom device property "size" |
| Message-ID | <urEl3-5X7-5@gated-at.bofh.it> |
Adding Tomasz...
> -----Original Message-----
> From: Mohandass, Divagar
> Sent: Monday, September 04, 2017 3:29 AM
> To: robh+dt@kernel.org; mark.rutland@arm.com; wsa@the-dreams.de;
> sakari.ailus@iki.fi
> Cc: devicetree@vger.kernel.org; linux-i2c@vger.kernel.org; linux-
> kernel@vger.kernel.org; Mani, Rajmohan <rajmohan.mani@intel.com>;
> Mohandass, Divagar <divagar.mohandass@intel.com>
> Subject: [PATCH v6 2/3] eeprom: at24: add support to fetch eeprom device
> property "size"
>
> Obtain the size of the EEPROM chip from DT if the "size" property is specified
> for the device.
>
> Signed-off-by: Divagar Mohandass <divagar.mohandass@intel.com>
> ---
> drivers/misc/eeprom/at24.c | 4 ++++
> 1 file changed, 4 insertions(+)
>
> diff --git a/drivers/misc/eeprom/at24.c b/drivers/misc/eeprom/at24.c index
> 764ff5df..2199c42 100644
> --- a/drivers/misc/eeprom/at24.c
> +++ b/drivers/misc/eeprom/at24.c
> @@ -570,6 +570,10 @@ static void at24_get_pdata(struct device *dev, struct
> at24_platform_data *chip)
> if (device_property_present(dev, "read-only"))
> chip->flags |= AT24_FLAG_READONLY;
>
> + err = device_property_read_u32(dev, "size", &val);
> + if (!err)
> + chip->byte_len = val;
> +
> err = device_property_read_u32(dev, "pagesize", &val);
> if (!err) {
> chip->page_size = val;
> --
> 1.9.1
[toc] | [next] | [standalone]
| From | Andy Shevchenko <andy.shevchenko@gmail.com> |
|---|---|
| Date | 2017-09-22 18:20 +0200 |
| Message-ID | <usyQi-8h6-9@gated-at.bofh.it> |
| In reply to | #1735427 |
On Wed, Sep 20, 2017 at 6:52 AM, Mani, Rajmohan <rajmohan.mani@intel.com> wrote:
> Adding Tomasz...
Please, don't top post.
Better to resend entire series with all stakeholders included.
>> Obtain the size of the EEPROM chip from DT if the "size" property is specified
>> for the device.
>>
I guess it lacks
Suggested-by: Andy Shevchenko <andriy.shevchenko@linux.intel.com>
Other than that, looks good to me!
Reviewed-by: Andy Shevchenko <andy.shevchenko@gmail.com>
Wolfram, the binding is ACKed by Rob and this one is in a good shape.
Moreover, it seems like a demand for the property since some IoT stuff
would like to use the driver on ACPI enabled platforms with different
sizes.
Can you go ahead and apply first two patches? It would be really appreciated.
>> Signed-off-by: Divagar Mohandass <divagar.mohandass@intel.com>
>> ---
>> drivers/misc/eeprom/at24.c | 4 ++++
>> 1 file changed, 4 insertions(+)
>>
>> diff --git a/drivers/misc/eeprom/at24.c b/drivers/misc/eeprom/at24.c index
>> 764ff5df..2199c42 100644
>> --- a/drivers/misc/eeprom/at24.c
>> +++ b/drivers/misc/eeprom/at24.c
>> @@ -570,6 +570,10 @@ static void at24_get_pdata(struct device *dev, struct
>> at24_platform_data *chip)
>> if (device_property_present(dev, "read-only"))
>> chip->flags |= AT24_FLAG_READONLY;
>>
>> + err = device_property_read_u32(dev, "size", &val);
>> + if (!err)
>> + chip->byte_len = val;
>> +
>> err = device_property_read_u32(dev, "pagesize", &val);
>> if (!err) {
>> chip->page_size = val;
>> --
>> 1.9.1
>
--
With Best Regards,
Andy Shevchenko
[toc] | [prev] | [next] | [standalone]
| From | "Mohandass, Divagar" <divagar.mohandass@intel.com> |
|---|---|
| Date | 2017-09-26 07:30 +0200 |
| Message-ID | <utQBr-k1-5@gated-at.bofh.it> |
| In reply to | #1737606 |
Hi Andy,
Sorry missed the 'Suggested-by' tag. Can I resend the v6 with this change, so that Wolfram can pick these patches ?
---
^Divagar
>-----Original Message-----
>From: Andy Shevchenko [mailto:andy.shevchenko@gmail.com]
>Sent: Friday, September 22, 2017 9:42 PM
>To: Mani, Rajmohan <rajmohan.mani@intel.com>
>Cc: Mohandass, Divagar <divagar.mohandass@intel.com>;
>robh+dt@kernel.org; mark.rutland@arm.com; wsa@the-dreams.de;
>sakari.ailus@iki.fi; devicetree@vger.kernel.org; linux-i2c@vger.kernel.org;
>linux-kernel@vger.kernel.org; tfiga@chromium.org
>Subject: Re: [PATCH v6 2/3] eeprom: at24: add support to fetch eeprom
>device property "size"
>
>On Wed, Sep 20, 2017 at 6:52 AM, Mani, Rajmohan
><rajmohan.mani@intel.com> wrote:
>> Adding Tomasz...
>
>Please, don't top post.
>Better to resend entire series with all stakeholders included.
>
>>> Obtain the size of the EEPROM chip from DT if the "size" property is
>>> specified for the device.
>>>
>
>I guess it lacks
>Suggested-by: Andy Shevchenko <andriy.shevchenko@linux.intel.com>
>
>Other than that, looks good to me!
>
>Reviewed-by: Andy Shevchenko <andy.shevchenko@gmail.com>
>
>
>Wolfram, the binding is ACKed by Rob and this one is in a good shape.
>Moreover, it seems like a demand for the property since some IoT stuff
>would like to use the driver on ACPI enabled platforms with different sizes.
>
>Can you go ahead and apply first two patches? It would be really appreciated.
>
>>> Signed-off-by: Divagar Mohandass <divagar.mohandass@intel.com>
>>> ---
>>> drivers/misc/eeprom/at24.c | 4 ++++
>>> 1 file changed, 4 insertions(+)
>>>
>>> diff --git a/drivers/misc/eeprom/at24.c b/drivers/misc/eeprom/at24.c
>>> index
>>> 764ff5df..2199c42 100644
>>> --- a/drivers/misc/eeprom/at24.c
>>> +++ b/drivers/misc/eeprom/at24.c
>>> @@ -570,6 +570,10 @@ static void at24_get_pdata(struct device *dev,
>>> struct at24_platform_data *chip)
>>> if (device_property_present(dev, "read-only"))
>>> chip->flags |= AT24_FLAG_READONLY;
>>>
>>> + err = device_property_read_u32(dev, "size", &val);
>>> + if (!err)
>>> + chip->byte_len = val;
>>> +
>>> err = device_property_read_u32(dev, "pagesize", &val);
>>> if (!err) {
>>> chip->page_size = val;
>>> --
>>> 1.9.1
>>
>
>
>
>--
>With Best Regards,
>Andy Shevchenko
[toc] | [prev] | [standalone]
Back to top | Article view | linux.kernel
csiph-web