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: <eee93a4c-2629-af0f-03b3-4379a128c7dd@kernel.org>
Date:   Mon, 14 Mar 2022 07:16:04 +0100
MIME-Version: 1.0
User-Agent: Mozilla/5.0 (X11; Linux x86_64; rv:91.0) Gecko/20100101
 Thunderbird/91.6.0
Subject: Re: [PATCH] docs: serial: fix a reference file name in driver.rst
Content-Language: en-US
To:     Greg Kroah-Hartman <gregkh@linuxfoundation.org>,
        Jonathan Corbet <corbet@lwn.net>
Cc:     Wan Jiabing <wanjiabing@vivo.com>, linux-doc@vger.kernel.org,
        linux-kernel@vger.kernel.org, kael_w@qq.com
References: <20220304100315.6732-1-wanjiabing@vivo.com>
 <f48dcaba-8015-380f-ac3b-54818c48350d@kernel.org>
 <87mthw2o93.fsf@meer.lwn.net> <YixE4K3ScGL3v5yQ@kroah.com>
From:   Jiri Slaby <jirislaby@kernel.org>
In-Reply-To: <YixE4K3ScGL3v5yQ@kroah.com>
Content-Type: text/plain; charset=UTF-8; format=flowed
Content-Transfer-Encoding: 7bit
Precedence: bulk

On 12. 03. 22, 7:59, Greg Kroah-Hartman wrote:
> On Fri, Mar 11, 2022 at 02:06:00PM -0700, Jonathan Corbet wrote:
>> Jiri Slaby <jirislaby@kernel.org> writes:
>>
>>> On 04. 03. 22, 11:03, Wan Jiabing wrote:
>>>> Fix the following 'make refcheckdocs' warning:
>>>> Warning: Documentation/driver-api/serial/driver.rst references a file
>>>> that doesn't exist: Documentation/driver-api/serial/tty.rst
>>>>
>>>> Signed-off-by: Wan Jiabing <wanjiabing@vivo.com>
>>>
>>> Reviewed-by: Jiri Slaby <jirislaby@kernel.org>
>>
>> I've applied this.  But I have to wonder why Documentation/tty exists at
>> all; is there any reason not to move it all under driver-api?
> 
> No reason at all, it should probably be moved someday.

The only reason was that I don't completely understand what "driver-api" 
should contain. To be presice, documentation of line disciplines, tty 
buffer and tty internals (which is all contained in Documentation/tty) 
doesn't belong to "driver-api" IMO. If it it preferred to be there, I 
can move it, of course.

thanks,
-- 
js
suse labs