linux-next.vger.kernel.org archive mirror
 help / color / mirror / Atom feed
* [PATCH] Documentation: coresight: fix `make refcheckdocs` warning
@ 2023-10-22 18:58 Vegard Nossum
  2023-10-23  4:53 ` Bagas Sanjaya
  2023-10-25 10:22 ` Suzuki K Poulose
  0 siblings, 2 replies; 3+ messages in thread
From: Vegard Nossum @ 2023-10-22 18:58 UTC (permalink / raw)
  To: Suzuki K Poulose, Mike Leach, James Clark, Leo Yan, Jonathan Corbet
  Cc: coresight, linux-arm-kernel, linux-doc, linux-kernel,
	Vegard Nossum, Mathieu Poirier, Rob Herring, linux-next,
	Stephen Rothwell, Bagas Sanjaya

This reference uses a glob pattern to match multiple files, but the
asterisk was escaped as \* in order to not be interpreted by sphinx
as reStructuredText markup.

refcheckdocs/documentation-file-ref-check doesn't know about rST syntax
and tries to interpret the \* literally (instead of as a glob).

We can work around the warning by putting the Documentation reference
inside double backticks (``..``), which allows us to not escape the
asterisk.

Fixes: c06475910b52 ("Documentation: coresight: Escape coresight bindings file wildcard")
Cc: Mathieu Poirier <mathieu.poirier@linaro.org>
Cc: Suzuki K Poulose <suzuki.poulose@arm.com>
Cc: Mike Leach <mike.leach@linaro.org>
Cc: Leo Yan <leo.yan@linaro.org>
Cc: Jonathan Corbet <corbet@lwn.net>
Cc: Rob Herring <robh@kernel.org>
Cc: coresight@lists.linaro.org
Cc: linux-arm-kernel@lists.infradead.org
Cc: linux-next@vger.kernel.org
Cc: linux-kernel@vger.kernel.org
Cc: Stephen Rothwell <sfr@canb.auug.org.au>
Cc: Bagas Sanjaya <bagasdotme@gmail.com>
Signed-off-by: Vegard Nossum <vegard.nossum@oracle.com>
---
 Documentation/trace/coresight/coresight.rst | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/Documentation/trace/coresight/coresight.rst b/Documentation/trace/coresight/coresight.rst
index 4a71ea6cb390..826e59a698da 100644
--- a/Documentation/trace/coresight/coresight.rst
+++ b/Documentation/trace/coresight/coresight.rst
@@ -130,7 +130,7 @@ Misc:
 Device Tree Bindings
 --------------------
 
-See Documentation/devicetree/bindings/arm/arm,coresight-\*.yaml for details.
+See ``Documentation/devicetree/bindings/arm/arm,coresight-*.yaml`` for details.
 
 As of this writing drivers for ITM, STMs and CTIs are not provided but are
 expected to be added as the solution matures.
-- 
2.34.1


^ permalink raw reply related	[flat|nested] 3+ messages in thread

* Re: [PATCH] Documentation: coresight: fix `make refcheckdocs` warning
  2023-10-22 18:58 [PATCH] Documentation: coresight: fix `make refcheckdocs` warning Vegard Nossum
@ 2023-10-23  4:53 ` Bagas Sanjaya
  2023-10-25 10:22 ` Suzuki K Poulose
  1 sibling, 0 replies; 3+ messages in thread
From: Bagas Sanjaya @ 2023-10-23  4:53 UTC (permalink / raw)
  To: Vegard Nossum, Suzuki K Poulose, Mike Leach, James Clark,
	Leo Yan, Jonathan Corbet
  Cc: coresight, linux-arm-kernel, linux-doc, linux-kernel,
	Mathieu Poirier, Rob Herring, linux-next, Stephen Rothwell

[-- Attachment #1: Type: text/plain, Size: 426 bytes --]

On Sun, Oct 22, 2023 at 08:58:06PM +0200, Vegard Nossum wrote:
> -See Documentation/devicetree/bindings/arm/arm,coresight-\*.yaml for details.
> +See ``Documentation/devicetree/bindings/arm/arm,coresight-*.yaml`` for details.

Oh, I should have inlined that path when I fixed the original warning. Anyway,

Reviewed-by: Bagas Sanjaya <bagasdotme@gmail.com>

-- 
An old man doll... just what I always wanted! - Clara

[-- Attachment #2: signature.asc --]
[-- Type: application/pgp-signature, Size: 228 bytes --]

^ permalink raw reply	[flat|nested] 3+ messages in thread

* Re: [PATCH] Documentation: coresight: fix `make refcheckdocs` warning
  2023-10-22 18:58 [PATCH] Documentation: coresight: fix `make refcheckdocs` warning Vegard Nossum
  2023-10-23  4:53 ` Bagas Sanjaya
@ 2023-10-25 10:22 ` Suzuki K Poulose
  1 sibling, 0 replies; 3+ messages in thread
From: Suzuki K Poulose @ 2023-10-25 10:22 UTC (permalink / raw)
  To: Vegard Nossum, James Clark, Jonathan Corbet, Mike Leach, Leo Yan
  Cc: Suzuki K Poulose, Rob Herring, linux-next, linux-arm-kernel,
	linux-doc, linux-kernel, Mathieu Poirier, coresight,
	Bagas Sanjaya, Stephen Rothwell

On Sun, 22 Oct 2023 20:58:06 +0200, Vegard Nossum wrote:
> This reference uses a glob pattern to match multiple files, but the
> asterisk was escaped as \* in order to not be interpreted by sphinx
> as reStructuredText markup.
> 
> refcheckdocs/documentation-file-ref-check doesn't know about rST syntax
> and tries to interpret the \* literally (instead of as a glob).
> 
> [...]

Applied, thanks!

[1/1] Documentation: coresight: fix `make refcheckdocs` warning
      https://git.kernel.org/coresight/c/fa55e63584f2

Best regards,
-- 
Suzuki K Poulose <suzuki.poulose@arm.com>

^ permalink raw reply	[flat|nested] 3+ messages in thread

end of thread, other threads:[~2023-10-25 10:23 UTC | newest]

Thread overview: 3+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2023-10-22 18:58 [PATCH] Documentation: coresight: fix `make refcheckdocs` warning Vegard Nossum
2023-10-23  4:53 ` Bagas Sanjaya
2023-10-25 10:22 ` Suzuki K Poulose

This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for NNTP newsgroup(s).