Re: [PATCH -pm] docs: driver-api/thermal/intel_dptf: Include isonum.txt

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 



Akira Yokosawa <akiyks@xxxxxxxxx> writes:

> For the "|copy|" substitution to work, need to include isonum.txt.
>
> Fixes: 16c02447f3e1 ("Documentation: thermal: DPTF Documentation")
> Reported by: Randy Dunlap <rdunlap@xxxxxxxxxxxxx>
> Signed-off-by: Akira Yokosawa <akiyks@xxxxxxxxx>
> Cc: "Rafael J. Wysocki" <rafael@xxxxxxxxxx>
> Cc: Srinivas Pandruvada <srinivas.pandruvada@xxxxxxxxxxxxxxx>
> Cc: linux-pm@xxxxxxxxxxxxxxx
> Cc: linux-doc@xxxxxxxxxxxxxxx
> ---
> Hi,
>
> This fixes a doc build warning reported by Randy at
> https://lore.kernel.org/r/73d98132-1d08-ef29-edb7-9e062b9154b8@xxxxxxxxxxxxx/.
>
>         Thanks, Akira
> --
>  Documentation/driver-api/thermal/intel_dptf.rst | 2 ++
>  1 file changed, 2 insertions(+)
>
> diff --git a/Documentation/driver-api/thermal/intel_dptf.rst b/Documentation/driver-api/thermal/intel_dptf.rst
> index 96668dca753a..2df82719c5be 100644
> --- a/Documentation/driver-api/thermal/intel_dptf.rst
> +++ b/Documentation/driver-api/thermal/intel_dptf.rst
> @@ -1,5 +1,7 @@
>  .. SPDX-License-Identifier: GPL-2.0
>  
> +.. include:: <isonum.txt>
> +

Interesting...I don't think that was always necessary.

I have to ask, though: is this worth it?  It adds up to a fair amount of
clutter to get one symbol - © - that can also just be inserted directly
if we really think we need it.  I'd vote for doing a global s/|copy|/©/
instead.

Thanks,

jon




[Index of Archives]     [Kernel Newbies]     [Security]     [Netfilter]     [Bugtraq]     [Linux FS]     [Yosemite Forum]     [MIPS Linux]     [ARM Linux]     [Linux Security]     [Linux RAID]     [Samba]     [Video 4 Linux]     [Device Mapper]     [Linux Resources]

  Powered by Linux