Re: [PATCH v4 1/2] dt-bindings: timer: Add bindings for Intel Keem Bay SoC Timer

From: Andy Shevchenko
Date: Wed Jul 14 2021 - 05:04:37 EST


On Tue, Jul 13, 2021 at 08:47:56PM -0600, Rob Herring wrote:
> On Mon, Jun 28, 2021 at 11:44:09AM +0530, shruthi.sanil@xxxxxxxxx wrote:

> > + The parent node represents the common general configuration details and
> > + the child nodes represents the counter and timers.
>
> I don't think all the child nodes are necessary. Are the counters and
> timers configurable (say on another SoC)? If not, then a single node
> here would suffice.

If you may notice the children may have different properties that can't be
known ahead, such as IRQ line. On some platforms it may be this mapping, on
another it maybe different.

With all respect for the simplification I think we can't do it here.

--
With Best Regards,
Andy Shevchenko