Received-SPF: pass (google.com: domain of linux-kernel-owner@vger.kernel.org designates 2620:137:e000::1:20 as permitted sender) client-ip=2620:137:e000::1:20;
Message-ID: <8a3a24b5-1f2f-d2bd-f2c9-2bda9298e23c@gmail.com>
Date:   Thu, 28 Jul 2022 11:23:34 +0900
MIME-Version: 1.0
User-Agent: Mozilla/5.0 (X11; Linux x86_64; rv:91.0) Gecko/20100101
 Thunderbird/91.11.0
To:     Logan Gunthorpe <logang@deltatee.com>
Cc:     corbet@lwn.net, hch@lst.de, linux-doc@vger.kernel.org,
        linux-kernel@vger.kernel.org, linux-next@vger.kernel.org,
        sfr@canb.auug.org.au
References: <838314fe-a85d-19be-71b6-264cad7e83c9@deltatee.com>
Subject: Re: linux-next: build warning after merge of the dma-mapping tree
Content-Language: en-US
From:   Akira Yokosawa <akiyks@gmail.com>
In-Reply-To: <838314fe-a85d-19be-71b6-264cad7e83c9@deltatee.com>
Content-Type: text/plain; charset=UTF-8
Content-Transfer-Encoding: 7bit
Precedence: bulk

Hi,

On Wed, 27 Jul 2022 12:32:12 -0600, Logan Gunthorpe wrote:
> On 2022-07-27 05:48, Stephen Rothwell wrote:
>> Hi all,
>> 
>> After merging the dma-mapping tree, today's linux-next build (htmldocs)
>> produced this warning:
>> 
>> Documentation/driver-api/infrastructure:50: kernel/dma/mapping.c:258: WARNING: Option list ends without a blank line; unexpected unindent.
>> 
>> Introduced by commit
>> 
>>   7c2645a2a30a ("dma-mapping: allow EREMOTEIO return code for P2PDMA transfers")
> 
> 
> Oh, hmm, I thought I addressed this when I added the extra line at the
> end of the comment, but it seems the htmldocs is a bit fussier than that
> with the white space. 
> 
> The only way I can seem to suppress that error is by adding another tab
> character in the list; see below.

I think all you need is to see the reference at:
https://docutils.sourceforge.io/docs/ref/rst/restructuredtext.html#option-lists

There are examples for long options.

        Thanks, Akira

> 
> I'll send a full patch shortly.
> 
> Logan
> 
> --
> 
> diff --git a/kernel/dma/mapping.c b/kernel/dma/mapping.c
> index 746d46825d08..99e1e0c9a295 100644
> --- a/kernel/dma/mapping.c
> +++ b/kernel/dma/mapping.c
> @@ -249,15 +249,15 @@ EXPORT_SYMBOL(dma_map_sg_attrs);
>   * Returns 0 on success or a negative error code on error. The following
>   * error codes are supported with the given meaning:
>   *
> - *   -EINVAL	An invalid argument, unaligned access or other error
> - *		in usage. Will not succeed if retried.
> - *   -ENOMEM	Insufficient resources (like memory or IOVA space) to
> - *		complete the mapping. Should succeed if retried later.
> - *   -EIO	Legacy error code with an unknown meaning. eg. this is
> - *		returned if a lower level call returned DMA_MAPPING_ERROR.
> - *   -EREMOTEIO	The DMA device cannot access P2PDMA memory specified in
> - *		the sg_table. This will not succeed if retried.
> - *
> + *   -EINVAL		An invalid argument, unaligned access or other error
> + *			in usage. Will not succeed if retried.
> + *   -ENOMEM		Insufficient resources (like memory or IOVA space) to
> + *			complete the mapping. Should succeed if retried later.
> + *   -EIO		Legacy error code with an unknown meaning. eg. this is
> + *			returned if a lower level call returned
> + *			DMA_MAPPING_ERROR.
> + *   -EREMOTEIO		The DMA device cannot access P2PDMA memory specified
> + *			in the sg_table. This will not succeed if retried.
>   */
>  int dma_map_sgtable(struct device *dev, struct sg_table *sgt,
>  		    enum dma_data_direction dir, unsigned long attrs)