* Re: [PATCH] docs: dt: fix broken links due to txt->yaml renames
2020-05-04 9:30 [PATCH] docs: dt: fix broken links due to txt->yaml renames Mauro Carvalho Chehab
@ 2020-05-04 10:08 ` Mark Brown
2020-05-04 17:28 ` Joe Perches
` (2 subsequent siblings)
3 siblings, 0 replies; 8+ messages in thread
From: Mark Brown @ 2020-05-04 10:08 UTC (permalink / raw)
To: Mauro Carvalho Chehab
Cc: Linux Doc Mailing List, Rob Herring, linux-kernel,
Jonathan Corbet, David Airlie, Daniel Vetter, Sandy Huang,
Heiko Stübner, Sean Wang, David S. Miller, Liam Girdwood,
Arnaud Pouliquen, Thomas Bogendoerfer, Matthias Brugger,
Jyri Sarha, Andrzej Hajda, Benjamin Gaignard, Olivier Moysan,
Greg Kroah-Hartman, dri-devel, devicetree, linux-arm-kernel,
linux-rockchip, linux-bluetooth, linux-mediatek, netdev,
alsa-devel, linux-mips
[-- Attachment #1: Type: text/plain, Size: 211 bytes --]
On Mon, May 04, 2020 at 11:30:20AM +0200, Mauro Carvalho Chehab wrote:
> There are some new broken doc links due to yaml renames
> at DT. Developers should really run:
Acked-by: Mark Brown <broonie@kernel.org>
[-- Attachment #2: signature.asc --]
[-- Type: application/pgp-signature, Size: 488 bytes --]
^ permalink raw reply [flat|nested] 8+ messages in thread* Re: [PATCH] docs: dt: fix broken links due to txt->yaml renames
2020-05-04 9:30 [PATCH] docs: dt: fix broken links due to txt->yaml renames Mauro Carvalho Chehab
2020-05-04 10:08 ` Mark Brown
@ 2020-05-04 17:28 ` Joe Perches
2020-05-04 17:45 ` Sam Ravnborg
2020-05-12 22:58 ` Rob Herring
3 siblings, 0 replies; 8+ messages in thread
From: Joe Perches @ 2020-05-04 17:28 UTC (permalink / raw)
To: Mauro Carvalho Chehab, Linux Doc Mailing List, Rob Herring
Cc: linux-kernel, Jonathan Corbet, David Airlie, Daniel Vetter,
Sandy Huang, Heiko Stübner, Sean Wang, David S. Miller,
Liam Girdwood, Mark Brown, Arnaud Pouliquen, Thomas Bogendoerfer,
Matthias Brugger, Jyri Sarha, Andrzej Hajda, Benjamin Gaignard,
Olivier Moysan, Greg Kroah-Hartman, dri-devel, devicetree,
linux-arm-kernel, linux-rockchip, linux-bluetooth,
linux-mediatek, netdev, alsa-devel, linux-mips
On Mon, 2020-05-04 at 11:30 +0200, Mauro Carvalho Chehab wrote:
> There are some new broken doc links due to yaml renames
> at DT. Developers should really run:
>
> ./scripts/documentation-file-ref-check
>
> in order to solve those issues while submitting patches.
> This tool can even fix most of the issues with:
>
> ./scripts/documentation-file-ref-check --fix
Thanks Mauro.
^ permalink raw reply [flat|nested] 8+ messages in thread
* Re: [PATCH] docs: dt: fix broken links due to txt->yaml renames
2020-05-04 9:30 [PATCH] docs: dt: fix broken links due to txt->yaml renames Mauro Carvalho Chehab
2020-05-04 10:08 ` Mark Brown
2020-05-04 17:28 ` Joe Perches
@ 2020-05-04 17:45 ` Sam Ravnborg
2020-05-04 17:55 ` Uwe Kleine-König
2020-05-12 22:58 ` Rob Herring
3 siblings, 1 reply; 8+ messages in thread
From: Sam Ravnborg @ 2020-05-04 17:45 UTC (permalink / raw)
To: Mauro Carvalho Chehab
Cc: Linux Doc Mailing List, Rob Herring, alsa-devel, Olivier Moysan,
David Airlie, dri-devel, Sandy Huang, Andrzej Hajda,
Jonathan Corbet, linux-rockchip, linux-mips, devicetree,
Sean Wang, Jyri Sarha, Mark Brown, linux-mediatek,
Matthias Brugger, linux-arm-kernel, Thomas Bogendoerfer,
Greg Kroah-Hartman, Arnaud Pouliquen, Liam Girdwood,
linux-kernel, linux-bluetooth, netdev, David S. Miller
Hi Mauro.
On Mon, May 04, 2020 at 11:30:20AM +0200, Mauro Carvalho Chehab wrote:
> There are some new broken doc links due to yaml renames
> at DT. Developers should really run:
>
> ./scripts/documentation-file-ref-check
>
> in order to solve those issues while submitting patches.
Would love if some bot could do this for me on any patches that creates
.yaml files or so.
I know I will forget this and it can be automated.
If I get a bot mail that my patch would broke a link I would
have it fixed before it hits any tree.
> This tool can even fix most of the issues with:
>
> ./scripts/documentation-file-ref-check --fix
>
> Signed-off-by: Mauro Carvalho Chehab <mchehab+huawei@kernel.org>
Patch looks good.
Acked-by: Sam Ravnborg <sam@ravnborg.org>
> ---
>
> PS.: This patch is against today's linux-next.
>
>
> .../devicetree/bindings/display/bridge/sii902x.txt | 2 +-
> .../devicetree/bindings/display/rockchip/rockchip-drm.yaml | 2 +-
> .../devicetree/bindings/net/mediatek-bluetooth.txt | 2 +-
> .../devicetree/bindings/sound/audio-graph-card.txt | 2 +-
> .../devicetree/bindings/sound/st,sti-asoc-card.txt | 2 +-
> Documentation/mips/ingenic-tcu.rst | 2 +-
> MAINTAINERS | 6 +++---
> 7 files changed, 9 insertions(+), 9 deletions(-)
>
> diff --git a/Documentation/devicetree/bindings/display/bridge/sii902x.txt b/Documentation/devicetree/bindings/display/bridge/sii902x.txt
> index 6e14e087c0d0..0d1db3f9da84 100644
> --- a/Documentation/devicetree/bindings/display/bridge/sii902x.txt
> +++ b/Documentation/devicetree/bindings/display/bridge/sii902x.txt
> @@ -37,7 +37,7 @@ Optional properties:
> simple-card or audio-graph-card binding. See their binding
> documents on how to describe the way the sii902x device is
> connected to the rest of the audio system:
> - Documentation/devicetree/bindings/sound/simple-card.txt
> + Documentation/devicetree/bindings/sound/simple-card.yaml
> Documentation/devicetree/bindings/sound/audio-graph-card.txt
> Note: In case of the audio-graph-card binding the used port
> index should be 3.
> diff --git a/Documentation/devicetree/bindings/display/rockchip/rockchip-drm.yaml b/Documentation/devicetree/bindings/display/rockchip/rockchip-drm.yaml
> index ec8ae742d4da..7204da5eb4c5 100644
> --- a/Documentation/devicetree/bindings/display/rockchip/rockchip-drm.yaml
> +++ b/Documentation/devicetree/bindings/display/rockchip/rockchip-drm.yaml
> @@ -24,7 +24,7 @@ properties:
> description: |
> Should contain a list of phandles pointing to display interface port
> of vop devices. vop definitions as defined in
> - Documentation/devicetree/bindings/display/rockchip/rockchip-vop.txt
> + Documentation/devicetree/bindings/display/rockchip/rockchip-vop.yaml
>
> required:
> - compatible
> diff --git a/Documentation/devicetree/bindings/net/mediatek-bluetooth.txt b/Documentation/devicetree/bindings/net/mediatek-bluetooth.txt
> index 219bcbd0d344..9ef5bacda8c1 100644
> --- a/Documentation/devicetree/bindings/net/mediatek-bluetooth.txt
> +++ b/Documentation/devicetree/bindings/net/mediatek-bluetooth.txt
> @@ -3,7 +3,7 @@ MediaTek SoC built-in Bluetooth Devices
>
> This device is a serial attached device to BTIF device and thus it must be a
> child node of the serial node with BTIF. The dt-bindings details for BTIF
> -device can be known via Documentation/devicetree/bindings/serial/8250.txt.
> +device can be known via Documentation/devicetree/bindings/serial/8250.yaml.
>
> Required properties:
>
> diff --git a/Documentation/devicetree/bindings/sound/audio-graph-card.txt b/Documentation/devicetree/bindings/sound/audio-graph-card.txt
> index 269682619a70..d5f6919a2d69 100644
> --- a/Documentation/devicetree/bindings/sound/audio-graph-card.txt
> +++ b/Documentation/devicetree/bindings/sound/audio-graph-card.txt
> @@ -5,7 +5,7 @@ It is based on common bindings for device graphs.
> see ${LINUX}/Documentation/devicetree/bindings/graph.txt
>
> Basically, Audio Graph Card property is same as Simple Card.
> -see ${LINUX}/Documentation/devicetree/bindings/sound/simple-card.txt
> +see ${LINUX}/Documentation/devicetree/bindings/sound/simple-card.yaml
>
> Below are same as Simple-Card.
>
> diff --git a/Documentation/devicetree/bindings/sound/st,sti-asoc-card.txt b/Documentation/devicetree/bindings/sound/st,sti-asoc-card.txt
> index 4d51f3f5ea98..a6ffcdec6f6a 100644
> --- a/Documentation/devicetree/bindings/sound/st,sti-asoc-card.txt
> +++ b/Documentation/devicetree/bindings/sound/st,sti-asoc-card.txt
> @@ -5,7 +5,7 @@ codec or external codecs.
>
> sti sound drivers allows to expose sti SoC audio interface through the
> generic ASoC simple card. For details about sound card declaration please refer to
> -Documentation/devicetree/bindings/sound/simple-card.txt.
> +Documentation/devicetree/bindings/sound/simple-card.yaml.
>
> 1) sti-uniperiph-dai: audio dai device.
> ---------------------------------------
> diff --git a/Documentation/mips/ingenic-tcu.rst b/Documentation/mips/ingenic-tcu.rst
> index c5a646b14450..2b75760619b4 100644
> --- a/Documentation/mips/ingenic-tcu.rst
> +++ b/Documentation/mips/ingenic-tcu.rst
> @@ -68,4 +68,4 @@ and frameworks can be controlled from the same registers, all of these
> drivers access their registers through the same regmap.
>
> For more information regarding the devicetree bindings of the TCU drivers,
> -have a look at Documentation/devicetree/bindings/timer/ingenic,tcu.txt.
> +have a look at Documentation/devicetree/bindings/timer/ingenic,tcu.yaml.
> diff --git a/MAINTAINERS b/MAINTAINERS
> index b6ec0b3c3125..b70842425302 100644
> --- a/MAINTAINERS
> +++ b/MAINTAINERS
> @@ -3911,7 +3911,7 @@ L: linux-crypto@vger.kernel.org
> S: Supported
> F: drivers/char/hw_random/cctrng.c
> F: drivers/char/hw_random/cctrng.h
> -F: Documentation/devicetree/bindings/rng/arm-cctrng.txt
> +F: Documentation/devicetree/bindings/rng/arm-cctrng.yaml
> W: https://developer.arm.com/products/system-ip/trustzone-cryptocell/cryptocell-700-family
>
> CEC FRAMEWORK
> @@ -5446,7 +5446,7 @@ F: include/uapi/drm/r128_drm.h
> DRM DRIVER FOR RAYDIUM RM67191 PANELS
> M: Robert Chiras <robert.chiras@nxp.com>
> S: Maintained
> -F: Documentation/devicetree/bindings/display/panel/raydium,rm67191.txt
> +F: Documentation/devicetree/bindings/display/panel/raydium,rm67191.yaml
> F: drivers/gpu/drm/panel/panel-raydium-rm67191.c
>
> DRM DRIVER FOR ROCKTECH JH057N00900 PANELS
> @@ -16294,7 +16294,7 @@ M: Hoan Tran <hoan@os.amperecomputing.com>
> M: Serge Semin <fancer.lancer@gmail.com>
> L: linux-gpio@vger.kernel.org
> S: Maintained
> -F: Documentation/devicetree/bindings/gpio/snps-dwapb-gpio.txt
> +F: Documentation/devicetree/bindings/gpio/snps,dw-apb-gpio.yaml
> F: drivers/gpio/gpio-dwapb.c
>
> SYNOPSYS DESIGNWARE AXI DMAC DRIVER
> --
> 2.25.4
>
> _______________________________________________
> dri-devel mailing list
> dri-devel@lists.freedesktop.org
> https://lists.freedesktop.org/mailman/listinfo/dri-devel
^ permalink raw reply [flat|nested] 8+ messages in thread* Re: [PATCH] docs: dt: fix broken links due to txt->yaml renames
2020-05-04 17:45 ` Sam Ravnborg
@ 2020-05-04 17:55 ` Uwe Kleine-König
2020-05-04 19:04 ` Joe Perches
2020-05-05 4:07 ` Rob Herring
0 siblings, 2 replies; 8+ messages in thread
From: Uwe Kleine-König @ 2020-05-04 17:55 UTC (permalink / raw)
To: Sam Ravnborg
Cc: Mauro Carvalho Chehab, alsa-devel, Olivier Moysan,
Linux Doc Mailing List, David Airlie, dri-devel, linux-mips,
Andrzej Hajda, Jonathan Corbet, linux-rockchip, devicetree,
Liam Girdwood, Sean Wang, Jyri Sarha, Mark Brown, linux-mediatek,
Matthias Brugger, linux-arm-kernel, Thomas Bogendoerfer,
Greg Kroah-Hartman, Arnaud Pouliquen, Sandy Huang, linux-kernel,
linux-bluetooth, Rob Herring, netdev, David S. Miller
Hi Sam,
On Mon, May 04, 2020 at 07:45:22PM +0200, Sam Ravnborg wrote:
> On Mon, May 04, 2020 at 11:30:20AM +0200, Mauro Carvalho Chehab wrote:
> > There are some new broken doc links due to yaml renames
> > at DT. Developers should really run:
> >
> > ./scripts/documentation-file-ref-check
> >
> > in order to solve those issues while submitting patches.
> Would love if some bot could do this for me on any patches that creates
> .yaml files or so.
> I know I will forget this and it can be automated.
> If I get a bot mail that my patch would broke a link I would
> have it fixed before it hits any tree.
What about adding a check to check_patch?
Best regards
Uwe
--
Pengutronix e.K. | Uwe Kleine-König |
Industrial Linux Solutions | https://www.pengutronix.de/ |
^ permalink raw reply [flat|nested] 8+ messages in thread
* Re: [PATCH] docs: dt: fix broken links due to txt->yaml renames
2020-05-04 17:55 ` Uwe Kleine-König
@ 2020-05-04 19:04 ` Joe Perches
2020-05-05 4:07 ` Rob Herring
1 sibling, 0 replies; 8+ messages in thread
From: Joe Perches @ 2020-05-04 19:04 UTC (permalink / raw)
To: Uwe Kleine-König, Sam Ravnborg
Cc: Mauro Carvalho Chehab, alsa-devel, Olivier Moysan,
Linux Doc Mailing List, David Airlie, dri-devel, linux-mips,
Andrzej Hajda, Jonathan Corbet, linux-rockchip, devicetree,
Liam Girdwood, Sean Wang, Jyri Sarha, Mark Brown, linux-mediatek,
Matthias Brugger, linux-arm-kernel, Thomas Bogendoerfer,
Greg Kroah-Hartman, Arnaud Pouliquen, Sandy Huang, linux-kernel,
linux-bluetooth, Rob Herring, netdev, David S. Miller
On Mon, 2020-05-04 at 19:55 +0200, Uwe Kleine-König wrote:
> Hi Sam,
>
> On Mon, May 04, 2020 at 07:45:22PM +0200, Sam Ravnborg wrote:
> > On Mon, May 04, 2020 at 11:30:20AM +0200, Mauro Carvalho Chehab wrote:
> > > There are some new broken doc links due to yaml renames
> > > at DT. Developers should really run:
> > >
> > > ./scripts/documentation-file-ref-check
> > >
> > > in order to solve those issues while submitting patches.
> > Would love if some bot could do this for me on any patches that creates
> > .yaml files or so.
> > I know I will forget this and it can be automated.
> > If I get a bot mail that my patch would broke a link I would
> > have it fixed before it hits any tree.
>
> What about adding a check to check_patch?
There's already a checkpatch warning when a patch renames
a file without a MAINTAINERS update.
^ permalink raw reply [flat|nested] 8+ messages in thread
* Re: [PATCH] docs: dt: fix broken links due to txt->yaml renames
2020-05-04 17:55 ` Uwe Kleine-König
2020-05-04 19:04 ` Joe Perches
@ 2020-05-05 4:07 ` Rob Herring
1 sibling, 0 replies; 8+ messages in thread
From: Rob Herring @ 2020-05-05 4:07 UTC (permalink / raw)
To: Uwe Kleine-König, Sam Ravnborg
Cc: Linux-ALSA, Olivier Moysan, Linux Doc Mailing List, David Airlie,
dri-devel, open list:MIPS, Andrzej Hajda, Jonathan Corbet,
Mauro Carvalho Chehab, open list:ARM/Rockchip SoC...,
devicetree, Liam Girdwood, Sean Wang, Jyri Sarha, Mark Brown,
moderated list:ARM/Mediatek SoC support, Matthias Brugger,
moderated list:ARM/FREESCALE IMX / MXC ARM ARCHITECTURE,
Thomas Bogendoerfer, Greg Kroah-Hartman, Arnaud Pouliquen,
Sandy Huang, linux-kernel, open list:BLUETOOTH DRIVERS, netdev,
David S. Miller
On Mon, May 4, 2020 at 12:56 PM Uwe Kleine-König
<u.kleine-koenig@pengutronix.de> wrote:
>
> Hi Sam,
>
> On Mon, May 04, 2020 at 07:45:22PM +0200, Sam Ravnborg wrote:
> > On Mon, May 04, 2020 at 11:30:20AM +0200, Mauro Carvalho Chehab wrote:
> > > There are some new broken doc links due to yaml renames
> > > at DT. Developers should really run:
> > >
> > > ./scripts/documentation-file-ref-check
> > >
> > > in order to solve those issues while submitting patches.
> > Would love if some bot could do this for me on any patches that creates
> > .yaml files or so.
> > I know I will forget this and it can be automated.
> > If I get a bot mail that my patch would broke a link I would
> > have it fixed before it hits any tree.
I can probably add this to what I'm already checking. Not completely
automated though as it depends on me to review before sending.
> What about adding a check to check_patch?
That would be the best way to get submitters to do this.
Rob
^ permalink raw reply [flat|nested] 8+ messages in thread
* Re: [PATCH] docs: dt: fix broken links due to txt->yaml renames
2020-05-04 9:30 [PATCH] docs: dt: fix broken links due to txt->yaml renames Mauro Carvalho Chehab
` (2 preceding siblings ...)
2020-05-04 17:45 ` Sam Ravnborg
@ 2020-05-12 22:58 ` Rob Herring
3 siblings, 0 replies; 8+ messages in thread
From: Rob Herring @ 2020-05-12 22:58 UTC (permalink / raw)
To: Mauro Carvalho Chehab
Cc: Linux Doc Mailing List, linux-kernel, Jonathan Corbet,
David Airlie, Daniel Vetter, Sandy Huang, Heiko Stübner,
Sean Wang, David S. Miller, Liam Girdwood, Mark Brown,
Arnaud Pouliquen, Thomas Bogendoerfer, Matthias Brugger,
Jyri Sarha, Andrzej Hajda, Benjamin Gaignard, Olivier Moysan,
Greg Kroah-Hartman, dri-devel, devicetree, linux-arm-kernel,
linux-rockchip, linux-bluetooth, linux-mediatek, netdev,
alsa-devel, linux-mips
On Mon, May 04, 2020 at 11:30:20AM +0200, Mauro Carvalho Chehab wrote:
> There are some new broken doc links due to yaml renames
> at DT. Developers should really run:
>
> ./scripts/documentation-file-ref-check
>
> in order to solve those issues while submitting patches.
> This tool can even fix most of the issues with:
>
> ./scripts/documentation-file-ref-check --fix
>
> Signed-off-by: Mauro Carvalho Chehab <mchehab+huawei@kernel.org>
> ---
>
> PS.: This patch is against today's linux-next.
That's not a base anyone can apply this patch against.
>
>
> .../devicetree/bindings/display/bridge/sii902x.txt | 2 +-
> .../devicetree/bindings/display/rockchip/rockchip-drm.yaml | 2 +-
> .../devicetree/bindings/net/mediatek-bluetooth.txt | 2 +-
> .../devicetree/bindings/sound/audio-graph-card.txt | 2 +-
> .../devicetree/bindings/sound/st,sti-asoc-card.txt | 2 +-
> Documentation/mips/ingenic-tcu.rst | 2 +-
> MAINTAINERS | 6 +++---
> 7 files changed, 9 insertions(+), 9 deletions(-)
^ permalink raw reply [flat|nested] 8+ messages in thread