From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: from mx0a-001b2d01.pphosted.com (mx0b-001b2d01.pphosted.com [148.163.158.5]) (using TLSv1.2 with cipher ECDHE-RSA-AES256-GCM-SHA384 (256/256 bits)) (No client certificate requested) by lists.ozlabs.org (Postfix) with ESMTPS id 41lwWZ1DJmzDrYj for ; Thu, 9 Aug 2018 01:38:01 +1000 (AEST) Received: from pps.filterd (m0098413.ppops.net [127.0.0.1]) by mx0b-001b2d01.pphosted.com (8.16.0.22/8.16.0.22) with SMTP id w78FOZ9x090374 for ; Wed, 8 Aug 2018 11:37:59 -0400 Received: from e16.ny.us.ibm.com (e16.ny.us.ibm.com [129.33.205.206]) by mx0b-001b2d01.pphosted.com with ESMTP id 2kr0mjg1ug-1 (version=TLSv1.2 cipher=AES256-GCM-SHA384 bits=256 verify=NOT) for ; Wed, 08 Aug 2018 11:37:59 -0400 Received: from localhost by e16.ny.us.ibm.com with IBM ESMTP SMTP Gateway: Authorized Use Only! Violators will be prosecuted for from ; Wed, 8 Aug 2018 11:37:59 -0400 Received: from b01ledav001.gho.pok.ibm.com (b01ledav001.gho.pok.ibm.com [9.57.199.106]) by b01cxnp23032.gho.pok.ibm.com (8.14.9/8.14.9/NCO v10.0) with ESMTP id w78Fbv4Z11010314 (version=TLSv1/SSLv3 cipher=DHE-RSA-AES256-GCM-SHA384 bits=256 verify=FAIL) for ; Wed, 8 Aug 2018 15:37:57 GMT Received: from b01ledav001.gho.pok.ibm.com (unknown [127.0.0.1]) by IMSVA (Postfix) with ESMTP id 66B632805E for ; Wed, 8 Aug 2018 11:36:45 -0400 (EDT) Received: from b01ledav001.gho.pok.ibm.com (unknown [127.0.0.1]) by IMSVA (Postfix) with ESMTP id 3984528059 for ; Wed, 8 Aug 2018 11:36:45 -0400 (EDT) Received: from oc5000245537.ibm.com (unknown [9.85.216.127]) by b01ledav001.gho.pok.ibm.com (Postfix) with ESMTP for ; Wed, 8 Aug 2018 11:36:45 -0400 (EDT) Subject: Re: [PATCH v02] powerpc/mobility: Fix node detach/rename problem To: linuxppc-dev@lists.ozlabs.org References: <87zhxx9cs7.fsf@concordia.ellerman.id.au> From: Michael Bringmann In-Reply-To: <87zhxx9cs7.fsf@concordia.ellerman.id.au> Date: Wed, 8 Aug 2018 10:37:56 -0500 MIME-Version: 1.0 Content-Type: text/plain; charset=utf-8 Message-Id: <19f9e274-26ae-2ae5-de9f-c0fcd6c00570@linux.vnet.ibm.com> List-Id: Linux on PowerPC Developers Mail List List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , On 08/08/2018 09:02 AM, Michael Ellerman wrote: > Michael Bringmann writes: >> diff --git a/arch/powerpc/platforms/pseries/mobility.c b/arch/powerpc/platforms/pseries/mobility.c >> index e245a88..efc9442 100644 >> --- a/arch/powerpc/platforms/pseries/mobility.c >> +++ b/arch/powerpc/platforms/pseries/mobility.c >> @@ -22,6 +22,9 @@ >> #include >> #include "pseries.h" >> >> +extern int of_free_phandle_cache(void); >> +extern void of_populate_phandle_cache(void); > > We don't do that, they should be in a header. > > But that's a minor problem given that the patch doesn't compile, because > both those functions are static. I am building against the latest 'linux-ppc' kernel. It includes patch Commit b9952b5218added5577e4a3443969bc20884cea9 Mon Sep 17 00:00:00 2001 From: Frank Rowand Date: Thu, 12 Jul 2018 14:00:07 -0700 Subject: of: overlay: update phandle cache on overlay apply and remove which makes the functions static. I will rebuild and test with an earlier version if you will specify which one. > > Presumably you have a hack in your tree to make them non-static? > Please try and compile your patches in a clean tree before sending. > > cheers Regards, Michael -- Michael W. Bringmann Linux Technology Center IBM Corporation Tie-Line 363-5196 External: (512) 286-5196 Cell: (512) 466-0650 mwb@linux.vnet.ibm.com