On 04/07/2014 08:02 PM, Steffen Trumtrar wrote: > Hi! > > On Mon, Apr 07, 2014 at 11:10:12AM -0600, Jason Gunthorpe wrote: >> On Mon, Apr 07, 2014 at 02:24:07PM +0200, Michal Simek wrote: >> >>> Device-tree BSP and in 2014.01 there will be new BSP which just >>> generate them directly from the Vivado tools which just target your >>> reference design. You can connect your custom IP (or Xilinx or 3rd >>> party) directly to the GIC which using different IRQ sensitivity >>> with whatever register addresses and make no sense to write it by >>> hand. >> >> On our Zynq design here we ended up being unwilling to use platform >> generation from Vivado. Basically all our IP was custom, so there was >> no win at all to invoking the complexity of the automatic tools. >> >> Thus we write the DT by hand, and our DT is complex, integrating >> peripherals that span two FPGAs. >> >> I think the in-kernel DT should use the kernel conventions, which >> means using #include and the binding constants over magic values. >> > > ACK. > > If in doubt follow common mainline practice. Although using includes > for DT is not necessarily common practice, readability of DTs is > really important IMHO. Let me give you one example. When you add xilinx intc controller to zynq HW design which uses gic with headers you are using then you will find out that sensitivity for both controllers are just different. This is just one case I am aware of. I expect there will be one more. Also dtses from kernel are copied to other project because separation from kernel hasn't be done but there is any plan regarding this. Using dtc preprocessor and macros improve DTS readability but at the same time force other users to copy all necessary files from the kernel to that projects which is just hassle. With example above there are also cases where using macros is just wrong that's why I don't want to start to use it. Thanks, Michal -- Michal Simek, Ing. (M.Eng), OpenPGP -> KeyID: FE3D1F91 w: www.monstr.eu p: +42-0-721842854 Maintainer of Linux kernel - Microblaze cpu - http://www.monstr.eu/fdt/ Maintainer of Linux kernel - Xilinx Zynq ARM architecture Microblaze U-BOOT custodian and responsible for u-boot arm zynq platform
Attachment:
signature.asc
Description: OpenPGP digital signature
