On 03/11/2022 12:48, Marek Vasut wrote: > On 11/3/22 14:40, Krzysztof Kozlowski wrote: > > [...] > >>> diff --git a/Documentation/devicetree/bindings/rtc/rtc-m41t80.yaml b/Documentation/devicetree/bindings/rtc/rtc-m41t80.yaml >>> new file mode 100644 >>> index 0000000000000..7f31a22caec9b >>> --- /dev/null >>> +++ b/Documentation/devicetree/bindings/rtc/rtc-m41t80.yaml >> >> Filename based on compatibles (more or less), so could be: >> st,m41t80.yaml >> or: >> st,m41txx.yaml >> >> The "rtc" name can be skipped as it is implied from subsystem folder, >> unless st,m41txx is a name of some SoC? > > It's dedicated I2C RTC, a discrete soic8 chip with optional coin cell > battery. > > We cannot really call it m41txx because there are other m41txx chips > with different register layouts and thus different RTC drivers: > > next$ ls -1 drivers/rtc/*m41t* > drivers/rtc/rtc-m41t80.c > drivers/rtc/rtc-m41t93.c > drivers/rtc/rtc-m41t94.c > > The m41t80 is a sane choice, the m41t80 was the first supported, the > other chips are just compatible to it. > > Whether I should drop the rtc- prefix or not, I will defer that > preference to Alexandre . so st,m41t80.yaml Best regards, Krzysztof
