Received-SPF: pass (google.com: domain of linux-kernel+bounces-142157-linux.lists.archive=gmail.com@vger.kernel.org designates 147.75.199.223 as permitted sender) client-ip=147.75.199.223;
Date: Fri, 12 Apr 2024 08:35:27 +0100
From: Lee Jones <lee@kernel.org>
To: Jakub Kicinski <kuba@kernel.org>
Cc: Min Li <lnimi@hotmail.com>, richardcochran@gmail.com,
	linux-kernel@vger.kernel.org, netdev@vger.kernel.org,
	Min Li <min.li.xe@renesas.com>
Subject: Re: [PATCH net-next v2 1/1] ptp: 82p33: move register definitions
 out of ptp folder
Message-ID: <20240412073527.GO2399047@google.com>
References: <LV3P220MB12026EB15790ABE932799E53A0062@LV3P220MB1202.NAMP220.PROD.OUTLOOK.COM>
 <20240411163240.GI2399047@google.com>
 <20240411185517.153a7dff@kernel.org>
Precedence: bulk
MIME-Version: 1.0
Content-Type: text/plain; charset=utf-8
Content-Disposition: inline
Content-Transfer-Encoding: 8bit
In-Reply-To: <20240411185517.153a7dff@kernel.org>

On Thu, 11 Apr 2024, Jakub Kicinski wrote:

> On Thu, 11 Apr 2024 17:32:40 +0100 Lee Jones wrote:
> > > From: Min Li <min.li.xe@renesas.com>
> > > 
> > > Relocate idt82p33 register definitions so that other multi-functional
> > > devices can access those definitions.
> 
> What I was asking for was to mention the other drivers which will make
> use of the header _in the future_.
> 
> Really, it'd be best if the patch was part of a patchset which adds
> a driver that needs it...

100%

> > > Signed-off-by: Min Li <min.li.xe@renesas.com>
> > > ---
> > > Submit to net-next and add driver name suggested by Jakub
> > > 
> > >  drivers/ptp/ptp_idt82p33.h       | 27 -----------------------
> > >  include/linux/mfd/idt82p33_reg.h | 38 +++++++++++++++++++++++++++++---  
> > 
> > Why does this need to go into net-next?
> 
> No preference here, FWIW

-- 
Lee Jones [李琼斯]