linux-kernel - Re: [PATCH v2 1/1] MAINTAINERS: I3C: Add Frank Li as nxp I3C controller driver maintainer

lists.openwall.net		lists / announce owl-users owl-dev john-users john-dev passwdqc-users yescrypt popa3d-users / oss-security kernel-hardening musl sabotage tlsify passwords / crypt-dev xvendor / Bugtraq Full-Disclosure linux-kernel linux-netdev linux-ext4 linux-hardening linux-cve-announce PHC
Open Source and information security mailing list archives

Hash Suite: Windows password security audit tool. GUI, reports in PDF.

[<prev] [next>] [<thread-prev] [thread-next>] [day] [month] [year] [list]

Message-ID: <20241001121027.29a51284@xps-13>
Date: Tue, 1 Oct 2024 12:10:27 +0200
From: Miquel Raynal <miquel.raynal@...tlin.com>
To: Frank Li <Frank.li@....com>
Cc: alexandre.belloni@...tlin.com, arnd@...db.de, bbrezillon@...nel.org,
 boris.brezillon@...labora.com, conor.culhane@...vaco.com,
 gregkh@...uxfoundation.org, imx@...ts.linux.dev,
 linux-i3c@...ts.infradead.org, linux-kernel@...r.kernel.org,
 pthombar@...ence.com
Subject: Re: [PATCH v2 1/1] MAINTAINERS: I3C: Add Frank Li as nxp I3C
 controller driver maintainer

Hi Frank,

Frank.li@....com wrote on Mon, 30 Sep 2024 13:08:06 -0400:

> On Mon, Sep 30, 2024 at 06:22:35PM +0200, Miquel Raynal wrote:
> > Hi Frank,
> >
> > Frank.li@....com wrote on Mon, 9 Sep 2024 16:05:32 -0400:
> >  
> > > On Thu, Aug 15, 2024 at 10:16:09AM -0400, Frank Li wrote:  
> > > > Add Frank Li as NXP i3c controller driver as maintainer and add mail list
> > > > imx@...ts.linux.dev for it.
> > > >
> > > > Add Frank Li as I3C subsystem reviewer.
> > > >
> > > > Signed-off-by: Frank Li <Frank.Li@....com>
> > > > ---  
> > >
> > > Miquel:
> > > 	Any comments?  
> >
> > I guess we'll happily take your feedback on patches targeting the SVC
> > driver, you've been working a lot on it. However, in general there has
> > been a lot of discussions happening on your substantial patchsets and
> > little to no convergence in the end.  
> 
> I think this can't prevent no any feedback at this thread for more than
> 1 month.

Well, there's been a lot of discussion, then no convergence, that's
what I am pointing out as well. You cannot say there has been no
discussion.

> Basic, this patch did two things,

Actually 3, and this is usually too much.

> one add mail list imx@...ts.linux.dev to
> keep us track all imx related patch in one mail list. generally, we don't
> check linux-i3c mail list frequently.
> 
> The other things is add me to maintainer, If you have concern about it.
> You'd better said as
> - point to the path to be maintainer, such as do reviewer first for some
> times.
> - or simple reject this, I can post the patch, which only add
> imx@...ts.linux.dev.
> 
> Keep simple, two questions:
> 1: are you agree on add imx@...ts.linux.dev?
> 2: are you agree on I as reviewer?

I've already answered these questions and made a difference between
reviewing driver changes and core changes.

> > I understand you want to push your
> > patches upstream and I also want to make things move forward, but maybe
> > one of the reasons for the stalls is the lack of fluent exchanges
> > between us and the difficulty to explain bigger pictures in a clear
> > way.  

Thanks,
Miquèl