Received-SPF: pass (google.com: domain of linux-kernel-owner@vger.kernel.org designates 23.128.96.18 as permitted sender) client-ip=23.128.96.18;
Message-ID: <1fd9ed1a-edd2-a154-da1c-022a89b2c722@kernel.org>
Date:   Wed, 22 Sep 2021 08:57:17 +0200
MIME-Version: 1.0
User-Agent: Mozilla/5.0 (X11; Linux x86_64; rv:91.0) Gecko/20100101
 Thunderbird/91.1.0
Subject: Re: [PATCH 16/16] tty: drop tty_flip_buffer_push
Content-Language: en-US
To:     Johan Hovold <johan@kernel.org>
Cc:     gregkh@linuxfoundation.org, linux-serial@vger.kernel.org,
        linux-kernel@vger.kernel.org
References: <20210914091134.17426-1-jslaby@suse.cz>
 <20210914091415.17918-1-jslaby@suse.cz>
 <20210914091415.17918-9-jslaby@suse.cz>
 <YUMWaCpT4s8dQKiy@hovoldconsulting.com>
From:   Jiri Slaby <jirislaby@kernel.org>
In-Reply-To: <YUMWaCpT4s8dQKiy@hovoldconsulting.com>
Content-Type: text/plain; charset=UTF-8; format=flowed
Content-Transfer-Encoding: 7bit
Precedence: bulk

On 16. 09. 21, 12:03, Johan Hovold wrote:
> On Tue, Sep 14, 2021 at 11:14:15AM +0200, Jiri Slaby wrote:
>> Since commit a9c3f68f3cd8d (tty: Fix low_latency BUG) in 2014,
>> tty_flip_buffer_push() is only a wrapper to tty_schedule_flip(). All
>> users were converted, so remove tty_flip_buffer_push() completely.
> 
> Did you consider inlining tty_flip_buffer_push() or unexporting
> tty_schedule_flip() instead?

Yes -- I see no reason for two functions doing the very same thing. It's 
only confusing.

> The name tty_flip_buffer_push() is arguable more descriptive since the
> work may already be running and is also less tied to the implementation.
> 
> The ratio of drivers using tty_flip_buffer_push() over
> tty_schedule_flip() is also something like 186 to 15 so that would
> amount to a lot less churn too.

OK, I can do either way. I chose this path as tty_schedule_flip was a 
wrapper to tty_flip_buffer_push. In any case, I wouldn't take the number 
of changed drivers as a measure. But if it makes more sense for people 
regarding the naming, I will "flip" the two flips.

> Also, can you please start adding cover letters to your series to
> provide an overview of what it is you're trying to accomplish?

I am not a fan of cover letters as they are not Cced to people who are 
Cced in separate patches. So what would you like to see in the letter? 
This series are just a random cleanup and IMO there is not much more to 
be said except what is in their commit logs.

thanks,
-- 
js
suse labs