On Thursday, August 07, 2014 12:12 PM, Fabio Estevam wrote: > On Thu, Aug 7, 2014 at 12:01 AM, Jingoo Han <jg1.han@xxxxxxxxxxx> wrote: > > > I agree with Lucas Stach's opinion. > > Printing errors is necessary to let people change to fix the DT. > > According to the DT spec, using *ranges* is wrong. So, dev_err() > > can be used. > > Could you please point me to a dts that uses 'config' in the PCI node, > so I could use it as reference? Kishon already sent the patch to use 'config' in the PCI node. As far as I know, there is no objections. Please refer to the following links. [RESEND PATCH 8/8] ARM: dts: dra7: Add dt data for PCIe controller (http://lists.infradead.org/pipermail/linux-arm-kernel/2014-July/272131.html) [PATCH v3 3/4] Documentation: pci: ti: Add dt binding documentation for PCIe in DRA7xx (http://www.spinics.net/lists/linux-pci/msg32988.html) Best regards, Jingoo Han > > Thanks -- To unsubscribe from this list: send the line "unsubscribe linux-pci" in the body of a message to majordomo@xxxxxxxxxxxxxxx More majordomo info at http://vger.kernel.org/majordomo-info.html