mbox series

[v4,0/2] ti: davinci, keystone: txt to yaml

Message ID 20240727050736.4756-1-five231003@gmail.com
Headers show
Series ti: davinci, keystone: txt to yaml | expand

Message

Kousik Sanagavarapu July 27, 2024, 5:04 a.m. UTC 
Convert txt bindings of DaVinci Timer and DaVinci/Keystone WDT
Controller to dtschema.

v3: https://lore.kernel.org/linux-devicetree/20240726150537.6873-1-five231003@gmail.com/

Changes since v3:
- No changes in 1/2.
- Explain why "power-domians" is needed in the commit msg and also add
  the Reviewed-by tag.

v2: https://lore.kernel.org/linux-devicetree/20240725153711.16101-1-five231003@gmail.com/

Changes since v2:
- Add Reviewed-by tag on 1/2.

- Change the compatible to match the DTS on 2/2 and also mention in the
  commit msg that we are adding "power-domins" as an extra optional
  property, which was missing in txt binding.

v1: https://lore.kernel.org/linux-devicetree/20240721170840.15569-1-five231003@gmail.com/

Changes vs v1:
- Change davinci timer binding's file name to match with the compatible.
  Also add "maxItems" for interrupts.

- Change the order of compatibles in wdt controller bindings touched in
  order to match DTS.

- Drop 3/3 from v1 which might effect users and should not have been
  included in this series in the first place.

Kousik Sanagavarapu (2):
  dt-bindings: timer: ti,davinci-timer: convert to dtschema
  dt-bindings: watchdog: ti,davinci-wdt: convert to dtschema

 .../bindings/timer/ti,da830-timer.yaml        | 68 +++++++++++++++++++
 .../bindings/timer/ti,davinci-timer.txt       | 37 ----------
 .../bindings/watchdog/davinci-wdt.txt         | 24 -------
 .../bindings/watchdog/ti,davinci-wdt.yaml     | 55 +++++++++++++++
 4 files changed, 123 insertions(+), 61 deletions(-)
 create mode 100644 Documentation/devicetree/bindings/timer/ti,da830-timer.yaml
 delete mode 100644 Documentation/devicetree/bindings/timer/ti,davinci-timer.txt
 delete mode 100644 Documentation/devicetree/bindings/watchdog/davinci-wdt.txt
 create mode 100644 Documentation/devicetree/bindings/watchdog/ti,davinci-wdt.yaml

Comments

Kousik Sanagavarapu Aug. 5, 2024, 7:06 a.m. UTC | #1 
On Sat, Jul 27, 2024 at 10:34:42AM +0530, Kousik Sanagavarapu wrote:
> Convert txt bindings of DaVinci Timer and DaVinci/Keystone WDT
> Controller to dtschema.
> 
> v3: https://lore.kernel.org/linux-devicetree/20240726150537.6873-1-five231003@gmail.com/
> 
> Changes since v3:
> - No changes in 1/2.
> - Explain why "power-domians" is needed in the commit msg and also add
>   the Reviewed-by tag.

Ping incase this got lost in traffic
Rob Herring Aug. 5, 2024, 2:44 p.m. UTC | #2 
On Mon, Aug 05, 2024 at 12:36:49PM +0530, Kousik Sanagavarapu wrote:
> On Sat, Jul 27, 2024 at 10:34:42AM +0530, Kousik Sanagavarapu wrote:
> > Convert txt bindings of DaVinci Timer and DaVinci/Keystone WDT
> > Controller to dtschema.
> > 
> > v3: https://lore.kernel.org/linux-devicetree/20240726150537.6873-1-five231003@gmail.com/
> > 
> > Changes since v3:
> > - No changes in 1/2.
> > - Explain why "power-domians" is needed in the commit msg and also add
> >   the Reviewed-by tag.
> 
> Ping incase this got lost in traffic

You can check the state in DT PW. It's still in my queue in case the 
timer and watchdog maintainers don't pick these up.

Note that these 2 patches are independent and go to different 
maintainers, so there was no need to make a series. No need to resend 
for that though.

Rob
Kousik Sanagavarapu Aug. 5, 2024, 3:33 p.m. UTC | #3 
On Mon, Aug 05, 2024 at 08:44:10AM -0600, Rob Herring wrote:
> Note that these 2 patches are independent and go to different 
> maintainers, so there was no need to make a series. No need to resend 
> for that though.

I'll keep that in mind from next time.

Thanks