Received-SPF: pass (google.com: domain of linux-kernel-owner@vger.kernel.org designates 23.128.96.18 as permitted sender) client-ip=23.128.96.18;
From:   "Keller, Jacob E" <jacob.e.keller@intel.com>
To:     Richard Cochran <richardcochran@gmail.com>,
        Arnd Bergmann <arnd@kernel.org>
CC:     Nicolas Pitre <nico@fluxnic.net>,
        "Brandeburg, Jesse" <jesse.brandeburg@intel.com>,
        "Nguyen, Anthony L" <anthony.l.nguyen@intel.com>,
        "David S. Miller" <davem@davemloft.net>,
        "Jakub Kicinski" <kuba@kernel.org>, Arnd Bergmann <arnd@arndb.de>,
        Kurt Kanzenbach <kurt@linutronix.de>,
        "Saleem, Shiraz" <shiraz.saleem@intel.com>,
        "Ertman, David M" <david.m.ertman@intel.com>,
        "intel-wired-lan@lists.osuosl.org" <intel-wired-lan@lists.osuosl.org>,
        "netdev@vger.kernel.org" <netdev@vger.kernel.org>,
        "linux-kernel@vger.kernel.org" <linux-kernel@vger.kernel.org>
Subject: RE: [PATCH net-next v2] ethernet/intel: fix PTP_1588_CLOCK
 dependencies
Thread-Topic: [PATCH net-next v2] ethernet/intel: fix PTP_1588_CLOCK
 dependencies
Thread-Index: AQHXh68Y1owt0G/CUEakzTAdq+RvtqtgbT+AgAAzuICAADaFgIAAgzoAgACWAgCAAAU1AIAADOyAgABBaICAAZHC4A==
Date:   Wed, 4 Aug 2021 20:53:05 +0000
Message-ID: <CO1PR11MB5089005017FE714B8A37F9B0D6F19@CO1PR11MB5089.namprd11.prod.outlook.com>
References: <20210802145937.1155571-1-arnd@kernel.org>
 <20210802164907.GA9832@hoboy.vegasvil.org>
 <bd631e36-1701-b120-a9b0-8825d14cc694@intel.com>
 <20210802230921.GA13623@hoboy.vegasvil.org>
 <CAK8P3a2XjgbEkYs6R7Q3RCZMV7v90gu_v82RVfFVs-VtUzw+_w@mail.gmail.com>
 <20210803155556.GD32663@hoboy.vegasvil.org>
 <20210803161434.GE32663@hoboy.vegasvil.org>
 <CAK8P3a2Wt9gnO4Ts_4Jw1+qpBj8HQc50jU2szjmR8MmZL9wrgQ@mail.gmail.com>
 <20210803205455.GA3517@hoboy.vegasvil.org>
In-Reply-To: <20210803205455.GA3517@hoboy.vegasvil.org>
Accept-Language: en-US
Content-Language: en-US
dlp-product: dlpe-windows
dlp-reaction: no-action
dlp-version: 11.5.1.3
x-ms-exchange-antispam-messagedata-chunkcount: 1
x-ms-exchange-antispam-messagedata-0: =?us-ascii?Q?9ew3QM1TPZrlmgiiw6OCIEgzu3JQOfXkA+nZhY3Tc+Q9gzfURrUIYsYBzqnt?=
 =?us-ascii?Q?NnyxijvS6cznewKJzScQuxuja2PJBR7qUiiFSY8TJ5ZeEd1XgL7Y3Cjt6duv?=
 =?us-ascii?Q?g5I9SNbZ7rNP7ZA1Kpm5/YSkYCPeL0ZNe4CRXHEnFae1U4nw42XYD0R6fhnC?=
 =?us-ascii?Q?2WNMTYiYsvr/i0Pd9rMZ/CipZxDFqlgQ6wui4liSAwPFEc1g+Hr+pAKBMzFb?=
 =?us-ascii?Q?/XY7vVafU71t3myUuNvWFCNg86pSE3nYiLjDqIfZR85QSZEM6441irGlmKhJ?=
 =?us-ascii?Q?5T+rqEYeLPdz13sedkdnGet/p83loB2SFi3/H8izsVKOHc+6k5njI7hz7tEr?=
 =?us-ascii?Q?irSyvaHKHle52y2zbyWxBCdgmBfqXoigE7iyCFlJIHU1uX4gWa1buzxbiHqX?=
 =?us-ascii?Q?4DtG1g76WmcZ2WB0PYndtniOPSzURGfipNVHJyj31+MWauW6/sOyAG3jEsPQ?=
 =?us-ascii?Q?eXWMn75a7QJ2LBPfOb604ll8i0U/4duZESfDcCPnmyqakj7r48JbH0ZPMAMp?=
 =?us-ascii?Q?2+oaWY+Agny++6CT+rEfskm27JoeIo0AWMKEum39zvEf7KpUEfLdpmiZPcTQ?=
 =?us-ascii?Q?mCi4Q/5uJLitbaWoKDAXEYTsjemRcWr86ZllwhjBsjQ7ZGNmwOrQv+kijqXT?=
 =?us-ascii?Q?30dlslzFsIwUwTZIYPguPKVlCn95RtfPWCO7eXsmvqxoPpIfbpRGv7tbRpJn?=
 =?us-ascii?Q?d8yW4B4sz00f18Qu+lXyt6Cs35BHI2iR2RHlNz+5d0s+SjCtNbVDQhs8s+rQ?=
 =?us-ascii?Q?cRMES1D88gQt38K1/ZkY7hoFi6+H2M7+QjG3vEjX0m0pvgTjGnQ5HEoGejIc?=
 =?us-ascii?Q?5qHiqbM0TINhgbrMm7Sh6sx7cFUdWisAoSVGItnoGQJDFDwnz9AI4AMtQPVM?=
 =?us-ascii?Q?wS1lHXEVxVRq5sz3VdgxfOP2Pv4ulDcDJvi7e6aWKVGPSXUkrN1zxu89Kvif?=
 =?us-ascii?Q?DwRSj0VvzS6FXbXPS+hijfavmTHhkseT49uBD3QOTFB8ewJIN08M4CPlNTp7?=
 =?us-ascii?Q?e76rVCoNxuGe1TBEqd0ol0AA/WYXlA1iJdr7nPM5PhErXy3Mdf3svDUHRrsv?=
 =?us-ascii?Q?Ritm8eznuI26UGjZMei40PT2XQhmkFO6t6L4vcw4SvnPo+BZa1yxN9xwYJQ+?=
 =?us-ascii?Q?cP+Kld+pMn/zJ+ZQ9EIyJblLa7lzPLdgQ5btIoZmAVhp7+6CO4Tp1UmrVmet?=
 =?us-ascii?Q?AmYrpAhjfARWkjiuNsbVluw57mIKSX/mxjR0MHqJwhQi4BrQ9djtnzzW8Rv2?=
 =?us-ascii?Q?1JHLS92jYDWRyFqf8tEOhF1gCRaoiw4s84Yd8qsVsc7eqG/4vPV27KZXTwI7?=
 =?us-ascii?Q?0PE=3D?=
