From mboxrd@z Thu Jan  1 00:00:00 1970
Return-Path: <linux-kernel-owner@vger.kernel.org>
Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand
	id S932903AbcCMVNE (ORCPT <rfc822;w@1wt.eu>);
	Sun, 13 Mar 2016 17:13:04 -0400
Received: from mail-wm0-f54.google.com ([74.125.82.54]:34022 "EHLO
	mail-wm0-f54.google.com" rhost-flags-OK-OK-OK-OK) by vger.kernel.org
	with ESMTP id S1754198AbcCMVMy (ORCPT
	<rfc822;linux-kernel@vger.kernel.org>);
	Sun, 13 Mar 2016 17:12:54 -0400
Subject: Re: [PATCH] drm/msm/dsi: fix definition of
 msm_dsi_pll_28nm_8960_init()
To: Archit Taneja <architt@codeaurora.org>,
        Luis Henriques <luis.henriques@canonical.com>,
        David Airlie <airlied@linux.ie>, Rob Clark <robdclark@gmail.com>,
        Linux Kernel <linux-kernel@vger.kernel.org>
References: <1454509515-2365-1-git-send-email-luis.henriques@canonical.com>
 <56B2DD05.1070405@codeaurora.org>
Cc: linux-kernel@vger.kernel.org, dri-devel@lists.freedesktop.org
From: Kieran Bingham <kieran.bingham@linaro.org>
Message-ID: <56E5D7CD.7070204@linaro.org>
Date: Sun, 13 Mar 2016 21:12:45 +0000
User-Agent: Mozilla/5.0 (X11; Linux x86_64; rv:38.0) Gecko/20100101
 Thunderbird/38.5.1
MIME-Version: 1.0
In-Reply-To: <56B2DD05.1070405@codeaurora.org>
Content-Type: text/plain; charset=utf-8
Content-Transfer-Encoding: 8bit
Sender: linux-kernel-owner@vger.kernel.org
List-ID: <linux-kernel.vger.kernel.org>
X-Mailing-List: linux-kernel@vger.kernel.org

On 04/02/16 05:09, Archit Taneja wrote:
> 
> 
> 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 <architt@codeaurora.org>
> 
> Dave,
> 
> Could you please queue this for the next -rc cycle since it causes a
> build break?
> 

I've just hit this build break on v4.5-rc7 ... was there any progress
getting this fix in ? I can't see any further reference to this thread
on LKML.

Just to note, changing to a static generates a build-warning for me:

  CC      drivers/gpu/drm/msm/dsi/pll/dsi_pll_28nm.o
In file included from
sources/linux/drivers/gpu/drm/msm/dsi/pll/dsi_pll_28nm.c:17:0:
sources/linux/drivers/gpu/drm/msm/dsi/pll/dsi_pll.h:100:28: warning:
‘msm_dsi_pll_28nm_8960_init’ defined but not used [-Wunused-function]
 static struct msm_dsi_pll *msm_dsi_pll_28nm_8960_init(struct
platform_device *pdev,
                            ^
--
Kieran

> Thanks,
> Archit
> 
>>
>> Signed-off-by: Luis Henriques <luis.henriques@canonical.com>
>> ---
>>   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);
>>   }
>>
>