dt-bindings: fpga: fix freeze controller compatible in region doc
Subsystems (2)
PATCH 1/1 dt-bindings: fpga: fix freeze controller compatible in region doc
Documentation/devicetree/bindings/fpga/fpga-region.txt has an error regarding the freeze controller bridge, secifically:
compatible = "altr,freeze-bridge";
The compatibility string should be "altr,freeze-bridge-controller" as set forth in altera-freeze-bridge.txt.
--- Documentation/devicetree/bindings/fpga/fpga-region.txt | 4 ++-- 1 file changed, 2 insertions(+), 2 deletions(-)
Documentation/devicetree/bindings/fpga/fpga-region.txt +2 -2
@@ -415,7 +415,7 @@ DT Overlay contains: firmware-name = "base.rbf"; fpga-bridge@4400 { - compatible = "altr,freeze-bridge"; + compatible = "altr,freeze-bridge-controller"; reg = <0x4400 0x10>; fpga_region1: fpga-region1 {
@@ -427,7 +427,7 @@ DT Overlay contains: }; fpga-bridge@4420 { - compatible = "altr,freeze-bridge"; + compatible = "altr,freeze-bridge-controller"; reg = <0x4420 0x10>; fpga_region2: fpga-region2 {
Signature · 1 line
2.7.4
Hi Alan,
looks like it's just the examples, right? So not sure someone would be affected by that, yet?
Quoted 9 lines · Alan Tull
Documentation/devicetree/bindings/fpga/fpga-region.txthas an error regarding the freeze controller bridge, secifically:compatible = "altr,freeze-bridge";
The compatibility string should be "altr,freeze-bridge-controller" as set forth in altera-freeze-bridge.txt.
Signed-off-by Alan Tull <[email protected]>
Cheers,
Moritz
Quoted 4 lines · Moritz Fischer
Hi Alan,
looks like it's just the examples, right?
Yes, it's in the examples of use, not the actual freeze bridge bindings doc that I'm correcting.
Quoted 2 lines
So not sure someone
would be affected by that, yet?
It's taken two years for anybody to notice and complain about it :) so the effect has not been very terrible. But it's good to correct it since it's been noticed.
Quoted 10 lines · Alan Tull
Quoted 8 lines
Documentation/devicetree/bindings/fpga/fpga-region.txthas an error regarding the freeze controller bridge, secifically:compatible = "altr,freeze-bridge";
The compatibility string should be "altr,freeze-bridge-controller" as set forth in altera-freeze-bridge.txt.
Quoted 7 lines
Quoted 1 line
Signed-off-by Alan Tull <[email protected]>Acked-by Moritz Fischer <[email protected]>Cheers,
Moritz
Thanks Moritz!
Alan
Quoted 5 lines · Alan Tull
Documentation/devicetree/bindings/fpga/fpga-region.txthas an error regarding the freeze controller bridge, secifically:compatible = "altr,freeze-bridge";
Quoted 8 lines
The compatibility string should be "altr,freeze-bridge-controller" as set forth in altera-freeze-bridge.txt.
Signed-off-by Alan Tull <[email protected]>Reported-by Dalon Westergreen <[email protected]>--- Documentation/devicetree/bindings/fpga/fpga-region.txt | 4 ++-- 1 file changed, 2 insertions(+), 2 deletions(-)
Applied.
Rob