From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1752502AbcFCIpv (ORCPT ); Fri, 3 Jun 2016 04:45:51 -0400 Received: from mx1.redhat.com ([209.132.183.28]:52736 "EHLO mx1.redhat.com" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1752250AbcFCIpt convert rfc822-to-8bit (ORCPT ); Fri, 3 Jun 2016 04:45:49 -0400 Message-ID: <1464943546.24775.131.camel@redhat.com> Subject: Re: [PATCH v2 7/9] ARM: bcm2835: Add devicetree for the Raspberry Pi 3. From: Gerd Hoffmann To: Arnd Bergmann Cc: linux-arm-kernel@lists.infradead.org, linux-rpi-kernel@lists.infradead.org, Mark Rutland , "open list:OPEN FIRMWARE AND FLATTENED DEVICE TREE BINDINGS" , Florian Fainelli , Catalin Marinas , Scott Branden , Pawel Moll , Ian Campbell , Ray Jui , Will Deacon , Russell King , open list , Eric Anholt , Rob Herring , "open list:BROADCOM BCM281XX/BCM11XXX/BCM216XX ARM ARCHITE..." , Kumar Gala Date: Fri, 03 Jun 2016 10:45:46 +0200 In-Reply-To: <5490184.9quKRk1Imx@wuerfel> References: <1464934708-24769-1-git-send-email-kraxel@redhat.com> <1464934708-24769-8-git-send-email-kraxel@redhat.com> <5490184.9quKRk1Imx@wuerfel> Content-Type: text/plain; charset="UTF-8" Content-Transfer-Encoding: 8BIT Mime-Version: 1.0 X-Greylist: Sender IP whitelisted, not delayed by milter-greylist-4.5.16 (mx1.redhat.com [10.5.110.26]); Fri, 03 Jun 2016 08:45:48 +0000 (UTC) Sender: linux-kernel-owner@vger.kernel.org List-ID: X-Mailing-List: linux-kernel@vger.kernel.org On Fr, 2016-06-03 at 09:38 +0200, Arnd Bergmann wrote: > On Friday, June 3, 2016 8:18:26 AM CEST Gerd Hoffmann wrote: > > diff --git a/arch/arm/boot/dts/Makefile b/arch/arm/boot/dts/Makefile > > index 06b6c2d..71e2d36 100644 > > --- a/arch/arm/boot/dts/Makefile > > +++ b/arch/arm/boot/dts/Makefile > > @@ -66,7 +66,8 @@ dtb-$(CONFIG_ARCH_BCM2835) += \ > > bcm2835-rpi-b-rev2.dtb \ > > bcm2835-rpi-b-plus.dtb \ > > bcm2835-rpi-a-plus.dtb \ > > - bcm2836-rpi-2-b.dtb > > + bcm2836-rpi-2-b.dtb \ > > + ../../../arm64/boot/dts/broadcom/bcm2837-rpi-3-b.dtb > > dtb-$(CONFIG_ARCH_BCM_5301X) += \ > > bcm4708-asus-rt-ac56u.dtb \ > > bcm4708-asus-rt-ac68u.dtb \ > > Let's make this hunk a separate patch so we can discuss it on its > own. Hmm, I through we where in agreement to do it this way ... > I'm not really happy about the way it looks (in general, using ../../../ > anywhere in the kernel looks hacky. > > How about a separate > > dtb-$(CONFIG_ARCH_BCM2835) += > ../../../arm64/boot/dts/broadcom/ Not working. Tried "subdir-$(CONFIG_ARCH_BCM) += ...". Didn't work either. Same for "dts-dirs += ...". Running out of ideas. Guess I have to leave this to someone who knows kbuild better than me. cheers, Gerd