From mboxrd@z Thu Jan 1 00:00:00 1970 From: Sergei Shtylyov Subject: Re: [PATCH v3 0/7] net: use platform_{get,set}_drvdata() Date: Fri, 16 Aug 2013 02:51:52 +0400 Message-ID: <520D5B88.7080304@cogentembedded.com> References: <520CD11D.8010202@huawei.com> <20130815.152359.920302339053249583.davem@davemloft.net> <20130815.153903.859166500375208182.davem@davemloft.net> Mime-Version: 1.0 Content-Type: text/plain; charset=ISO-8859-7; format=flowed Content-Transfer-Encoding: QUOTED-PRINTABLE Cc: netdev@vger.kernel.org, lizefan@huawei.com, leoli@freescale.com, linuxppc-dev@lists.ozlabs.org, pantelis.antoniou@gmail.com, vbordug@ru.mvista.com, gregkh@linuxfoundation.org, jg1.han@samsung.com To: David Miller , clbchenlibo.chen@huawei.com Return-path: Received: from mail-lb0-f178.google.com ([209.85.217.178]:42638 "EHLO mail-lb0-f178.google.com" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1751737Ab3HOWvp (ORCPT ); Thu, 15 Aug 2013 18:51:45 -0400 Received: by mail-lb0-f178.google.com with SMTP id z5so1070477lbh.9 for ; Thu, 15 Aug 2013 15:51:44 -0700 (PDT) In-Reply-To: <20130815.153903.859166500375208182.davem@davemloft.net> Sender: netdev-owner@vger.kernel.org List-ID: Hello. On 08/16/2013 02:39 AM, David Miller wrote: >> From: Libo Chen >> Date: Thu, 15 Aug 2013 21:01:17 +0800 >>> Use the wrapper functions for getting and setting the driver data u= sing >>> platform_device instead of using dev_{get,set}_drvdata() with &pdev= ->dev, >>> so we can directly pass a struct platform_device. >>> changelog v3: >>> remove modify about dev_set_drvdata() >>> changelog v2: >>> this version add modify record about dev_set_drvdata(). >> Series applied. > Actually, I had to revert, these patches break the build. > drivers/net/ethernet/sun/sunhme.c: In function =A1happy_meal_pci_prob= e=A2: > drivers/net/ethernet/sun/sunhme.c:3114:2: error: implicit declaration= of function =A1platform_set_drvdata=A2 [-Werror=3Dimplicit-function-de= claration] > drivers/net/ethernet/sun/sunhme.c: In function =A1happy_meal_pci_remo= ve=A2: > drivers/net/ethernet/sun/sunhme.c:3162:9: error: implicit declaration= of function =A1platform_get_drvdata=A2 [-Werror=3Dimplicit-function-de= claration] > drivers/net/ethernet/sun/sunhme.c:3162:26: warning: initialization ma= kes pointer from integer without a cast [enabled by default] Hm, patch #5 was clearly defective as it tried to call=20 platform_{get|set}_drvdata() on PCI devices -- I've read the patch but=20 overlooked that. And the driver lacks #include , so=20 I'm not sure it always compiled flawlessly. WBR, Sergei