Groups | Search | Server Info | Keyboard shortcuts | Login | Register [http] [https] [nntp] [nntps]

Groups > linux.kernel > #1373096

Re: [PATCH 1/9] iio: adc: exynos_adc: use regmap to retrieve struct device

Show all headers | View raw

Hello,

On 2016-04-06 22:33, Alison Schofield wrote:
> On Wed, Apr 06, 2016 at 09:03:00AM +0200, Marek Szyprowski wrote:
>> Hello,
>>
>> On 2016-04-06 07:15, Alison Schofield wrote:
>>> Driver includes struct regmap and struct device in its global data.
>>> Remove the struct device and use regmap API to retrieve device info.
>>>
>>> Patch created using Coccinelle plus manual edits.
>>>
>>> Signed-off-by: Alison Schofield <amsfield22@gmail.com>
>> This patch changes the struct device which is used by the driver to report
>> errors. The driver used correctly the struct device associated with its
>> device tree node, while after the patch it will use device which is
>> associated with PMU regmap, which is a different device. PMU regmap is there
>> only to enable/disable the ADC block and it is not the regmap used to access
>> registers of the ADC device.
>>
>> I would prefer to drop this patch.
> Thanks Marek!  Please check my understanding.  Driver is not carrying
> a duplicate struct device. The regmap in exynos_adc is *not* this
> devices regmap. It belongs to the PMU, (power mgmt unit?)

Exactly.

> It seemed excessive to carry around a struct device just for the
> dev_err messages, but, we need that struct to extract the correct
> iio_dev struct. Without a regmap belonging to this actual device,
> no efficiencies can be gained in exynos, and the patch will be
> dropped from set v2.

Thanks.

> Now I need to be able to recognize such cases elsewhere. I'm going
> back though other patches in this set looking for that, but I'm not
> so sure I would recognize it.
>
> Jonathan & all, Any hints on the rule of regmap?

You may check how the regmap is initialized. When it is retrieved by
phandle from device tree, then you might be almost sure that it refers
to the different device.

Best regards
-- 
Marek Szyprowski, PhD
Samsung R&D Institute Poland

Back to linux.kernel | Previous | Next — Previous in thread | Next in thread | Find similar | Unroll thread

Thread

[PATCH 0/9] iio: use regmap to retrieve struct device Alison Schofield <amsfield22@gmail.com> - 2016-04-06 07:20 +0200
  [PATCH 1/9] iio: adc: exynos_adc: use regmap to retrieve struct  device Alison Schofield <amsfield22@gmail.com> - 2016-04-06 07:20 +0200
    Re: [PATCH 1/9] iio: adc: exynos_adc: use regmap to retrieve struct  device Marek Szyprowski <m.szyprowski@samsung.com> - 2016-04-06 09:10 +0200
      Re: [PATCH 1/9] iio: adc: exynos_adc: use regmap to retrieve struct  device Alison Schofield <amsfield22@gmail.com> - 2016-04-06 22:40 +0200
        Re: [PATCH 1/9] iio: adc: exynos_adc: use regmap to retrieve struct  device Marek Szyprowski <m.szyprowski@samsung.com> - 2016-04-07 07:40 +0200
          Re: [PATCH 1/9] iio: adc: exynos_adc: use regmap to retrieve struct  device Jonathan Cameron <jic23@kernel.org> - 2016-04-10 15:50 +0200
  [PATCH 3/9] iio: adc: qcom-spmi-vadc: use regmap to retrieve struct  device Alison Schofield <amsfield22@gmail.com> - 2016-04-06 07:20 +0200
    Re: [PATCH 3/9] iio: adc: qcom-spmi-vadc: use regmap to retrieve  struct device Jonathan Cameron <jic23@kernel.org> - 2016-04-10 16:00 +0200
  [PATCH 6/9] iio: accel: mxc4005: use regmap to retrieve struct device Alison Schofield <amsfield22@gmail.com> - 2016-04-06 07:20 +0200
  [PATCH 4/9] iio: accel: bmc150: use regmap to retrieve struct device Alison Schofield <amsfield22@gmail.com> - 2016-04-06 07:20 +0200
  [PATCH 8/9] iio: health: afe4404: use regmap to retrieve struct  device Alison Schofield <amsfield22@gmail.com> - 2016-04-06 07:30 +0200
  [PATCH 7/9] iio: health: afe4403: use regmap to retrieve struct  device Alison Schofield <amsfield22@gmail.com> - 2016-04-06 07:30 +0200
  [PATCH 9/9] iio: gyro: bmg160_core: use regmap to retrieve struct  device Alison Schofield <amsfield22@gmail.com> - 2016-04-06 07:30 +0200
  [PATCH v2 0/5] iio: use regmap to retrieve struct device Alison Schofield <amsfield22@gmail.com> - 2016-04-10 21:10 +0200
    [PATCH v2 2/5] iio: accel: mxc4005: use regmap to retrieve struct device Alison Schofield <amsfield22@gmail.com> - 2016-04-10 21:10 +0200

csiph-web