From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S965290AbbLORDV (ORCPT ); Tue, 15 Dec 2015 12:03:21 -0500 Received: from mout.kundenserver.de ([217.72.192.73]:65017 "EHLO mout.kundenserver.de" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S965044AbbLORDO (ORCPT ); Tue, 15 Dec 2015 12:03:14 -0500 From: Arnd Bergmann To: Sergei Ianovich Cc: linux-kernel@vger.kernel.org, Daniel Mack , Robert Jarzmik , Rob Herring , Pawel Moll , Mark Rutland , Ian Campbell , Kumar Gala , Russell King , Thierry Reding , Jonathan Cameron , Dmitry Torokhov , Philipp Zabel , Arnaud Ebalard , Kuninori Morimoto , "open list:OPEN FIRMWARE AND FLATTENED DEVICE TREE BINDINGS" , "moderated list:ARM PORT" Subject: Re: [PATCH v5 2/2] arm: pxa27x: support for ICP DAS LP-8x4x w/ DT Date: Tue, 15 Dec 2015 18:02:19 +0100 Message-ID: <1843454.zCnLJtF7B6@wuerfel> User-Agent: KMail/4.11.5 (Linux/3.16.0-10-generic; KDE/4.11.5; x86_64; ; ) In-Reply-To: <1450197727.21989.9.camel@gmail.com> References: <1449700088-28076-3-git-send-email-ynvich@gmail.com> <7496839.vMjVcqPe8Q@wuerfel> <1450197727.21989.9.camel@gmail.com> MIME-Version: 1.0 Content-Transfer-Encoding: 7Bit Content-Type: text/plain; charset="us-ascii" X-Provags-ID: V03:K0:Lkm+KY+5Nt6toOze6i5oy3oqGz/STSWLKlPA371gPzEpsxTbavw 9JJuKYou+jIF27ooj0T7cBxOFCVaxRW+cFpn6zfc21i/5kqDrExOjcoKvl1i7I7YOcuEYyn Raps6VVNJ18yp2QSvvajzjlpkKLSs5Fi6w70kpsFkLzxxVD6uOLj3/Sg2D3OHu22qxrV8AK bq/vieiQYIUdCKWKBEQrw== X-UI-Out-Filterresults: notjunk:1;V01:K0:XKNOaXxGY9c=:nXQ9qbtGNtEdab+JXLvMLW 5E5DhfWmXF8nFgai4rKegreKXQW1VO+2oZ7CAr2jjl99itIX+uxJkQ3POTIhGhWiTAwtKT63D 9gm4TKf22TBfSgfkWivk9bB5RN7XDrkvPOQG0cEksBt1HDbPHboVs0e+pkLKGmx+Mzbx798pS Fdl1AYiRDdTajx3+3vEPR04su+DMw188Tg4Dugu81gX3nTvX1CbjKPFa5RJFxlNFFV1nFwHR8 k+frjC0xzszoUziI75tzFOr+6ONogMKvv7JVeq5R2ImjaI8f133VevU9fgKwok+bj1s0teEI9 +Q21a4ffwUWIudRqkBd/paY6U8YMuiTjh0ElWLAB4T/B9KgOKjimE5T4vK5pDDc9TRtTn5QYb zkniDH3Hq0GCV1kziZcQ0QPLgIiFso62ZljMQ7EDFR501pW0PSYQsG4Zcpqk40TsW8FG+vhm+ EjJnxCoDrr6gHP9dj2yAcTlo9AzWy0hksaTuNVb0rf7pN1G6+BrEKvx1Rj7KUy9/iCPXKIPMD EsCKy50YxoJ8mjmqCx+zJNxp7HrxHTBngTGSn+WHEbDw4rmc04Oz1Htaj3TgZcjzgaq18AJ2x HoUt2zRVXN2LdrXFdLPuryv0gSdnF2Aj3YfZuMvO//Hq3HdoDZsPeslDr90g3ufCwAakj9kKZ LWPopllbHyPeWEX3630zDNEhQQV5wWE+8WEP/zxogUDatT6zCk5raPs7exlTwJUcTBGNaHIeP 4S1R4PHgPPVFWSVR Sender: linux-kernel-owner@vger.kernel.org List-ID: X-Mailing-List: linux-kernel@vger.kernel.org On Tuesday 15 December 2015 19:42:07 Sergei Ianovich wrote: > On Tue, 2015-12-15 at 17:32 +0100, Arnd Bergmann wrote: > > On Tuesday 15 December 2015 19:27:50 Sergei Ianovich wrote: > > > .../devicetree/bindings/vendor-prefixes.txt | 1 + > > > arch/arm/boot/dts/Makefile | 3 + > > > arch/arm/boot/dts/pxa27x-lp8x4x-i105.dts | 50 ++++ > > > arch/arm/boot/dts/pxa27x-lp8x4x.dts | 259 > > > +++++++++++++++++++++ > > > arch/arm/configs/lp8x4x_defconfig | 176 > > > ++++++++++++++ > > > > > > > I had not noticed earlier that you are adding a new defconfig file. > > PXA is > > already the platform with the most defconfig files, and I'd rather > > like > > to see that reduced than increased. > > > > Is there a chance you could merge this one with some of the existing > > files > > into one configuration that handles them all? > > There are several board-specific devices on LP8x4x: custom FPGA, custom > UART, custom IRQ on FPGA, custom parallel bus for industrial IO. The > defconfig file could alert potential users to this fact. If this is not > a sufficient reason to have a defconfig file, it can be dropped. > > I use the full .config anyway I would like those drivers to be enabled in some defconfig, so we get compile-time coverage, but we generally stopped having one-config-per-board files. Maybe we can have a pxa_defconfig file that enables lots of boards and then we remove the individual configs? We don't have to remove them all at once, but it would make me very happy if we could at least kill off some of the ones that are not used regularly. Arnd From mboxrd@z Thu Jan 1 00:00:00 1970 From: Arnd Bergmann Subject: Re: [PATCH v5 2/2] arm: pxa27x: support for ICP DAS LP-8x4x w/ DT Date: Tue, 15 Dec 2015 18:02:19 +0100 Message-ID: <1843454.zCnLJtF7B6@wuerfel> References: <1449700088-28076-3-git-send-email-ynvich@gmail.com> <7496839.vMjVcqPe8Q@wuerfel> <1450197727.21989.9.camel@gmail.com> Mime-Version: 1.0 Content-Type: text/plain; charset="us-ascii" Content-Transfer-Encoding: 7Bit Return-path: In-Reply-To: <1450197727.21989.9.camel@gmail.com> Sender: linux-kernel-owner@vger.kernel.org To: Sergei Ianovich Cc: linux-kernel@vger.kernel.org, Daniel Mack , Robert Jarzmik , Rob Herring , Pawel Moll , Mark Rutland , Ian Campbell , Kumar Gala , Russell King , Thierry Reding , Jonathan Cameron , Dmitry Torokhov , Philipp Zabel , Arnaud Ebalard , Kuninori Morimoto , "open list:OPEN FIRMWARE AND FLATTENED DEVICE TREE BINDINGS" , "moderated list:ARM PORT" List-Id: devicetree@vger.kernel.org On Tuesday 15 December 2015 19:42:07 Sergei Ianovich wrote: > On Tue, 2015-12-15 at 17:32 +0100, Arnd Bergmann wrote: > > On Tuesday 15 December 2015 19:27:50 Sergei Ianovich wrote: > > > .../devicetree/bindings/vendor-prefixes.txt | 1 + > > > arch/arm/boot/dts/Makefile | 3 + > > > arch/arm/boot/dts/pxa27x-lp8x4x-i105.dts | 50 ++++ > > > arch/arm/boot/dts/pxa27x-lp8x4x.dts | 259 > > > +++++++++++++++++++++ > > > arch/arm/configs/lp8x4x_defconfig | 176 > > > ++++++++++++++ > > > > > > > I had not noticed earlier that you are adding a new defconfig file. > > PXA is > > already the platform with the most defconfig files, and I'd rather > > like > > to see that reduced than increased. > > > > Is there a chance you could merge this one with some of the existing > > files > > into one configuration that handles them all? > > There are several board-specific devices on LP8x4x: custom FPGA, custom > UART, custom IRQ on FPGA, custom parallel bus for industrial IO. The > defconfig file could alert potential users to this fact. If this is not > a sufficient reason to have a defconfig file, it can be dropped. > > I use the full .config anyway I would like those drivers to be enabled in some defconfig, so we get compile-time coverage, but we generally stopped having one-config-per-board files. Maybe we can have a pxa_defconfig file that enables lots of boards and then we remove the individual configs? We don't have to remove them all at once, but it would make me very happy if we could at least kill off some of the ones that are not used regularly. Arnd From mboxrd@z Thu Jan 1 00:00:00 1970 From: arnd@arndb.de (Arnd Bergmann) Date: Tue, 15 Dec 2015 18:02:19 +0100 Subject: [PATCH v5 2/2] arm: pxa27x: support for ICP DAS LP-8x4x w/ DT In-Reply-To: <1450197727.21989.9.camel@gmail.com> References: <1449700088-28076-3-git-send-email-ynvich@gmail.com> <7496839.vMjVcqPe8Q@wuerfel> <1450197727.21989.9.camel@gmail.com> Message-ID: <1843454.zCnLJtF7B6@wuerfel> To: linux-arm-kernel@lists.infradead.org List-Id: linux-arm-kernel.lists.infradead.org On Tuesday 15 December 2015 19:42:07 Sergei Ianovich wrote: > On Tue, 2015-12-15 at 17:32 +0100, Arnd Bergmann wrote: > > On Tuesday 15 December 2015 19:27:50 Sergei Ianovich wrote: > > > .../devicetree/bindings/vendor-prefixes.txt | 1 + > > > arch/arm/boot/dts/Makefile | 3 + > > > arch/arm/boot/dts/pxa27x-lp8x4x-i105.dts | 50 ++++ > > > arch/arm/boot/dts/pxa27x-lp8x4x.dts | 259 > > > +++++++++++++++++++++ > > > arch/arm/configs/lp8x4x_defconfig | 176 > > > ++++++++++++++ > > > > > > > I had not noticed earlier that you are adding a new defconfig file. > > PXA is > > already the platform with the most defconfig files, and I'd rather > > like > > to see that reduced than increased. > > > > Is there a chance you could merge this one with some of the existing > > files > > into one configuration that handles them all? > > There are several board-specific devices on LP8x4x: custom FPGA, custom > UART, custom IRQ on FPGA, custom parallel bus for industrial IO. The > defconfig file could alert potential users to this fact. If this is not > a sufficient reason to have a defconfig file, it can be dropped. > > I use the full .config anyway I would like those drivers to be enabled in some defconfig, so we get compile-time coverage, but we generally stopped having one-config-per-board files. Maybe we can have a pxa_defconfig file that enables lots of boards and then we remove the individual configs? We don't have to remove them all at once, but it would make me very happy if we could at least kill off some of the ones that are not used regularly. Arnd