On Thu, 20 May 2021 21:37:34 -0500 Samuel Holland <samuel@xxxxxxxxxxxx> wrote: Hi, > On 5/19/21 5:41 AM, Andre Przywara wrote: > > Add the obvious compatible name to the existing RTC binding. > > The actual RTC part of the device uses a different day/month/year > > storage scheme, so it's not compatible with the previous devices. > > > > Signed-off-by: Andre Przywara <andre.przywara@xxxxxxx> > > --- > > .../devicetree/bindings/rtc/allwinner,sun6i-a31-rtc.yaml | 5 ++++- > > 1 file changed, 4 insertions(+), 1 deletion(-) > > > > diff --git a/Documentation/devicetree/bindings/rtc/allwinner,sun6i-a31-rtc.yaml b/Documentation/devicetree/bindings/rtc/allwinner,sun6i-a31-rtc.yaml > > index b1b0ee769b71..178c955f88bf 100644 > > --- a/Documentation/devicetree/bindings/rtc/allwinner,sun6i-a31-rtc.yaml > > +++ b/Documentation/devicetree/bindings/rtc/allwinner,sun6i-a31-rtc.yaml > > @@ -26,6 +26,7 @@ properties: > > - const: allwinner,sun50i-a64-rtc > > - const: allwinner,sun8i-h3-rtc > > - const: allwinner,sun50i-h6-rtc > > + - const: allwinner,sun50i-h616-rtc > > > > reg: > > maxItems: 1 > > @@ -97,7 +98,9 @@ allOf: > > properties: > > compatible: > > contains: > > - const: allwinner,sun50i-h6-rtc > > + enum: > > + - allwinner,sun50i-h6-rtc > > + - allwinner,sun50i-h616-rtc > > > > then: > > properties: > > > > This binding is missing a clock reference for the pll-periph0-2x input > to the 32kHz clock fanout. Right. So do I get this correctly that we don't model the OSC24M input explicitly so far in the DT? I only see one possible input clock, which is for an optional 32K crystal oscillator. And this means we need to change some code also? Because at the moment a clock specified is assumed to be the 32K OSC, and having this clock means we switch to the external 32K OSC. And who would decide which clock source to use? What would be the reason to use PLL_PERIPH(2x) over the RC16M based clock or the divided down 24MHz? So shall we ignore the PLL based input clock for now, put "0 input clocks" in the current binding, then later on extend this to allow choosing the PLL? And have it that way that having the PLL reference means we use it? > It is also missing a clock reference to the RTC register gate (and that > clock is in turn missing from the r_ccu driver). Do you mean a gate bit somewhere in the PRCM? Do you have any pointer/documentation for that? Cheers, Andre
