Re: [PATCH V2 03/10] Documentation: dt-bindings: firmware: tegra: add bindings of the BPMP

[Joseph Lo]

On 07/14/2016 03:41 AM, Stephen Warren wrote:
> On 07/05/2016 03:04 AM, Joseph Lo wrote:
> > The BPMP is a specific processor in Tegra chip, which is designed for
> > booting process handling and offloading the power management, clock
> > management, and reset control tasks from the CPU. The binding document
> > defines the resources that would be used by the BPMP firmware driver,
> > which can create the interprocessor communication (IPC) between the CPU
> > and BPMP.
>
> > diff --git
> > a/Documentation/devicetree/bindings/firmware/nvidia,tegra186-bpmp.txt
> > b/Documentation/devicetree/bindings/firmware/nvidia,tegra186-bpmp.txt
>
> > +- Documentation/devicetree/bindings/mailbox/mailbox.txt
> > +- Documentation/devicetree/bindings/mailbox/nvidia,tegra186-hsp.txt
>
> > +- Documentation/devicetree/bindings/clock/clock-bindings.txt
> > +- include/dt-bindings/clock/tegra186-clock.h
> > +- Documentation/devicetree/bindings/reset/reset.txt
> > +- include/dt-bindings/reset/tegra186-reset.h
>
> If you end up needing to repost this, it would be nice to make all those
> file references more generic. In particular, some SW projects store
> binding docs somewhere other than Documentation/devicetree/bindings/
> (e.g. U-Boot uses doc/device-tree-bindings/), and it's possible that the
> header files aren't stored in include/ but somewhere else. To make these
> file references valid everywhere, I'd suggest using relative paths for
> the binding docs, and #include style paths for the headers, e.g.:
>
> ../clock/clock-bindings.txt
>
> <dt-bindings/clock/tegra186-clock.h>
OK. Will fix this.

Thanks,
-Joseph