Re: [PATCH v1 3/3] dt-bindings: mfd: sophgo: add misc MFD for Sophgo CV1800 series SoC.

From: Jingbao Qiu
Date: Thu Jan 04 2024 - 06:43:17 EST


On Thu, Jan 4, 2024 at 7:16 PM Krzysztof Kozlowski
<krzysztof.kozlowski@xxxxxxxxxx> wrote:
>
> On 04/01/2024 12:05, Jingbao Qiu wrote:
> >>> +
> >>> +required:
> >>> + - compatible
> >>> + - reg
> >>> + - rtc
> >>> + - por
> >>> +
> >>> +unevaluatedProperties: false
> >>> +
> >>> +examples:
> >>> + - |
> >>> + #include <dt-bindings/interrupt-controller/irq.h>
> >>> +
> >>> + misc@5025000 {
> >>
> >>
> >> misc can be anything.
> >
> > Actually, there are RTC and (Power On Reset/POR) here. I can't find a suitable
> > word to describe him. Can you give me some advice?
>
> Then maybe just rtc? If there is nothing else, why RTC is separate subnode?
>

There is also a por submodule used to provide power off and restart functions.
Do you mean to use RTC as the parent node like this.
rtc{
//something
por{
}
}

Best regards,
Jingbao Qiu


> Best regards,
> Krzysztof
>