Received-SPF: pass (google.com: domain of linux-kernel+bounces-166805-linux.lists.archive=gmail.com@vger.kernel.org designates 2604:1380:45e3:2400::1 as permitted sender) client-ip=2604:1380:45e3:2400::1;
Date: Thu, 2 May 2024 13:49:49 -0400
From: Parker Newman <parker@finest.io>
To: Andy Shevchenko <andriy.shevchenko@linux.intel.com>
Cc: Parker Newman <pnewman@connecttech.com>, linux-kernel@vger.kernel.org,
 linux-serial@vger.kernel.org, Greg Kroah-Hartman
 <gregkh@linuxfoundation.org>, Jiri Slaby <jirislaby@kernel.org>
Subject: Re: [PATCH v1 00/13] serial: 8250_exar: Clean up the driver
Message-ID: <20240502134949.5e780635@SWDEV2.connecttech.local>
In-Reply-To: <ZjPL5z7ah-Qkct6l@smile.fi.intel.com>
References: <20240502144626.2716994-1-andriy.shevchenko@linux.intel.com>
	<20240502114645.4445b3da@SWDEV2.connecttech.local>
	<ZjO4vYEBzxU3fpzC@smile.fi.intel.com>
	<20240502120840.02c65f30@SWDEV2.connecttech.local>
	<ZjPL5z7ah-Qkct6l@smile.fi.intel.com>
Organization: Connect Tech Inc.
Precedence: bulk
MIME-Version: 1.0
Content-Type: text/plain; charset=US-ASCII
Content-Transfer-Encoding: quoted-printable
UI-OutboundReport: notjunk:1;M01:P0:+bIjzS2iBZI=;UExRNUQgw6ik7VFbXAxSflRW0PK
 ge9BfOaEPX4xh03uQGRtU4dX1/oqEmxWfouVfy5wvTJfb6NqvGOxzGXPQUP9yTNBrx2DGQZG7
 /7jnTgE0ieOnOBa+7l19N4aeclF55eJncEW4fPet6Js2oGx96D5BDIs6uKxqNxr+b6mAANgjW
 l5PMGCOYP5qm7g7BoXbCNyHrlIVxQvROGXv7EtVdPrAJWSV1hSatay6FZXBYwDXvipUD0myi2
 Fr+Hu+S/unQ2tVL2U5TCWRhUQArquxTjAG87BZpTK0YeDhqkSv5+7LsMWX4dTEsoE6f7yuUh7
 2vq1R8NEOnaeEzpeNsnxwTQIR5WnCOvpTYT/4/hNraCZGkQFXTLPPGu8E29jn7kjL3x45Z3vq
 qcOqku+D+MQnoKsgRQAwQqo9kGmRqkMm1hPokHU6ErEa/1qbK0tXFaE2aEvl0GO7CHx0jUJCz
 AM+78rZuFhG0eo1gDq0Z83+GRkMKcxPYnD8fuZ2qAulmKBQNESgHB/xICPsYOZ4ta22rnKlcq
 j6cR55iqwcd4MZLm/OExmnyJgRcuhkeM4kc77wt5TKyDfnIhfI8mWS13VHqTMGPPqtOPLXIaq
 Q29v4mnhcTUM/i3oNOq0xDc4TSLuBbZF3bq5tUPgc7tBNc9FdZrDH4UUbgrMTwThKkXxDFmCl
 FIKWo4CxwZT+8is/CB8835hnAP1yO0Vk4obwC1HgvqvfgLY5J/kY8ou6xrEZp/ZP0zCXRuWcJ
 e5ngO1S4JcwQFVGVRfiLNTtPxvbgBp57iRUltqVoaWSOXxk+2Eiayk=

On Thu, 2 May 2024 20:22:47 +0300
Andy Shevchenko <andriy.shevchenko@linux.intel.com> wrote:

> On Thu, May 02, 2024 at 12:08:40PM -0400, Parker Newman wrote:
> > On Thu, 2 May 2024 19:01:01 +0300
> > Andy Shevchenko <andriy.shevchenko@linux.intel.com> wrote:
> > > On Thu, May 02, 2024 at 11:46:45AM -0400, Parker Newman wrote:
> > > > On Thu,  2 May 2024 17:43:54 +0300
> > > > Andy Shevchenko <andriy.shevchenko@linux.intel.com> wrote:
> > > >
> > > > > After a rework for CONNTECH was done, the driver may need a bit =
of
> > > > > love in order to become less verbose (in terms of indentation an=
d
> > > > > code duplication) and hence easier to read.
> > > > >
> > > > > This clean up series fixes a couple of (not so critical) issues =
and
> > > > > cleans up the recently added code. No functional change indented=
 by
> > > > > the cleaning up part.
> > > >
> > > > Just an FYI I submitted a patch series that fixed several of these=
 issues but I
> > > > think it fell through the cracks (I know everyone is very busy!).
> > > >
> > > > Link: https://lore.kernel.org/linux-serial/cover.1713533298.git.pn=
ewman@connecttech.com/
> > > >
> > > > I believe my previous patch series is no longer required. This one=
 fixes
> > > > everything.
> > >
> > > I haven't noticed that, if it contains duplicated patches, I may rep=
lace mine
> > > with yours if you insist.
> > >
> > > In any case it's better to reply there that you prefer this series t=
o be
> > > applied, so Greg will not pick it up.
> > >
> >
> > I do not have a preference. I am fine with using yours if it is easier=
 on
> > the maintainers.
>
> Up to you, there is no issue to take your patches in case they are the s=
ame
> (or quite similar) as mine. I can pick them up, just tell me if you want=
 this
> to happen with a list of the patches (as mail Message-Id).
>

Just use yours.
Thanks,
Parker

> > I will send a reply on my previous series that it is not needed and li=
nk
> > to this. I am new to the mailing lists so I didn't know what the proce=
dure
> > is for this situation.
>
> It's not about mailing lists, it's just a common sense.
>
> > Thanks for the fixes :),
>
> You are welcome!
>