[PATCH] dt-bindings: MediaTek: Add binding document for the AUXADC
Matthias Brugger
matthias.bgg at gmail.com
Wed Feb 24 10:34:19 PST 2016
On 24/02/16 16:12, Sascha Hauer wrote:
> Hi Matthias,
>
> On Wed, Feb 24, 2016 at 03:36:46PM +0100, Matthias Brugger wrote:
>>
>>
>> On 23/02/16 21:32, Rob Herring wrote:
>>> On Fri, Feb 19, 2016 at 08:18:46AM +0100, Sascha Hauer wrote:
>>>> Signed-off-by: Sascha Hauer <s.hauer at pengutronix.de>
>>>> ---
>>>>
>>>> This is a supplement to the MTK Thermal series which was lacking the
>>>> description for the AUXADC.
>>>
>>> So it will be applied with that I presume.
>>>
>>> Acked-by: Rob Herring <robh at kernel.org>
>>>
>>
>> Well the basic problem that I see is, that the auxadc has some more
>> registers (at least in mt6589) for the touch screen registers.
>> So this makes it a candidate for syscon and regmap.
>> But this needs a patch in the thermal driver first...
>
> When we want to have a driver for the auxadc we are free to write one.
> No need to use syscon and/or regmap for it.
>
> Note that the auxadc is not used by the thermal *driver*, it's used by
> the thermal *hardware*. It's the hardware that directly reads/writes
> auxadc registers. If MTK did it right then a real auxadc driver should
> not be disturbed by the thermal controller accesses. At least from
Right, didn't realize this.
> checking the datasheet the registers the thermal controller accesses
> are completely orthogonal to the ones a auxadc driver would use.
>
As you already explained, syscon/regmap is not needed here. Just for
completion: we already had completely orthogonal registers for the reset
controller, and we decided to use regmap there as well.
We should stay consistent on this.
Thanks,
Matthias
More information about the Linux-mediatek
mailing list