Hi Sascha,
On 07/03/16 07:52, Sascha Hauer wrote:
On Wed, Feb 24, 2016 at 07:34:19PM +0100, Matthias Brugger wrote:
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@xxxxxxxxxxxxxx>
---
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@xxxxxxxxxx>
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.
Matthias, is this some kind of action topic for me? We can use regmap
for the AUXADC driver if we want to, I have no problem with that.
No, no need for that, I will take the patch as is. I will let you know
when it got queued.
Sorry for the delay.
--
To unsubscribe from this list: send the line "unsubscribe devicetree" in
the body of a message to majordomo@xxxxxxxxxxxxxxx
More majordomo info at http://vger.kernel.org/majordomo-info.html