
11 Jul
2012
11 Jul
'12
7:10 a.m.
Hi Stephen,
On Fri, Jun 15, 2012 at 1:17 AM, Stephen Warren swarren@wwwdotorg.orgwrote:
On 06/13/2012 10:19 AM, Simon Glass wrote:
This is a commonly-used requirement, so add a function to support it easily.
Uggh. Why would this ever be needed; shouldn't the driver for the node referenced by the phandle fully control its own registers; why would any other driver randomly trample on them?
You are making assumptions. This is used for the pwm and only accessed by the pwm driver.
It's basically to allow a simple and efficient way of getting access to a peripheral. Since the new binding you pointed me to uses a single pwm with multiple channels, this doesn't offer any benefit so I will punt it.
Regards, Simon