[PATCH v2 02/18] dt-bindings: timer: sp804: add timer-width property
narmstrong at baylibre.com
Tue Mar 22 02:21:08 PDT 2016
On 03/17/2016 08:21 PM, Robin Murphy wrote:
> On 17/03/16 19:00, Rob Herring wrote:
>> On Thu, Mar 17, 2016 at 1:06 PM, Robin Murphy <robin.murphy at arm.com> wrote:
>>> Hi Rob,
>>> On 17/03/16 17:09, Rob Herring wrote:
>>>> That would not be an SP804 nor would the vendor be ARM in that case. So
>>>> add a new compatible string for the vendor that decided to hack up ARM's
>>>> IP block.
>>> By all accounts this is some ancient reference design which later evolved
>>> _into_ the SP804, so that vendor would probably still be ARM ;)
>>> A separate compatible string would indeed make more sense, though. Both
>>> semantically and in terms of letting the driver account for the differences
>> Humm, same as integrator timers perhaps?
> Having had a quick look, what the Integrator/AP manual describes certainly smells like the same basic block as the "AMBA Timer" - 16 bit counters and the same control register layout - albeit in a mutant triple-timer version with a bigger offset between each register set. Integrator/CP, on the other hand, looks much more SP804-like.
I will switch to oxsemi,ox810se-rps-timer since it need a specific register width that will be handled by the driver.
More information about the linux-arm-kernel