On 3/8/19 12:27 AM, Markus Heiser wrote:

Am 08.03.19 um 04:51 schrieb Randy Dunlap:

quoted

On 3/7/19 1:11 PM, Tobin C. Harding wrote:

quoted

Current RST file contains an unknown directive causing Sphinx to emit

    ERROR: Unexpected indentation.

Use normal language construct instead.

Signed-off-by: Tobin C. Harding <tobin@kernel.org>

This is a good idea.  My previous patch eliminated the warning but the
..hint is not presented very well in the generated output.  :)

Thanks.

quoted

---
  Documentation/driver-api/dmaengine/dmatest.rst | 4 ++--
  1 file changed, 2 insertions(+), 2 deletions(-)

diff --git a/Documentation/driver-api/dmaengine/dmatest.rst b/Documentation/driver-api/dmaengine/dmatest.rst
index 8d81f1a7169b..25eecd2769b0 100644
--- a/Documentation/driver-api/dmaengine/dmatest.rst
+++ b/Documentation/driver-api/dmaengine/dmatest.rst

@@ -59,8 +59,8 @@ parameter, that specific channel is requested using the dmaengine and a thread

  is created with the existing parameters. This thread is set as pending
  and will be executed once run is set to 1. Any parameters set after the thread
  is created are not applied.

Here a blank line is missed. Thats while '.. hint:' directive is not detected
well.  Without the blank line the '.. hint:` string is a part of the section
above.

I added a blank line and the ..hint still is not handled in any special
manner.  That's why I prefer Tobin's patch.

quoted

quoted

-.. hint::
-  available channel list could be extracted by running the following command::
+
+Hint: available channel list could be extracted by running the following command::
        % ls -1 /sys/class/dma/

  -- Markus --

-- 
~Randy