| Message ID | cover.1714450308.git.zhoubinbin@loongson.cn |
|---|---|
| Headers | show |
| Series | dt-bindings: pwm: Cleanup pwm-cells required twice | expand |
On Tue, Apr 30, 2024 at 03:32:01PM +0800, Binbin Zhou wrote: > Hi all: > > When I was submitting the Loongson PWM dt-binding, Rob reminded me that > the pwm-cells property was already required in pwm.yaml and did not need > to be repeated. > > This patchset attempts to clean up the required pwm-cells attribute twice. Acked-by: Conor Dooley <conor.dooley@microchip.com> Cheers, Conor.
On Tue, Apr 30, 2024 at 2:32 AM Binbin Zhou <zhoubinbin@loongson.cn> wrote: > > Hi all: > > When I was submitting the Loongson PWM dt-binding, Rob reminded me that > the pwm-cells property was already required in pwm.yaml and did not need > to be repeated. > > This patchset attempts to clean up the required pwm-cells attribute twice. Thanks for this. > Thanks. > > Binbin Zhou (6): > dt-bindings: pwm: bcm2835: Do not require pwm-cells twice > dt-bindings: pwm: google,cros-ec: Do not require pwm-cells twice > dt-bindings: pwm: marvell,pxa: Do not require pwm-cells twice > dt-bindings: pwm: mediatek,mt2712: Do not require pwm-cells twice > dt-bindings: pwm: mediatek,pwm-disp: Do not require pwm-cells twice > dt-bindings: pwm: snps,dw-apb-timers: Do not require pwm-cells twice This could really just be one patch. It's the same repeated change and it's all going to 1 maintainer. No need to respin for that though. Rob
Hello, On Tue, Apr 30, 2024 at 03:32:01PM +0800, Binbin Zhou wrote: > When I was submitting the Loongson PWM dt-binding, Rob reminded me that > the pwm-cells property was already required in pwm.yaml and did not need > to be repeated. > > This patchset attempts to clean up the required pwm-cells attribute twice. > > Thanks. > > Binbin Zhou (6): > dt-bindings: pwm: bcm2835: Do not require pwm-cells twice > dt-bindings: pwm: google,cros-ec: Do not require pwm-cells twice > dt-bindings: pwm: marvell,pxa: Do not require pwm-cells twice > dt-bindings: pwm: mediatek,mt2712: Do not require pwm-cells twice > dt-bindings: pwm: mediatek,pwm-disp: Do not require pwm-cells twice > dt-bindings: pwm: snps,dw-apb-timers: Do not require pwm-cells twice > > Documentation/devicetree/bindings/pwm/google,cros-ec-pwm.yaml | 1 - > Documentation/devicetree/bindings/pwm/marvell,pxa-pwm.yaml | 1 - > Documentation/devicetree/bindings/pwm/mediatek,mt2712-pwm.yaml | 1 - > Documentation/devicetree/bindings/pwm/mediatek,pwm-disp.yaml | 1 - > Documentation/devicetree/bindings/pwm/pwm-bcm2835.yaml | 1 - > .../devicetree/bindings/pwm/snps,dw-apb-timers-pwm2.yaml | 1 - > 6 files changed, 6 deletions(-) Applied the series to https://git.kernel.org/pub/scm/linux/kernel/git/ukleinek/linux.git pwm/for-next Documentation/devicetree/bindings/pwm/renesas,tpu-pwm.yaml seems to be another candidate for a similar cleanup. Thanks for your cleanup, Uwe
On Wed, May 1, 2024 at 3:58 PM Uwe Kleine-König <u.kleine-koenig@pengutronix.de> wrote: > > Hello, > > On Tue, Apr 30, 2024 at 03:32:01PM +0800, Binbin Zhou wrote: > > When I was submitting the Loongson PWM dt-binding, Rob reminded me that > > the pwm-cells property was already required in pwm.yaml and did not need > > to be repeated. > > > > This patchset attempts to clean up the required pwm-cells attribute twice. > > > > Thanks. > > > > Binbin Zhou (6): > > dt-bindings: pwm: bcm2835: Do not require pwm-cells twice > > dt-bindings: pwm: google,cros-ec: Do not require pwm-cells twice > > dt-bindings: pwm: marvell,pxa: Do not require pwm-cells twice > > dt-bindings: pwm: mediatek,mt2712: Do not require pwm-cells twice > > dt-bindings: pwm: mediatek,pwm-disp: Do not require pwm-cells twice > > dt-bindings: pwm: snps,dw-apb-timers: Do not require pwm-cells twice > > > > Documentation/devicetree/bindings/pwm/google,cros-ec-pwm.yaml | 1 - > > Documentation/devicetree/bindings/pwm/marvell,pxa-pwm.yaml | 1 - > > Documentation/devicetree/bindings/pwm/mediatek,mt2712-pwm.yaml | 1 - > > Documentation/devicetree/bindings/pwm/mediatek,pwm-disp.yaml | 1 - > > Documentation/devicetree/bindings/pwm/pwm-bcm2835.yaml | 1 - > > .../devicetree/bindings/pwm/snps,dw-apb-timers-pwm2.yaml | 1 - > > 6 files changed, 6 deletions(-) > > Applied the series to > https://git.kernel.org/pub/scm/linux/kernel/git/ukleinek/linux.git pwm/for-next > > Documentation/devicetree/bindings/pwm/renesas,tpu-pwm.yaml seems to be > another candidate for a similar cleanup. Hi Uwe: Sorry for the late reply. Indeed, I missed this file. I'll post another patch later to clean it up. Thanks. Binbin > > Thanks for your cleanup, > Uwe > > -- > Pengutronix e.K. | Uwe Kleine-König | > Industrial Linux Solutions | https://www.pengutronix.de/ |