From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1752172AbcLESo6 (ORCPT ); Mon, 5 Dec 2016 13:44:58 -0500 Received: from mail-io0-f173.google.com ([209.85.223.173]:32993 "EHLO mail-io0-f173.google.com" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1751639AbcLESo4 (ORCPT ); Mon, 5 Dec 2016 13:44:56 -0500 Date: Mon, 5 Dec 2016 13:44:53 -0500 (EST) From: Nicolas Pitre To: David Miller cc: arnd@arndb.de, felix.manlunas@caviumnetworks.com, tglx@linutronix.de, david.daney@cavium.com, satananda.burla@caviumnetworks.com, rvatsavayi@caviumnetworks.com, sgoutham@cavium.com, netdev@vger.kernel.org, linux-kernel@vger.kernel.org Subject: Re: [PATCH net-next] liquidio: 'imply' ptp instead of 'select' In-Reply-To: <20161205.122029.1310361702811893159.davem@davemloft.net> Message-ID: References: <20161202230451.1639318-1-arnd@arndb.de> <20161203.233945.1757084444220812603.davem@davemloft.net> <20161205.122029.1310361702811893159.davem@davemloft.net> User-Agent: Alpine 2.20 (LFD 67 2015-01-07) MIME-Version: 1.0 Content-Type: text/plain; charset=US-ASCII Sender: linux-kernel-owner@vger.kernel.org List-ID: X-Mailing-List: linux-kernel@vger.kernel.org On Mon, 5 Dec 2016, David Miller wrote: > From: Nicolas Pitre > Date: Mon, 5 Dec 2016 10:44:32 -0500 (EST) > > > On Sat, 3 Dec 2016, David Miller wrote: > > > >> From: Arnd Bergmann > >> Date: Sat, 3 Dec 2016 00:04:32 +0100 > >> > >> > ptp now depends on the optional POSIX_TIMERS setting and fails to build > >> > if we select it without that: > >> > > >> > warning: (LIQUIDIO_VF && TI_CPTS) selects PTP_1588_CLOCK which has unmet direct dependencies (NET && POSIX_TIMERS) > >> > warning: (LIQUIDIO_VF && TI_CPTS) selects PTP_1588_CLOCK which has unmet direct dependencies (NET && POSIX_TIMERS) > >> > ERROR: "posix_clock_unregister" [drivers/ptp/ptp.ko] undefined! > >> > ERROR: "posix_clock_register" [drivers/ptp/ptp.ko] undefined! > >> > ERROR: "pps_unregister_source" [drivers/ptp/ptp.ko] undefined! > >> > ERROR: "pps_event" [drivers/ptp/ptp.ko] undefined! > >> > ERROR: "pps_register_source" [drivers/ptp/ptp.ko] undefined! > >> > > >> > It seems that two patches have collided here, the build failure > >> > is a result of the combination. Changing the new option to 'imply' > >> > as well fixes it. > >> > > >> > Fixes: 111fc64a237f ("liquidio CN23XX: VF registration") > >> > Fixes: d1cbfd771ce8 ("ptp_clock: Allow for it to be optional") > >> > Signed-off-by: Arnd Bergmann > >> > >> Like the kbuild robot, when I apply this it complains about 'imply' being > >> an unknown option. > >> > >> I guess it worked for you because support for 'imply' exists in the -next > >> tree and gets pulled in from somewhere else. > > > > Exact. > > > >> In any event, as-is I cannot apply this. > > > > It should be carried in linux-next for the time being, and suggested as > > a probable "merge resolution" to Linus when submitting your tree for > > merging. I think that's the best that can be done. > > This means the build of pure net-next is broken, which is unacceptble, as > this is the tree that many developers do their work and build tests against. Why would it be broken? The only thing that needs to happen is for that "select" to be turned into a "imply" when your tree is merged into Linus' tree. Or when your tree is merged into linux-next (I'm sure Stephen can carry a patch to that effect). It doesn't have to live in net-next directly. The "imply" support has been available in linux-next for almost a month now, and it has been discussed for much longer than that. And if people forget then we'll fix it then. That's not a big issue. Nicolas