Return-Path: Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S976275AbdDXS1D (ORCPT ); Mon, 24 Apr 2017 14:27:03 -0400 Received: from mail-ua0-f193.google.com ([209.85.217.193]:33575 "EHLO mail-ua0-f193.google.com" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S976322AbdDXS04 (ORCPT ); Mon, 24 Apr 2017 14:26:56 -0400 MIME-Version: 1.0 X-Originating-IP: [209.133.79.5] In-Reply-To: <20170330002605.15213-1-eric@anholt.net> References: <20170330002605.15213-1-eric@anholt.net> From: Olof Johansson Date: Mon, 24 Apr 2017 11:26:54 -0700 Message-ID: Subject: Re: [PATCH] ARM: dts: Add devicetree for the Raspberry Pi 3, for arm32 (v4) To: Eric Anholt Cc: "linux-arm-kernel@lists.infradead.org" , Lee Jones , Florian Fainelli , Rob Herring , Mark Rutland , "devicetree@vger.kernel.org" , linux-rpi-kernel@lists.infradead.org, "linux-kernel@vger.kernel.org" , Stephen Warren , Stefan Wahren , Broadcom Kernel Feedback List , Gerd Hoffmann Content-Type: text/plain; charset=UTF-8 Sender: linux-kernel-owner@vger.kernel.org List-ID: X-Mailing-List: linux-kernel@vger.kernel.org Content-Length: 1520 Lines: 43 Hi, On Wed, Mar 29, 2017 at 5:26 PM, Eric Anholt wrote: > Raspbian and Fedora have decided to support the Pi3 in 32-bit mode for > now, so it's useful to be able to test that mode on an upstream > kernel. It's also been useful for me to use the same board for 32-bit > and 64-bit development. > > Signed-off-by: Eric Anholt > --- > > v1: Gerd's patch that put the ../../../arm64/... link in the Makefile > v2: Michael's patch that #included from ../../../arm64/... in a new > bcm2837-rpi-3-b.dts. > v3: Mine, using symlinks to make sure that we don't break the split DT > tree. > v4: Rely on the new include/arm64 symlink. > > Assuming positive review feedback, I assume it would be acceptable to > merge the shared/dt-symlinks branch in a PR of my own for the 32-bit > DT branch? > > arch/arm/boot/dts/Makefile | 1 + > 1 file changed, 1 insertion(+) > > diff --git a/arch/arm/boot/dts/Makefile b/arch/arm/boot/dts/Makefile > index 011808490fed..27d258cb50f2 100644 > --- a/arch/arm/boot/dts/Makefile > +++ b/arch/arm/boot/dts/Makefile > @@ -72,6 +72,7 @@ dtb-$(CONFIG_ARCH_BCM2835) += \ > bcm2835-rpi-b-plus.dtb \ > bcm2835-rpi-a-plus.dtb \ > bcm2836-rpi-2-b.dtb \ > + include/arm64/broadcom/bcm2837-rpi-3-b.dtb \ Building straight out of (and into) the include dir is a little odd here. A tiny wrapper *.dtb in this dir, that just includes a shared dts/dtsi would be a lot nicer. If you do that, we can still pick it up for 4.12. -Olof