From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S934544AbeE2NfE (ORCPT ); Tue, 29 May 2018 09:35:04 -0400 Received: from szxga04-in.huawei.com ([45.249.212.190]:8162 "EHLO huawei.com" rhost-flags-OK-OK-OK-FAIL) by vger.kernel.org with ESMTP id S934136AbeE2NfC (ORCPT ); Tue, 29 May 2018 09:35:02 -0400 Subject: Re: [PATCH net-next] net: davinci_mdio: fix building error without CONFIG_OF To: , , References: <20180529115651.3420-1-yuehaibing@huawei.com> CC: , , , , From: YueHaibing Message-ID: <747c9847-8e54-fa2e-231d-ac8b64d7ff47@huawei.com> Date: Tue, 29 May 2018 21:34:32 +0800 User-Agent: Mozilla/5.0 (Windows NT 6.1; WOW64; rv:45.0) Gecko/20100101 Thunderbird/45.2.0 MIME-Version: 1.0 In-Reply-To: <20180529115651.3420-1-yuehaibing@huawei.com> Content-Type: text/plain; charset="utf-8" Content-Transfer-Encoding: 8bit X-Originating-IP: [10.177.31.96] X-CFilter-Loop: Reflected Sender: linux-kernel-owner@vger.kernel.org List-ID: X-Mailing-List: linux-kernel@vger.kernel.org pls ignore this, there has been a Patch from Arnd Bergmann [PATCH] net: davinci: fix building davinci mdio code without CONFIG_OF On 2018/5/29 19:56, YueHaibing wrote: > gcc report a build error when compiling without CONFIG_OF > drivers/net/ethernet/ti/davinci_mdio.c: In function ‘davinci_mdio_probe’: > drivers/net/ethernet/ti/davinci_mdio.c:380:9: error: implicit declaration of function ‘davinci_mdio_probe_dt’ [-Werror=implicit-function-declaration] > ret = davinci_mdio_probe_dt(&data->pdata, pdev); > ^ > Fixes: 9eae9c7d0875 ("drivers: net: davinci_mdio: enable pm runtime auto for ti cpsw-mdio") > Signed-off-by: YueHaibing > --- > drivers/net/ethernet/ti/davinci_mdio.c | 8 ++++++-- > 1 file changed, 6 insertions(+), 2 deletions(-) > > diff --git a/drivers/net/ethernet/ti/davinci_mdio.c b/drivers/net/ethernet/ti/davinci_mdio.c > index 8ac7283..6e544d9 100644 > --- a/drivers/net/ethernet/ti/davinci_mdio.c > +++ b/drivers/net/ethernet/ti/davinci_mdio.c > @@ -339,9 +339,7 @@ static int davinci_mdio_probe_dt(struct mdio_platform_data *data, > > return 0; > } > -#endif > > -#if IS_ENABLED(CONFIG_OF) > static const struct davinci_mdio_of_param of_cpsw_mdio_data = { > .autosuspend_delay_ms = 100, > }; > @@ -352,6 +350,12 @@ static const struct of_device_id davinci_mdio_of_mtable[] = { > { /* sentinel */ }, > }; > MODULE_DEVICE_TABLE(of, davinci_mdio_of_mtable); > +#else > +static int davinci_mdio_probe_dt(struct mdio_platform_data *data, > + struct platform_device *pdev) > +{ > + return -EINVAL; > +} > #endif > > static int davinci_mdio_probe(struct platform_device *pdev) >