[PATCH v2 03/10] Documentation/dts: Add bindings for QIXIS FPGA controller found on FSL boards
From: Sharma Bhupesh <hidden>
Date: 2015-09-05 08:11:59
Also in:
linux-clk, linux-devicetree
From: pku.leo at gmail.com [mailto:pku.leo at gmail.com] Sent: Saturday, September 05, 2015 2:43 AM On Fri, Sep 4, 2015 at 3:16 PM, Sharma Bhupesh [off-list ref] wrote:quoted
quoted
From: pku.leo at gmail.com [mailto:pku.leo at gmail.com] Sent: Friday, September 04, 2015 10:27 PM On Fri, Sep 4, 2015 at 1:57 AM, Bhupesh Sharma [off-list ref] wrote:quoted
This patch adds bindings for QIXIS FPGA controller found on FSLboards.quoted
quoted
A general comment: when you are updating the device tree bindings. You should cc the devicetree at vger.kernel.org mailing list.quoted
Some Freescale boards like LS2080AQDS/LS2080ARDB have an on-board FPGA/CPLD connected to the IFC controller. The bindings specified in this patch cater to those on-board FPGA/CPLD controllers.We already have the binding defined in Documentation/devicetree/bindings/powerpc/fsl/board.txt. We probably should just move it to a more generic location.Consolidation of powerpc and ARM bindings is something that needs to be targeted by a separate patch-series, perhaps in future. There are a number of duplications that can be looked into, but I don't think this patchset should depend on that.There might be some duplication, but we shouldn't be adding more. :) I would rather you directly updating that file than creating a new file even though the original one was in a wrong directory.
IMO a PowerPC binding update makes no sense in a ARM machine patchset sent on a arm specific mailing list. I would like to wait to hear what maintainers have to say on the same. Regards, Bhupesh