On 24/02/16 16:12, Sascha Hauer wrote:

Hi Matthias,

On Wed, Feb 24, 2016 at 03:36:46PM +0100, Matthias Brugger wrote:

quoted

On 23/02/16 21:32, Rob Herring wrote:

quoted

On Fri, Feb 19, 2016 at 08:18:46AM +0100, Sascha Hauer wrote:

quoted

Signed-off-by: Sascha Hauer <redacted>
---

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 <redacted>

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
--
To unsubscribe from this list: send the line "unsubscribe devicetree" in
the body of a message to majordomo-u79uwXL29TY76Z2rM5mHXA@public.gmane.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html