From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1751396AbcBDFJe (ORCPT ); Thu, 4 Feb 2016 00:09:34 -0500 Received: from smtp.codeaurora.org ([198.145.29.96]:41140 "EHLO smtp.codeaurora.org" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1750987AbcBDFJd (ORCPT ); Thu, 4 Feb 2016 00:09:33 -0500 Subject: Re: [PATCH] drm/msm/dsi: fix definition of msm_dsi_pll_28nm_8960_init() To: Luis Henriques , David Airlie , Rob Clark References: <1454509515-2365-1-git-send-email-luis.henriques@canonical.com> Cc: dri-devel@lists.freedesktop.org, linux-kernel@vger.kernel.org From: Archit Taneja Message-ID: <56B2DD05.1070405@codeaurora.org> Date: Thu, 4 Feb 2016 10:39:25 +0530 User-Agent: Mozilla/5.0 (X11; Linux x86_64; rv:38.0) Gecko/20100101 Thunderbird/38.2.0 MIME-Version: 1.0 In-Reply-To: <1454509515-2365-1-git-send-email-luis.henriques@canonical.com> Content-Type: text/plain; charset=windows-1252; format=flowed Content-Transfer-Encoding: 7bit Sender: linux-kernel-owner@vger.kernel.org List-ID: X-Mailing-List: linux-kernel@vger.kernel.org On 02/03/2016 07:55 PM, Luis Henriques wrote: > This fixes the following build failure: > > drivers/gpu/drm/msm/dsi/pll/dsi_pll_28nm.o: In function `msm_dsi_pll_28nm_8960_init': > dsi_pll_28nm.c:(.text+0x1198): multiple definition of `msm_dsi_pll_28nm_8960_init' > drivers/gpu/drm/msm/dsi/pll/dsi_pll.o:dsi_pll.c:(.text+0x0): first defined here Thanks for the fix. Acked-by: Archit Taneja Dave, Could you please queue this for the next -rc cycle since it causes a build break? Thanks, Archit > > Signed-off-by: Luis Henriques > --- > drivers/gpu/drm/msm/dsi/pll/dsi_pll.h | 4 ++-- > 1 file changed, 2 insertions(+), 2 deletions(-) > > diff --git a/drivers/gpu/drm/msm/dsi/pll/dsi_pll.h b/drivers/gpu/drm/msm/dsi/pll/dsi_pll.h > index 80b6038334a6..2cf1664723e8 100644 > --- a/drivers/gpu/drm/msm/dsi/pll/dsi_pll.h > +++ b/drivers/gpu/drm/msm/dsi/pll/dsi_pll.h > @@ -97,8 +97,8 @@ static inline struct msm_dsi_pll *msm_dsi_pll_28nm_init( > struct msm_dsi_pll *msm_dsi_pll_28nm_8960_init(struct platform_device *pdev, > int id); > #else > -struct msm_dsi_pll *msm_dsi_pll_28nm_8960_init(struct platform_device *pdev, > - int id) > +static inline struct msm_dsi_pll *msm_dsi_pll_28nm_8960_init( > + struct platform_device *pdev, int id) > { > return ERR_PTR(-ENODEV); > } > -- The Qualcomm Innovation Center, Inc. is a member of the Code Aurora Forum, hosted by The Linux Foundation