Re: [PATCH] dt-bindings: rtc: convert MT2717 RTC to the json-schema

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 



On 22.01.2024 14:19, Matthias Brugger wrote:
On 22/01/2024 13:49, Rafał Miłecki wrote:
From: Rafał Miłecki <rafal@xxxxxxxxxx>

This helps validating DTS files. Introduced changes:
1. Reworded title
2. Dropper redundant properties descriptions
3. Added required #include and adjusted "reg" in example

Signed-off-by: Rafał Miłecki <rafal@xxxxxxxxxx>
---
  .../bindings/rtc/mediatek,mt2712-rtc.yaml     | 39 +++++++++++++++++++
  .../devicetree/bindings/rtc/rtc-mt2712.txt    | 14 -------
  2 files changed, 39 insertions(+), 14 deletions(-)
  create mode 100644 Documentation/devicetree/bindings/rtc/mediatek,mt2712-rtc.yaml
  delete mode 100644 Documentation/devicetree/bindings/rtc/rtc-mt2712.txt

diff --git a/Documentation/devicetree/bindings/rtc/mediatek,mt2712-rtc.yaml b/Documentation/devicetree/bindings/rtc/mediatek,mt2712-rtc.yaml

In this schema can easily fit "mediatek,mt7622-rtc", so how about renaming it and add reference for that RTC as well?

I see your point but by looking at existing Linux drivers:
drivers/rtc/rtc-mt2712.c
drivers/rtc/rtc-mt7622.c
it seems like quite different hardware blocks.

Different registers, different programming, clk in MT7622.

Should they really share a YAML binding just because they use similar
properties?

--
Rafał Miłecki




[Index of Archives]     [Linux Sound]     [ALSA Users]     [ALSA Devel]     [Linux Audio Users]     [Linux Media]     [Kernel]     [Gimp]     [Yosemite News]     [Linux Media]

  Powered by Linux