| Message ID | 20200124105753.15976-1-dafna.hirschfeld@collabora.com (mailing list archive) |
|---|---|
| State | Mainlined |
| Commit | 0ea5115a02f08a86e1fe01b837d941263ba870a3 |
| Headers | show |
| Series | dt-bindings: fix compilation error of the example in marvell,mmp3-hsic-phy.yaml | expand |
On Fri, Jan 24, 2020 at 08:08:58AM -0600, Rob Herring wrote: > On Fri, Jan 24, 2020 at 4:58 AM Dafna Hirschfeld > <dafna.hirschfeld@collabora.com> wrote: > > > > Running `make dt_binging_check`, gives the error: > > > > DTC Documentation/devicetree/bindings/phy/marvell,mmp3-hsic-phy.example.dt.yaml > > Error: Documentation/devicetree/bindings/phy/marvell,mmp3-hsic-phy.example.dts:20.41-42 syntax error > > FATAL ERROR: Unable to parse input tree > > > > This is because the example uses the macro GPIO_ACTIVE_HIGH which > > is defined in gpio.h but the include of this header is missing. > > Add the include to fix the error. > > > > Signed-off-by: Dafna Hirschfeld <dafna.hirschfeld@collabora.com> > > Fixes: f6f149604eef ("dt-bindings: phy: Add binding for marvell,mmp3-hsic-phy") > Acked-by: Rob Herring <robh@kernel.org> > > Arnd, Olof, The above commit is in your tree. Please apply this. Applied. Thanks, -Olof -=-=-=-=-=-=-=-=-=-=-=- Links: You receive all messages sent to this group. View/Reply Online (#1317): https://linux.kernel.org/g/patchwork-soc/message/1317 Mute This Topic: https://linux.kernel.org/mt/70071013/1554929 Group Owner: patchwork-soc+owner@linux.kernel.org Unsubscribe: https://linux.kernel.org/g/patchwork-soc/unsub [patchwork-linux-kernel-org@patchwork.kernel.org] -=-=-=-=-=-=-=-=-=-=-=-
Hello: This patch was applied to soc/soc.git (refs/heads/for-next). On Fri, 24 Jan 2020 11:57:53 +0100 you wrote: > Running `make dt_binging_check`, gives the error: > > DTC Documentation/devicetree/bindings/phy/marvell,mmp3-hsic-phy.example.dt.yaml > Error: Documentation/devicetree/bindings/phy/marvell,mmp3-hsic-phy.example.dts:20.41-42 syntax error > FATAL ERROR: Unable to parse input tree > > This is because the example uses the macro GPIO_ACTIVE_HIGH which > is defined in gpio.h but the include of this header is missing. > Add the include to fix the error. > > [...] Here is a summary with links: - dt-bindings: fix compilation error of the example in marvell,mmp3-hsic-phy.yaml https://git.kernel.org/soc/soc/c/0ea5115a02f08a86e1fe01b837d941263ba870a3 You are awesome, thank you!
diff --git a/Documentation/devicetree/bindings/phy/marvell,mmp3-hsic-phy.yaml b/Documentation/devicetree/bindings/phy/marvell,mmp3-hsic-phy.yaml index 7917a95cda78..5ab436189f3b 100644 --- a/Documentation/devicetree/bindings/phy/marvell,mmp3-hsic-phy.yaml +++ b/Documentation/devicetree/bindings/phy/marvell,mmp3-hsic-phy.yaml @@ -33,6 +33,7 @@ required: examples: - | + #include <dt-bindings/gpio/gpio.h> hsic-phy@f0001800 { compatible = "marvell,mmp3-hsic-phy"; reg = <0xf0001800 0x40>;
Running `make dt_binging_check`, gives the error: DTC Documentation/devicetree/bindings/phy/marvell,mmp3-hsic-phy.example.dt.yaml Error: Documentation/devicetree/bindings/phy/marvell,mmp3-hsic-phy.example.dts:20.41-42 syntax error FATAL ERROR: Unable to parse input tree This is because the example uses the macro GPIO_ACTIVE_HIGH which is defined in gpio.h but the include of this header is missing. Add the include to fix the error. Signed-off-by: Dafna Hirschfeld <dafna.hirschfeld@collabora.com> --- Documentation/devicetree/bindings/phy/marvell,mmp3-hsic-phy.yaml | 1 + 1 file changed, 1 insertion(+)