[<prev] [next>] [<thread-prev] [day] [month] [year] [list]
Message-ID: <CO1PR11MB5089005017FE714B8A37F9B0D6F19@CO1PR11MB5089.namprd11.prod.outlook.com>
Date: Wed, 4 Aug 2021 20:53:05 +0000
From: "Keller, Jacob E" <jacob.e.keller@...el.com>
To: Richard Cochran <richardcochran@...il.com>,
Arnd Bergmann <arnd@...nel.org>
CC: Nicolas Pitre <nico@...xnic.net>,
"Brandeburg, Jesse" <jesse.brandeburg@...el.com>,
"Nguyen, Anthony L" <anthony.l.nguyen@...el.com>,
"David S. Miller" <davem@...emloft.net>,
"Jakub Kicinski" <kuba@...nel.org>, Arnd Bergmann <arnd@...db.de>,
Kurt Kanzenbach <kurt@...utronix.de>,
"Saleem, Shiraz" <shiraz.saleem@...el.com>,
"Ertman, David M" <david.m.ertman@...el.com>,
"intel-wired-lan@...ts.osuosl.org" <intel-wired-lan@...ts.osuosl.org>,
"netdev@...r.kernel.org" <netdev@...r.kernel.org>,
"linux-kernel@...r.kernel.org" <linux-kernel@...r.kernel.org>
Subject: RE: [PATCH net-next v2] ethernet/intel: fix PTP_1588_CLOCK
dependencies
> -----Original Message-----
> From: Richard Cochran <richardcochran@...il.com>
> Sent: Tuesday, August 03, 2021 1:55 PM
> To: Arnd Bergmann <arnd@...nel.org>
> Cc: Nicolas Pitre <nico@...xnic.net>; Keller, Jacob E <jacob.e.keller@...el.com>;
> Brandeburg, Jesse <jesse.brandeburg@...el.com>; Nguyen, Anthony L
> <anthony.l.nguyen@...el.com>; David S. Miller <davem@...emloft.net>; Jakub
> Kicinski <kuba@...nel.org>; Arnd Bergmann <arnd@...db.de>; Kurt
> Kanzenbach <kurt@...utronix.de>; Saleem, Shiraz <shiraz.saleem@...el.com>;
> Ertman, David M <david.m.ertman@...el.com>; intel-wired-lan@...ts.osuosl.org;
> netdev@...r.kernel.org; linux-kernel@...r.kernel.org
> Subject: Re: [PATCH net-next v2] ethernet/intel: fix PTP_1588_CLOCK
> dependencies
>
>
> On Tue, Aug 03, 2021 at 07:00:49PM +0200, Arnd Bergmann wrote:
>
> > If you turn all those 'select' lines into 'depends on', this will work, but it's
> > not actually much different from what I'm suggesting.
>
> "depends" instead of "select" works for me. I just want it simple and clear.
>
> > Maybe we can do it
> > in two steps: first fix the build failure by replacing all the 'imply'
> > statements
> > with the correct dependencies, and then you send a patch on top that
> > turns PPS and PTP_1588_CLOCK into bool options.
>
> Sounds good.
>
> Thanks,
> Richard
Works for me too.
Thanks,
Jake
Powered by blists - more mailing lists