Content-Type: text/plain; charset="us-ascii"
Content-Transfer-Encoding: quoted-printable
MIME-Version: 1.0
X-MS-Exchange-CrossTenant-AuthAs: Internal
X-MS-Exchange-CrossTenant-AuthSource: CO1PR11MB5089.namprd11.prod.outlook.com
X-MS-Exchange-CrossTenant-Network-Message-Id: e0f9a0bd-0197-4af9-568e-08d95789db7b
X-MS-Exchange-CrossTenant-originalarrivaltime: 04 Aug 2021 20:53:05.1496
 (UTC)
X-MS-Exchange-CrossTenant-fromentityheader: Hosted
X-MS-Exchange-CrossTenant-id: 46c98d88-e344-4ed4-8496-4ed7712e255d
X-MS-Exchange-CrossTenant-mailboxtype: HOSTED
X-MS-Exchange-CrossTenant-userprincipalname: 28nPTEC+A/R1e79oHa6GCnahOzTAu/AcfA5JlSTRX/0WLS0/seoL7KIaSUqKR2BHXun8N60OLQ+YIDYg+3oL5qcrVSU0b8P/wYKFITQz4vU=
X-MS-Exchange-Transport-CrossTenantHeadersStamped: MWHPR11MB1407
X-OriginatorOrg: intel.com
Precedence: bulk
List-ID: <linux-kernel.vger.kernel.org>
X-Mailing-List: linux-kernel@vger.kernel.org


> -----Original Message-----
> From: Richard Cochran <richardcochran@gmail.com>
> Sent: Tuesday, August 03, 2021 1:55 PM
> To: Arnd Bergmann <arnd@kernel.org>
> Cc: Nicolas Pitre <nico@fluxnic.net>; Keller, Jacob E <jacob.e.keller@int=
el.com>;
> Brandeburg, Jesse <jesse.brandeburg@intel.com>; Nguyen, Anthony L
> <anthony.l.nguyen@intel.com>; David S. Miller <davem@davemloft.net>; Jaku=
b
> Kicinski <kuba@kernel.org>; Arnd Bergmann <arnd@arndb.de>; Kurt
> Kanzenbach <kurt@linutronix.de>; Saleem, Shiraz <shiraz.saleem@intel.com>=
;
> Ertman, David M <david.m.ertman@intel.com>; intel-wired-lan@lists.osuosl.=
org;
> netdev@vger.kernel.org; linux-kernel@vger.kernel.org
> Subject: Re: [PATCH net-next v2] ethernet/intel: fix PTP_1588_CLOCK
> dependencies
>=20
>=20
> On Tue, Aug 03, 2021 at 07:00:49PM +0200, Arnd Bergmann wrote:
>=20
> > 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.
>=20
> "depends" instead of "select" works for me.  I just want it simple and cl=
ear.
>=20
> > 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.
>=20
> Sounds good.
>=20
> Thanks,
> Richard

Works for me too.

Thanks,
Jake