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;
Date:   Thu, 24 Jun 2021 12:14:24 -0700
From:   Jakub Kicinski <kuba@kernel.org>
To:     Marcin Wojtas <mw@semihalf.com>
Cc:     David Miller <davem@davemloft.net>,
        Stephen Rothwell <sfr@canb.auug.org.au>,
        Naresh Kamboju <naresh.kamboju@linaro.org>,
        Networking <netdev@vger.kernel.org>,
        Linux Kernel Mailing List <linux-kernel@vger.kernel.org>,
        Linux Next Mailing List <linux-next@vger.kernel.org>,
        lkft-triage@lists.linaro.org, Nathan Chancellor <nathan@kernel.org>
Subject: Re: linux-next: build failure after merge of the net-next tree
Message-ID: <20210624121424.51d754bb@kicinski-fedora-pc1c0hjn.dhcp.thefacebook.com>
In-Reply-To: <CAPv3WKdjE5ywVFB+94invSLg=jG5JHBdvLQLKDTPq13+8PjqmA@mail.gmail.com>
References: <20210624082911.5d013e8c@canb.auug.org.au>
        <CAPv3WKfiL+sR+iK_BjGKDhtNgjoxKEPv49bU1X9_7+v+ytdR1w@mail.gmail.com>
        <YNPt91bfjrgSt8G3@Ryzen-9-3900X.localdomain>
        <CA+G9fYtb07aySOpB6=wc4ip_9S4Rr2UUYNgEOG6i76g--uPryQ@mail.gmail.com>
        <20210624185430.692d4b60@canb.auug.org.au>
        <CAPv3WKf6HguRC_2ckau99d4iWG-FV71kn8wiX9r5wuK335EEFw@mail.gmail.com>
        <3d6ea68a-9654-6def-9533-56640ceae69f@kernel.org>
        <CAPv3WKdjE5ywVFB+94invSLg=jG5JHBdvLQLKDTPq13+8PjqmA@mail.gmail.com>
MIME-Version: 1.0
Content-Type: text/plain; charset=US-ASCII
Content-Transfer-Encoding: 7bit
Precedence: bulk

On Thu, 24 Jun 2021 20:15:00 +0200 Marcin Wojtas wrote:
> TL;DR, we need to get rid of a helper routine (introduced so that to
> address review comments of v1), as it causes a depmod cycles when
> fwnode_/of_/acpi_mdio are built as modules.
> It can be done twofold:
> a. 3 commits, i.e:
>   Revert "net: mdiobus: Introduce fwnode_mdbiobus_register()"
>   Revert "net/fsl: switch to fwnode_mdiobus_register"
>   net: mvmdio: resign from fwnode_mdiobus_register
> b. Same diff but squashed.
> 
> Please let me know your preference, so that I can do it properly up front.

Not sure if Dave will notice this mid-thread question. It probably
doesn't matter as long as (1) doesn't introduce further transient 
build failures.