Received-SPF: pass (google.com: domain of linux-kernel-owner@vger.kernel.org designates 2620:137:e000::3:2 as permitted sender) client-ip=2620:137:e000::3:2;
Subject: Re: [PATCH v2 2/7] serial: amba-pl011: get rid of useless wrapper
 pl011_get_rs485_mode()
To:     Greg KH <gregkh@linuxfoundation.org>,
        Lino Sanfilippo <l.sanfilippo@kunbus.com>
Cc:     jirislaby@kernel.org, ilpo.jarvinen@linux.intel.com,
        shawnguo@kernel.org, s.hauer@pengutronix.de,
        mcoquelin.stm32@gmail.com, alexandre.torgue@foss.st.com,
        cniedermaier@dh-electronics.com, linux-kernel@vger.kernel.org,
        linux-serial@vger.kernel.org, lukas@wunner.de,
        p.rosenberger@kunbus.com
References: <20231011173829.10508-1-l.sanfilippo@kunbus.com>
 <20231011173829.10508-3-l.sanfilippo@kunbus.com>
 <2023101134-theater-oversleep-a58b@gregkh>
From:   Lino Sanfilippo <LinoSanfilippo@gmx.de>
Message-ID: <20d22e11-b9be-b547-ec4e-9964a57924a4@gmx.de>
Date:   Wed, 11 Oct 2023 19:44:51 +0200
User-Agent: Mozilla/5.0 (X11; Linux x86_64; rv:68.0) Gecko/20100101
 Thunderbird/68.10.0
MIME-Version: 1.0
In-Reply-To: <2023101134-theater-oversleep-a58b@gregkh>
Content-Type: text/plain; charset=utf-8
Content-Language: en-US
Content-Transfer-Encoding: quoted-printable
UI-OutboundReport: notjunk:1;M01:P0:Hjmy0vdrhM4=;lCtslM8a4Xh0e5FLhh4FRudlg9E
 2zL5H5QMHijoG5dK8WLTjDyQSoUeJpH58O+7bB3e2zGZ1myHc9AIJKYAZ8p+5fNdw/R2PzZid
 ael+5bhm3jShdm/YUGh5jaSii9FayOjBUfSxFLe6SkiVeK6MlCq94NRs3cKdq+mQaG6zZC2o7
 El/soumIB33UexP0SMkUygZHGiv/KIwsWfV3pjb7++uHL5RiUu07lwCbIX48t9FiIA8JW7hhe
 6uyqKkNYQTJhk3tuZrDsufytNjaJQYY5JetBGAJu0oa4HrxdQIYtgS0F3aW7Ee/luc95MYCMX
 mL4kD7bv7zgD+/1cov6ZxeS+Hqlia08zaq66CTZ3+38dnSvOI7WrN5Rh0gSpCw9MVID8dDmAM
 ih9cKR470sgjLshr3PASsofc15yywYg9W1Pwfql1Plz8HV64/Bb249mC7CGSjLcj3uJdhIZP+
 yvHS7LTnZDvXGyYlq4aLVDS2O0jfUTuJoTYfE2pxQi5oF+zeBy/bciXD8UlHUHt+O714tnYnq
 UT+B35wFIaOTm10PVDRtb8yiC621Xf1H22YzEkSvy0Al2zcBCz2HtbYldBlwZjYQwoN/x/zQh
 eQDCGGMS3F29MBAx6gYkqJ+AaRm1ClsvgN6I5DXbuAWk/X78SiIGVVIGH4nWzOLdLNgDfxI8C
 crKoPDD4GGsamxJiRK+L4kq/pQ+7gNtq77/CCrvNcQRcrw6/pgVLRzfKG1wSxDzHJna9gzTDS
 k5RUi/cfWTw5sPID9qm5zmbeLXYkJccIa7FBFfsxDmryJn4PPDOgCl5mK3gKIyRFz0RUf/Zi6
 y64HXGKWOOUlcNMtpExFLW2KwznXnX1rjRDn0H+6qnwT3feL9UIGydGwbAt9SC/VCEYOdAJds
 EKYt7Om7/54um9zAfz7q7ryEbmHr1PlMk2rGgJDVmxPmWEVmfJqgivbol1C/E5vvZ9W41I0it
 rCYDfA==
Precedence: bulk


Hi,

On 11.10.23 at 19:42, Greg KH wrote:
> On Wed, Oct 11, 2023 at 07:38:24PM +0200, Lino Sanfilippo wrote:
>> Due to earlier code changes function pl011_get_rs485_mode() is now mere=
ly
>> a wrapper for uart_get_rs485_mode() which does not add any further
>> functionality. So remove it and instead call uart_get_rs485_mode()
>> directly.
>>
>> Reviewed-by: Lukas Wunner <lukas@wunner.de>
>> Signed-off-by: Lino Sanfilippo <l.sanfilippo@kunbus.com>
>> ---
>>  drivers/tty/serial/amba-pl011.c | 14 +-------------
>>  1 file changed, 1 insertion(+), 13 deletions(-)
>
> Why is patch 2/7 not cc: stable, when patches 3-7 are?  Either this
> patch isn't needed in this series, and can go later (or to a different
> branch), or it also needs to be marked for stable as the later patches
> depend on it?
>

2/7 is really only a cleanup patch that does not provide a bugfix.
Should I remove it from this series?

Regards,
Lino