mbox series

[v5,0/7] Removal of bcm63xx-wdt

Message ID 20211112224636.395101-1-f.fainelli@gmail.com
Headers show
Series Removal of bcm63xx-wdt | expand

Message

Florian Fainelli Nov. 12, 2021, 10:46 p.m. UTC
his patch series prepares the bcm7038_wdt driver to support its bcm63xx
counter part, updates the MIPS BCM63xx platform code to provide the
necessary information about the "periph" clock, and finally proceeds
with removing the bcm63xx_wdt altogether.

This was only compiled tested as I did not have a readily available
BCM63xx system to test with.

This should also help with adding support for BCM4908 which Rafal is
working on.

Changes in v5:
- removed unnecessary "bcm7038-wdt" platform devtype
- added Rob's Reviewed-by tags to the 2 binding patches

Changes in v4:
- fixed binding patch (Rob, Guenter)
- updated Kconfig description title to mention BCM63xx

Changes in v3:

- added Guenter's and Thomas' tags to patch 6
- added missing initialization of id_table
- use Rafal's latest binding patch

Changes in v2:

- added Guenter's Reviewed-by where given
- update binding patch to pass make dt_bindings_check (Rob)

Florian Fainelli (6):
  dt-bindings: watchdog: Add BCM6345 compatible to BCM7038 binding
  watchdog: bcm7038_wdt: Support platform data configuration
  watchdog: Allow building BCM7038_WDT for BCM63XX
  watchdog: bcm7038_wdt: Add platform device id for bcm63xx-wdt
  MIPS: BCM63XX: Provide platform data to watchdog device
  watchdog: Remove BCM63XX_WDT

Rafał Miłecki (1):
  dt-bindings: watchdog: convert Broadcom's WDT to the json-schema

 .../bindings/watchdog/brcm,bcm7038-wdt.txt    |  19 --
 .../bindings/watchdog/brcm,bcm7038-wdt.yaml   |  43 +++
 arch/mips/bcm63xx/dev-wdt.c                   |   8 +
 drivers/watchdog/Kconfig                      |  17 +-
 drivers/watchdog/Makefile                     |   1 -
 drivers/watchdog/bcm63xx_wdt.c                | 317 ------------------
 drivers/watchdog/bcm7038_wdt.c                |  15 +-
 include/linux/platform_data/bcm7038_wdt.h     |   8 +
 8 files changed, 77 insertions(+), 351 deletions(-)
 delete mode 100644 Documentation/devicetree/bindings/watchdog/brcm,bcm7038-wdt.txt
 create mode 100644 Documentation/devicetree/bindings/watchdog/brcm,bcm7038-wdt.yaml
 delete mode 100644 drivers/watchdog/bcm63xx_wdt.c
 create mode 100644 include/linux/platform_data/bcm7038_wdt.h

Comments

Florian Fainelli Dec. 3, 2021, 8:07 p.m. UTC | #1
On 11/12/21 2:46 PM, Florian Fainelli wrote:
> his patch series prepares the bcm7038_wdt driver to support its bcm63xx
> counter part, updates the MIPS BCM63xx platform code to provide the
> necessary information about the "periph" clock, and finally proceeds
> with removing the bcm63xx_wdt altogether.
> 
> This was only compiled tested as I did not have a readily available
> BCM63xx system to test with.
> 
> This should also help with adding support for BCM4908 which Rafal is
> working on.
> 
> Changes in v5:
> - removed unnecessary "bcm7038-wdt" platform devtype
> - added Rob's Reviewed-by tags to the 2 binding patches
> 
> Changes in v4:
> - fixed binding patch (Rob, Guenter)
> - updated Kconfig description title to mention BCM63xx
> 
> Changes in v3:
> 
> - added Guenter's and Thomas' tags to patch 6
> - added missing initialization of id_table
> - use Rafal's latest binding patch
> 
> Changes in v2:
> 
> - added Guenter's Reviewed-by where given
> - update binding patch to pass make dt_bindings_check (Rob)

Are we good with this version? If so, could it be picked up so we have
one less driver to care about, and we have one additional binding
converted to YAML?

Thanks!
Guenter Roeck Dec. 3, 2021, 8:24 p.m. UTC | #2
On 12/3/21 12:07 PM, Florian Fainelli wrote:
> On 11/12/21 2:46 PM, Florian Fainelli wrote:
>> his patch series prepares the bcm7038_wdt driver to support its bcm63xx
>> counter part, updates the MIPS BCM63xx platform code to provide the
>> necessary information about the "periph" clock, and finally proceeds
>> with removing the bcm63xx_wdt altogether.
>>
>> This was only compiled tested as I did not have a readily available
>> BCM63xx system to test with.
>>
>> This should also help with adding support for BCM4908 which Rafal is
>> working on.
>>
>> Changes in v5:
>> - removed unnecessary "bcm7038-wdt" platform devtype
>> - added Rob's Reviewed-by tags to the 2 binding patches
>>
>> Changes in v4:
>> - fixed binding patch (Rob, Guenter)
>> - updated Kconfig description title to mention BCM63xx
>>
>> Changes in v3:
>>
>> - added Guenter's and Thomas' tags to patch 6
>> - added missing initialization of id_table
>> - use Rafal's latest binding patch
>>
>> Changes in v2:
>>
>> - added Guenter's Reviewed-by where given
>> - update binding patch to pass make dt_bindings_check (Rob)
> 
> Are we good with this version? If so, could it be picked up so we have
> one less driver to care about, and we have one additional binding
> converted to YAML?
> 

It is queued in my watchdog-next branch, so Wim will hopefully
pick it up either from there or from the mailing list.

Guenter