Re: [PATCH 1/3] dt-bindings: rtc: Add Apple SMC RTC

[Sven Peter <sven@kernel.org>]

On 14.08.25 23:19, Rob Herring wrote:
> On Wed, Aug 13, 2025 at 08:14:51AM +0200, Krzysztof Kozlowski wrote:
>> On 12/08/2025 20:25, Sven Peter wrote:
>>> +maintainers:
>>> +  - Sven Peter <sven@kernel.org>
>>> +
>>> +properties:
>>> +  compatible:
>>> +    const: apple,smc-rtc
>>> +
>>
>> No resources except nvmem? This should be folded into the parent. Don't
>> create device node to instantiate drivers.
> 
> Well, the reboot node has nvmem entries too, so probably better to keep
> this as child node.
> 
> Is there more functionality planned/needed here (for the mfd)? If so,
> please send it all at once. One child node at a time makes DT
> maintainers grumpy.

I'd very much like to avoid that!

There's one more driver (hwmon) that needs another child node and two 
more sub-devices that don't need any or expose any resources (hid for 
the power button and power/supply for batteries). I've talked to James
(who wrote hwmon) and he's going to submit the hwmon series together 
with at least rtc (and hid) so that you can see all additional bindings 
at once.


Thanks,


Sven