From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: from mail-bn1bon0066.outbound.protection.outlook.com ([157.56.111.66]:13799 "EHLO na01-bn1-obe.outbound.protection.outlook.com" rhost-flags-OK-OK-OK-FAIL) by vger.kernel.org with ESMTP id S1756536AbbICOT6 (ORCPT ); Thu, 3 Sep 2015 10:19:58 -0400 Subject: Re: [PATCH v3 4/4] arm64: dts: Drop linux,pci-probe-only from the Seattle DTS To: Marc Zyngier , Benjamin Herrenschmidt , Paul Mackerras , "Michael Ellerman" , Will Deacon , "Bjorn Helgaas" , Lorenzo Pieralisi , Grant Likely , "Rob Herring" References: <1441282576-7304-1-git-send-email-marc.zyngier@arm.com> <1441282576-7304-5-git-send-email-marc.zyngier@arm.com> CC: Alexander Graf , , , , , From: "Suthikulpanit, Suravee" Message-ID: <55E856FC.4040006@amd.com> Date: Thu, 3 Sep 2015 21:19:40 +0700 MIME-Version: 1.0 In-Reply-To: <1441282576-7304-5-git-send-email-marc.zyngier@arm.com> Content-Type: text/plain; charset="windows-1252"; format=flowed Sender: linux-pci-owner@vger.kernel.org List-ID: Hi Marc On 9/3/2015 7:16 PM, Marc Zyngier wrote: > The linux,pci-probe-only property mandates an argument to indicate > whether or not to engage the "probe-only" mode, but the Seattle > DTS just provides a naked property, which is illegal. > > Also, it turns out that the board is perfectly happy without > probe-only, so let's drop this altogether. > > Signed-off-by: Marc Zyngier > --- > arch/arm64/boot/dts/amd/amd-overdrive.dts | 1 - > 1 file changed, 1 deletion(-) > > diff --git a/arch/arm64/boot/dts/amd/amd-overdrive.dts b/arch/arm64/boot/dts/amd/amd-overdrive.dts > index 564a3f7..128fa94 100644 > --- a/arch/arm64/boot/dts/amd/amd-overdrive.dts > +++ b/arch/arm64/boot/dts/amd/amd-overdrive.dts > @@ -14,7 +14,6 @@ > > chosen { > stdout-path = &serial0; > - linux,pci-probe-only; > }; > }; > > Thanks, I was planning to do this as well. The embedded DT in the UEFI FW will soon remove this property. Acked-by: Suravee Suthikulpanit Suravee From mboxrd@z Thu Jan 1 00:00:00 1970 From: suravee.suthikulpanit@amd.com (Suthikulpanit, Suravee) Date: Thu, 3 Sep 2015 21:19:40 +0700 Subject: [PATCH v3 4/4] arm64: dts: Drop linux,pci-probe-only from the Seattle DTS In-Reply-To: <1441282576-7304-5-git-send-email-marc.zyngier@arm.com> References: <1441282576-7304-1-git-send-email-marc.zyngier@arm.com> <1441282576-7304-5-git-send-email-marc.zyngier@arm.com> Message-ID: <55E856FC.4040006@amd.com> To: linux-arm-kernel@lists.infradead.org List-Id: linux-arm-kernel.lists.infradead.org Hi Marc On 9/3/2015 7:16 PM, Marc Zyngier wrote: > The linux,pci-probe-only property mandates an argument to indicate > whether or not to engage the "probe-only" mode, but the Seattle > DTS just provides a naked property, which is illegal. > > Also, it turns out that the board is perfectly happy without > probe-only, so let's drop this altogether. > > Signed-off-by: Marc Zyngier > --- > arch/arm64/boot/dts/amd/amd-overdrive.dts | 1 - > 1 file changed, 1 deletion(-) > > diff --git a/arch/arm64/boot/dts/amd/amd-overdrive.dts b/arch/arm64/boot/dts/amd/amd-overdrive.dts > index 564a3f7..128fa94 100644 > --- a/arch/arm64/boot/dts/amd/amd-overdrive.dts > +++ b/arch/arm64/boot/dts/amd/amd-overdrive.dts > @@ -14,7 +14,6 @@ > > chosen { > stdout-path = &serial0; > - linux,pci-probe-only; > }; > }; > > Thanks, I was planning to do this as well. The embedded DT in the UEFI FW will soon remove this property. Acked-by: Suravee Suthikulpanit Suravee From mboxrd@z Thu Jan 1 00:00:00 1970 From: "Suthikulpanit, Suravee" Subject: Re: [PATCH v3 4/4] arm64: dts: Drop linux,pci-probe-only from the Seattle DTS Date: Thu, 3 Sep 2015 21:19:40 +0700 Message-ID: <55E856FC.4040006@amd.com> References: <1441282576-7304-1-git-send-email-marc.zyngier@arm.com> <1441282576-7304-5-git-send-email-marc.zyngier@arm.com> Mime-Version: 1.0 Content-Type: text/plain; charset="windows-1252"; format=flowed Content-Transfer-Encoding: 7bit Return-path: In-Reply-To: <1441282576-7304-5-git-send-email-marc.zyngier@arm.com> Sender: linux-kernel-owner@vger.kernel.org To: Marc Zyngier , Benjamin Herrenschmidt , Paul Mackerras , Michael Ellerman , Will Deacon , Bjorn Helgaas , Lorenzo Pieralisi , Grant Likely , Rob Herring Cc: Alexander Graf , linux-kernel@vger.kernel.org, linux-arm-kernel@lists.infradead.org, linuxppc-dev@lists.ozlabs.org, linux-pci@vger.kernel.org, devicetree@vger.kernel.org List-Id: devicetree@vger.kernel.org Hi Marc On 9/3/2015 7:16 PM, Marc Zyngier wrote: > The linux,pci-probe-only property mandates an argument to indicate > whether or not to engage the "probe-only" mode, but the Seattle > DTS just provides a naked property, which is illegal. > > Also, it turns out that the board is perfectly happy without > probe-only, so let's drop this altogether. > > Signed-off-by: Marc Zyngier > --- > arch/arm64/boot/dts/amd/amd-overdrive.dts | 1 - > 1 file changed, 1 deletion(-) > > diff --git a/arch/arm64/boot/dts/amd/amd-overdrive.dts b/arch/arm64/boot/dts/amd/amd-overdrive.dts > index 564a3f7..128fa94 100644 > --- a/arch/arm64/boot/dts/amd/amd-overdrive.dts > +++ b/arch/arm64/boot/dts/amd/amd-overdrive.dts > @@ -14,7 +14,6 @@ > > chosen { > stdout-path = &serial0; > - linux,pci-probe-only; > }; > }; > > Thanks, I was planning to do this as well. The embedded DT in the UEFI FW will soon remove this property. Acked-by: Suravee Suthikulpanit Suravee