Re: [PATCH v5 2/2] arm: pxa27x: support for ICP DAS LP-8x4x w/ DT

From: Sergei Ianovich
Date: Tue Dec 15 2015 - 11:42:22 EST


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 :)
--
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majordomo@xxxxxxxxxxxxxxx
More majordomo info at http://vger.kernel.org/majordomo-info.html
Please read the FAQ at http://www.tux.org/lkml/