[coreboot-gerrit] Change in coreboot[master]: mainboard/opencellular/elgon: Add mainboard support

build bot (Jenkins) (Code Review) gerrit at coreboot.org
Fri Aug 24 14:37:48 CEST 2018


build bot (Jenkins) has posted comments on this change. ( https://review.coreboot.org/28024 )

Change subject: mainboard/opencellular/elgon: Add mainboard support
......................................................................


Patch Set 8:

(129 comments)

https://review.coreboot.org/#/c/28024/8/src/mainboard/opencellular/elgon/gbcv2.dts
File src/mainboard/opencellular/elgon/gbcv2.dts:

https://review.coreboot.org/#/c/28024/8/src/mainboard/opencellular/elgon/gbcv2.dts@20
PS8, Line 20: 	compatible = "cavium,thunder-81xx";
DT compatible string "cavium,thunder-81xx" appears un-documented -- check util/lint/checkpatch.pl/Documentation/devicetree/bindings/


https://review.coreboot.org/#/c/28024/8/src/mainboard/opencellular/elgon/gbcv2.dts@20
PS8, Line 20: 	compatible = "cavium,thunder-81xx";
DT compatible string vendor "cavium" appears un-documented -- check util/lint/checkpatch.pl/Documentation/devicetree/bindings/vendor-prefixes.txt


https://review.coreboot.org/#/c/28024/8/src/mainboard/opencellular/elgon/gbcv2.dts@26
PS8, Line 26: 		compatible = "arm,psci-0.2";
DT compatible string "arm,psci-0.2" appears un-documented -- check util/lint/checkpatch.pl/Documentation/devicetree/bindings/


https://review.coreboot.org/#/c/28024/8/src/mainboard/opencellular/elgon/gbcv2.dts@26
PS8, Line 26: 		compatible = "arm,psci-0.2";
DT compatible string vendor "arm" appears un-documented -- check util/lint/checkpatch.pl/Documentation/devicetree/bindings/vendor-prefixes.txt


https://review.coreboot.org/#/c/28024/8/src/mainboard/opencellular/elgon/gbcv2.dts@58
PS8, Line 58: 			compatible = "cavium,thunder", "arm,armv8";
DT compatible string "cavium,thunder" appears un-documented -- check util/lint/checkpatch.pl/Documentation/devicetree/bindings/


https://review.coreboot.org/#/c/28024/8/src/mainboard/opencellular/elgon/gbcv2.dts@58
PS8, Line 58: 			compatible = "cavium,thunder", "arm,armv8";
DT compatible string vendor "cavium" appears un-documented -- check util/lint/checkpatch.pl/Documentation/devicetree/bindings/vendor-prefixes.txt


https://review.coreboot.org/#/c/28024/8/src/mainboard/opencellular/elgon/gbcv2.dts@58
PS8, Line 58: 			compatible = "cavium,thunder", "arm,armv8";
DT compatible string "arm,armv8" appears un-documented -- check util/lint/checkpatch.pl/Documentation/devicetree/bindings/


https://review.coreboot.org/#/c/28024/8/src/mainboard/opencellular/elgon/gbcv2.dts@58
PS8, Line 58: 			compatible = "cavium,thunder", "arm,armv8";
DT compatible string vendor "arm" appears un-documented -- check util/lint/checkpatch.pl/Documentation/devicetree/bindings/vendor-prefixes.txt


https://review.coreboot.org/#/c/28024/8/src/mainboard/opencellular/elgon/gbcv2.dts@69
PS8, Line 69: 			compatible = "cavium,thunder", "arm,armv8";
DT compatible string "cavium,thunder" appears un-documented -- check util/lint/checkpatch.pl/Documentation/devicetree/bindings/


https://review.coreboot.org/#/c/28024/8/src/mainboard/opencellular/elgon/gbcv2.dts@69
PS8, Line 69: 			compatible = "cavium,thunder", "arm,armv8";
DT compatible string vendor "cavium" appears un-documented -- check util/lint/checkpatch.pl/Documentation/devicetree/bindings/vendor-prefixes.txt


https://review.coreboot.org/#/c/28024/8/src/mainboard/opencellular/elgon/gbcv2.dts@69
PS8, Line 69: 			compatible = "cavium,thunder", "arm,armv8";
DT compatible string "arm,armv8" appears un-documented -- check util/lint/checkpatch.pl/Documentation/devicetree/bindings/


https://review.coreboot.org/#/c/28024/8/src/mainboard/opencellular/elgon/gbcv2.dts@69
PS8, Line 69: 			compatible = "cavium,thunder", "arm,armv8";
DT compatible string vendor "arm" appears un-documented -- check util/lint/checkpatch.pl/Documentation/devicetree/bindings/vendor-prefixes.txt


https://review.coreboot.org/#/c/28024/8/src/mainboard/opencellular/elgon/gbcv2.dts@80
PS8, Line 80: 			compatible = "cavium,thunder", "arm,armv8";
DT compatible string "cavium,thunder" appears un-documented -- check util/lint/checkpatch.pl/Documentation/devicetree/bindings/


https://review.coreboot.org/#/c/28024/8/src/mainboard/opencellular/elgon/gbcv2.dts@80
PS8, Line 80: 			compatible = "cavium,thunder", "arm,armv8";
DT compatible string vendor "cavium" appears un-documented -- check util/lint/checkpatch.pl/Documentation/devicetree/bindings/vendor-prefixes.txt


https://review.coreboot.org/#/c/28024/8/src/mainboard/opencellular/elgon/gbcv2.dts@80
PS8, Line 80: 			compatible = "cavium,thunder", "arm,armv8";
DT compatible string "arm,armv8" appears un-documented -- check util/lint/checkpatch.pl/Documentation/devicetree/bindings/


https://review.coreboot.org/#/c/28024/8/src/mainboard/opencellular/elgon/gbcv2.dts@80
PS8, Line 80: 			compatible = "cavium,thunder", "arm,armv8";
DT compatible string vendor "arm" appears un-documented -- check util/lint/checkpatch.pl/Documentation/devicetree/bindings/vendor-prefixes.txt


https://review.coreboot.org/#/c/28024/8/src/mainboard/opencellular/elgon/gbcv2.dts@91
PS8, Line 91: 			compatible = "cavium,thunder", "arm,armv8";
DT compatible string "cavium,thunder" appears un-documented -- check util/lint/checkpatch.pl/Documentation/devicetree/bindings/


https://review.coreboot.org/#/c/28024/8/src/mainboard/opencellular/elgon/gbcv2.dts@91
PS8, Line 91: 			compatible = "cavium,thunder", "arm,armv8";
DT compatible string vendor "cavium" appears un-documented -- check util/lint/checkpatch.pl/Documentation/devicetree/bindings/vendor-prefixes.txt


https://review.coreboot.org/#/c/28024/8/src/mainboard/opencellular/elgon/gbcv2.dts@91
PS8, Line 91: 			compatible = "cavium,thunder", "arm,armv8";
DT compatible string "arm,armv8" appears un-documented -- check util/lint/checkpatch.pl/Documentation/devicetree/bindings/


https://review.coreboot.org/#/c/28024/8/src/mainboard/opencellular/elgon/gbcv2.dts@91
PS8, Line 91: 			compatible = "cavium,thunder", "arm,armv8";
DT compatible string vendor "arm" appears un-documented -- check util/lint/checkpatch.pl/Documentation/devicetree/bindings/vendor-prefixes.txt


https://review.coreboot.org/#/c/28024/8/src/mainboard/opencellular/elgon/gbcv2.dts@102
PS8, Line 102: 		compatible = "cache";
DT compatible string "cache" appears un-documented -- check util/lint/checkpatch.pl/Documentation/devicetree/bindings/


https://review.coreboot.org/#/c/28024/8/src/mainboard/opencellular/elgon/gbcv2.dts@109
PS8, Line 109: 		compatible = "arm,armv8-timer";
DT compatible string "arm,armv8-timer" appears un-documented -- check util/lint/checkpatch.pl/Documentation/devicetree/bindings/


https://review.coreboot.org/#/c/28024/8/src/mainboard/opencellular/elgon/gbcv2.dts@109
PS8, Line 109: 		compatible = "arm,armv8-timer";
DT compatible string vendor "arm" appears un-documented -- check util/lint/checkpatch.pl/Documentation/devicetree/bindings/vendor-prefixes.txt


https://review.coreboot.org/#/c/28024/8/src/mainboard/opencellular/elgon/gbcv2.dts@114
PS8, Line 114: 		compatible = "cavium,thunder-pmu", "arm,armv8-pmuv3";
DT compatible string "cavium,thunder-pmu" appears un-documented -- check util/lint/checkpatch.pl/Documentation/devicetree/bindings/


https://review.coreboot.org/#/c/28024/8/src/mainboard/opencellular/elgon/gbcv2.dts@114
PS8, Line 114: 		compatible = "cavium,thunder-pmu", "arm,armv8-pmuv3";
DT compatible string vendor "cavium" appears un-documented -- check util/lint/checkpatch.pl/Documentation/devicetree/bindings/vendor-prefixes.txt


https://review.coreboot.org/#/c/28024/8/src/mainboard/opencellular/elgon/gbcv2.dts@114
PS8, Line 114: 		compatible = "cavium,thunder-pmu", "arm,armv8-pmuv3";
DT compatible string "arm,armv8-pmuv3" appears un-documented -- check util/lint/checkpatch.pl/Documentation/devicetree/bindings/


https://review.coreboot.org/#/c/28024/8/src/mainboard/opencellular/elgon/gbcv2.dts@114
PS8, Line 114: 		compatible = "cavium,thunder-pmu", "arm,armv8-pmuv3";
DT compatible string vendor "arm" appears un-documented -- check util/lint/checkpatch.pl/Documentation/devicetree/bindings/vendor-prefixes.txt


https://review.coreboot.org/#/c/28024/8/src/mainboard/opencellular/elgon/gbcv2.dts@119
PS8, Line 119: 		compatible = "regulator-fixed";
DT compatible string "regulator-fixed" appears un-documented -- check util/lint/checkpatch.pl/Documentation/devicetree/bindings/


https://review.coreboot.org/#/c/28024/8/src/mainboard/opencellular/elgon/gbcv2.dts@130
PS8, Line 130: 		compatible = "arm,gic-v3";
DT compatible string "arm,gic-v3" appears un-documented -- check util/lint/checkpatch.pl/Documentation/devicetree/bindings/


https://review.coreboot.org/#/c/28024/8/src/mainboard/opencellular/elgon/gbcv2.dts@130
PS8, Line 130: 		compatible = "arm,gic-v3";
DT compatible string vendor "arm" appears un-documented -- check util/lint/checkpatch.pl/Documentation/devicetree/bindings/vendor-prefixes.txt


https://review.coreboot.org/#/c/28024/8/src/mainboard/opencellular/elgon/gbcv2.dts@143
PS8, Line 143: 			compatible = "arm,gic-v3-its";
DT compatible string "arm,gic-v3-its" appears un-documented -- check util/lint/checkpatch.pl/Documentation/devicetree/bindings/


https://review.coreboot.org/#/c/28024/8/src/mainboard/opencellular/elgon/gbcv2.dts@143
PS8, Line 143: 			compatible = "arm,gic-v3-its";
DT compatible string vendor "arm" appears un-documented -- check util/lint/checkpatch.pl/Documentation/devicetree/bindings/vendor-prefixes.txt


https://review.coreboot.org/#/c/28024/8/src/mainboard/opencellular/elgon/gbcv2.dts@153
PS8, Line 153: 		compatible = "simple-bus";
DT compatible string "simple-bus" appears un-documented -- check util/lint/checkpatch.pl/Documentation/devicetree/bindings/


https://review.coreboot.org/#/c/28024/8/src/mainboard/opencellular/elgon/gbcv2.dts@160
PS8, Line 160: 			compatible = "fixed-clock";
DT compatible string "fixed-clock" appears un-documented -- check util/lint/checkpatch.pl/Documentation/devicetree/bindings/


https://review.coreboot.org/#/c/28024/8/src/mainboard/opencellular/elgon/gbcv2.dts@169
PS8, Line 169: 			compatible = "fixed-clock";
DT compatible string "fixed-clock" appears un-documented -- check util/lint/checkpatch.pl/Documentation/devicetree/bindings/


https://review.coreboot.org/#/c/28024/8/src/mainboard/opencellular/elgon/gbcv2.dts@178
PS8, Line 178: 			compatible = "arm,pl011", "arm,primecell";
DT compatible string "arm,pl011" appears un-documented -- check util/lint/checkpatch.pl/Documentation/devicetree/bindings/


https://review.coreboot.org/#/c/28024/8/src/mainboard/opencellular/elgon/gbcv2.dts@178
PS8, Line 178: 			compatible = "arm,pl011", "arm,primecell";
DT compatible string vendor "arm" appears un-documented -- check util/lint/checkpatch.pl/Documentation/devicetree/bindings/vendor-prefixes.txt


https://review.coreboot.org/#/c/28024/8/src/mainboard/opencellular/elgon/gbcv2.dts@178
PS8, Line 178: 			compatible = "arm,pl011", "arm,primecell";
DT compatible string "arm,primecell" appears un-documented -- check util/lint/checkpatch.pl/Documentation/devicetree/bindings/


https://review.coreboot.org/#/c/28024/8/src/mainboard/opencellular/elgon/gbcv2.dts@178
PS8, Line 178: 			compatible = "arm,pl011", "arm,primecell";
DT compatible string vendor "arm" appears un-documented -- check util/lint/checkpatch.pl/Documentation/devicetree/bindings/vendor-prefixes.txt


https://review.coreboot.org/#/c/28024/8/src/mainboard/opencellular/elgon/gbcv2.dts@187
PS8, Line 187: 			compatible = "arm,pl011", "arm,primecell";
DT compatible string "arm,pl011" appears un-documented -- check util/lint/checkpatch.pl/Documentation/devicetree/bindings/


https://review.coreboot.org/#/c/28024/8/src/mainboard/opencellular/elgon/gbcv2.dts@187
PS8, Line 187: 			compatible = "arm,pl011", "arm,primecell";
DT compatible string vendor "arm" appears un-documented -- check util/lint/checkpatch.pl/Documentation/devicetree/bindings/vendor-prefixes.txt


https://review.coreboot.org/#/c/28024/8/src/mainboard/opencellular/elgon/gbcv2.dts@187
PS8, Line 187: 			compatible = "arm,pl011", "arm,primecell";
DT compatible string "arm,primecell" appears un-documented -- check util/lint/checkpatch.pl/Documentation/devicetree/bindings/


https://review.coreboot.org/#/c/28024/8/src/mainboard/opencellular/elgon/gbcv2.dts@187
PS8, Line 187: 			compatible = "arm,pl011", "arm,primecell";
DT compatible string vendor "arm" appears un-documented -- check util/lint/checkpatch.pl/Documentation/devicetree/bindings/vendor-prefixes.txt


https://review.coreboot.org/#/c/28024/8/src/mainboard/opencellular/elgon/gbcv2.dts@196
PS8, Line 196: 			compatible = "arm,pl011", "arm,primecell";
DT compatible string "arm,pl011" appears un-documented -- check util/lint/checkpatch.pl/Documentation/devicetree/bindings/


https://review.coreboot.org/#/c/28024/8/src/mainboard/opencellular/elgon/gbcv2.dts@196
PS8, Line 196: 			compatible = "arm,pl011", "arm,primecell";
DT compatible string vendor "arm" appears un-documented -- check util/lint/checkpatch.pl/Documentation/devicetree/bindings/vendor-prefixes.txt


https://review.coreboot.org/#/c/28024/8/src/mainboard/opencellular/elgon/gbcv2.dts@196
PS8, Line 196: 			compatible = "arm,pl011", "arm,primecell";
DT compatible string "arm,primecell" appears un-documented -- check util/lint/checkpatch.pl/Documentation/devicetree/bindings/


https://review.coreboot.org/#/c/28024/8/src/mainboard/opencellular/elgon/gbcv2.dts@196
PS8, Line 196: 			compatible = "arm,pl011", "arm,primecell";
DT compatible string vendor "arm" appears un-documented -- check util/lint/checkpatch.pl/Documentation/devicetree/bindings/vendor-prefixes.txt


https://review.coreboot.org/#/c/28024/8/src/mainboard/opencellular/elgon/gbcv2.dts@205
PS8, Line 205: 			compatible = "arm,pl011", "arm,primecell";
DT compatible string "arm,pl011" appears un-documented -- check util/lint/checkpatch.pl/Documentation/devicetree/bindings/


https://review.coreboot.org/#/c/28024/8/src/mainboard/opencellular/elgon/gbcv2.dts@205
PS8, Line 205: 			compatible = "arm,pl011", "arm,primecell";
DT compatible string vendor "arm" appears un-documented -- check util/lint/checkpatch.pl/Documentation/devicetree/bindings/vendor-prefixes.txt


https://review.coreboot.org/#/c/28024/8/src/mainboard/opencellular/elgon/gbcv2.dts@205
PS8, Line 205: 			compatible = "arm,pl011", "arm,primecell";
DT compatible string "arm,primecell" appears un-documented -- check util/lint/checkpatch.pl/Documentation/devicetree/bindings/


https://review.coreboot.org/#/c/28024/8/src/mainboard/opencellular/elgon/gbcv2.dts@205
PS8, Line 205: 			compatible = "arm,pl011", "arm,primecell";
DT compatible string vendor "arm" appears un-documented -- check util/lint/checkpatch.pl/Documentation/devicetree/bindings/vendor-prefixes.txt


https://review.coreboot.org/#/c/28024/8/src/mainboard/opencellular/elgon/gbcv2.dts@214
PS8, Line 214: 			compatible = "arm,sbsa-gwdt";
DT compatible string "arm,sbsa-gwdt" appears un-documented -- check util/lint/checkpatch.pl/Documentation/devicetree/bindings/


https://review.coreboot.org/#/c/28024/8/src/mainboard/opencellular/elgon/gbcv2.dts@214
PS8, Line 214: 			compatible = "arm,sbsa-gwdt";
DT compatible string vendor "arm" appears un-documented -- check util/lint/checkpatch.pl/Documentation/devicetree/bindings/vendor-prefixes.txt


https://review.coreboot.org/#/c/28024/8/src/mainboard/opencellular/elgon/gbcv2.dts@215
PS8, Line 215: 			reg = <0x8440 0xa0000 0x0 0x1000 0x8440 0xb0000 0x0 0x1000>;
line over 80 characters


https://review.coreboot.org/#/c/28024/8/src/mainboard/opencellular/elgon/gbcv2.dts@220
PS8, Line 220: 			compatible = "cavium,smmu-v2";
DT compatible string "cavium,smmu-v2" appears un-documented -- check util/lint/checkpatch.pl/Documentation/devicetree/bindings/


https://review.coreboot.org/#/c/28024/8/src/mainboard/opencellular/elgon/gbcv2.dts@220
PS8, Line 220: 			compatible = "cavium,smmu-v2";
DT compatible string vendor "cavium" appears un-documented -- check util/lint/checkpatch.pl/Documentation/devicetree/bindings/vendor-prefixes.txt


https://review.coreboot.org/#/c/28024/8/src/mainboard/opencellular/elgon/gbcv2.dts@223
PS8, Line 223: 			interrupts = <0x0 0x44 0x4 0x0 0x45 0x4 0x0 0x45 0x4 0x0 0x45 0x4 0x0 0x45 0x4 0x0 0x45 0x4 0x0 0x45 0x4 0x0 0x45 0x4 0x0 0x45 0x4 0x0 0x45 0x4 0x0 0x45 0x4 0x0 0x45 0x4 0x0 0x45 0x4 0x0 0x45 0x4 0x0 0x45 0x4 0x0 0x45 0x4 0x0 0x45 0x4 0x0 0x45 0x4 0x0 0x45 0x4 0x0 0x45 0x4 0x0 0x45 0x4 0x0 0x45 0x4 0x0 0x45 0x4 0x0 0x45 0x4 0x0 0x45 0x4 0x0 0x45 0x4 0x0 0x45 0x4 0x0 0x45 0x4 0x0 0x45 0x4 0x0 0x45 0x4 0x0 0x45 0x4 0x0 0x45 0x4 0x0 0x45 0x4 0x0 0x45 0x4 0x0 0x45 0x4 0x0 0x45 0x4 0x0 0x45 0x4 0x0 0x45 0x4 0x0 0x45 0x4 0x0 0x45 0x4 0x0 0x45 0x4 0x0 0x45 0x4 0x0 0x45 0x4 0x0 0x45 0x4 0x0 0x45 0x4 0x0 0x45 0x4 0x0 0x45 0x4 0x0 0x45 0x4 0x0 0x45 0x4 0x0 0x45 0x4 0x0 0x45 0x4 0x0 0x45 0x4 0x0 0x45 0x4 0x0 0x45 0x4 0x0 0x45 0x4 0x0 0x45 0x4 0x0 0x45 0x4 0x0 0x45 0x4 0x0 0x45 0x4 0x0 0x45 0x4 0x0 0x45 0x4 0x0 0x45 0x4 0x0 0x45 0x4 0x0 0x45 0x4 0x0 0x45 0x4>;
line over 80 characters


https://review.coreboot.org/#/c/28024/8/src/mainboard/opencellular/elgon/gbcv2.dts@228
PS8, Line 228: 			compatible = "pci-host-ecam-generic";
DT compatible string "pci-host-ecam-generic" appears un-documented -- check util/lint/checkpatch.pl/Documentation/devicetree/bindings/


https://review.coreboot.org/#/c/28024/8/src/mainboard/opencellular/elgon/gbcv2.dts@238
PS8, Line 238: 			ranges = <0x3000000 0x8010 0x0 0x8010 0x0 0x80 0x0 0x3000000 0x8100 0x0 0x8100 0x0 0x80 0x0 0x3000000 0x8680 0x0 0x8680 0x0 0x160 0x28000000 0x3000000 0x87e0 0x2c000000 0x87e0 0x2c000000 0x0 0x94000000 0x3000000 0x8400 0x0 0x8400 0x0 0x10 0x0 0x3000000 0x8430 0x0 0x8430 0x0 0x2 0x0 0x3000000 0x87e0 0xc6000000 0x87e0 0xc6000000 0x1f 0x3a000000>;
line over 80 characters


https://review.coreboot.org/#/c/28024/8/src/mainboard/opencellular/elgon/gbcv2.dts@243
PS8, Line 243: 				compatible = "pci-bridge", "cavium,thunder-8890-mrml-bridge";
DT compatible string "pci-bridge" appears un-documented -- check util/lint/checkpatch.pl/Documentation/devicetree/bindings/


https://review.coreboot.org/#/c/28024/8/src/mainboard/opencellular/elgon/gbcv2.dts@243
PS8, Line 243: 				compatible = "pci-bridge", "cavium,thunder-8890-mrml-bridge";
DT compatible string "cavium,thunder-8890-mrml-bridge" appears un-documented -- check util/lint/checkpatch.pl/Documentation/devicetree/bindings/


https://review.coreboot.org/#/c/28024/8/src/mainboard/opencellular/elgon/gbcv2.dts@243
PS8, Line 243: 				compatible = "pci-bridge", "cavium,thunder-8890-mrml-bridge";
DT compatible string vendor "cavium" appears un-documented -- check util/lint/checkpatch.pl/Documentation/devicetree/bindings/vendor-prefixes.txt


https://review.coreboot.org/#/c/28024/8/src/mainboard/opencellular/elgon/gbcv2.dts@243
PS8, Line 243: 				compatible = "pci-bridge", "cavium,thunder-8890-mrml-bridge";
line over 80 characters


https://review.coreboot.org/#/c/28024/8/src/mainboard/opencellular/elgon/gbcv2.dts@246
PS8, Line 246: 				ranges = <0x3000000 0x87e0 0x0 0x3000000 0x87e0 0x0 0x10 0x0>;
line over 80 characters


https://review.coreboot.org/#/c/28024/8/src/mainboard/opencellular/elgon/gbcv2.dts@251
PS8, Line 251: 					compatible = "cavium,thunder-8890-mdio-nexus";
DT compatible string "cavium,thunder-8890-mdio-nexus" appears un-documented -- check util/lint/checkpatch.pl/Documentation/devicetree/bindings/


https://review.coreboot.org/#/c/28024/8/src/mainboard/opencellular/elgon/gbcv2.dts@251
PS8, Line 251: 					compatible = "cavium,thunder-8890-mdio-nexus";
DT compatible string vendor "cavium" appears un-documented -- check util/lint/checkpatch.pl/Documentation/devicetree/bindings/vendor-prefixes.txt


https://review.coreboot.org/#/c/28024/8/src/mainboard/opencellular/elgon/gbcv2.dts@251
PS8, Line 251: 					compatible = "cavium,thunder-8890-mdio-nexus";
line over 80 characters


https://review.coreboot.org/#/c/28024/8/src/mainboard/opencellular/elgon/gbcv2.dts@255
PS8, Line 255: 					assigned-addresses = <0x3000000 0x87e0 0x5000000 0x0 0x800000>;
line over 80 characters


https://review.coreboot.org/#/c/28024/8/src/mainboard/opencellular/elgon/gbcv2.dts@256
PS8, Line 256: 					ranges = <0x87e0 0x5000000 0x3000000 0x87e0 0x5000000 0x0 0x800000>;
line over 80 characters


https://review.coreboot.org/#/c/28024/8/src/mainboard/opencellular/elgon/gbcv2.dts@259
PS8, Line 259: 						compatible = "cavium,thunder-8890-mdio";
DT compatible string "cavium,thunder-8890-mdio" appears un-documented -- check util/lint/checkpatch.pl/Documentation/devicetree/bindings/


https://review.coreboot.org/#/c/28024/8/src/mainboard/opencellular/elgon/gbcv2.dts@259
PS8, Line 259: 						compatible = "cavium,thunder-8890-mdio";
DT compatible string vendor "cavium" appears un-documented -- check util/lint/checkpatch.pl/Documentation/devicetree/bindings/vendor-prefixes.txt


https://review.coreboot.org/#/c/28024/8/src/mainboard/opencellular/elgon/gbcv2.dts@259
PS8, Line 259: 						compatible = "cavium,thunder-8890-mdio";
line over 80 characters


https://review.coreboot.org/#/c/28024/8/src/mainboard/opencellular/elgon/gbcv2.dts@262
PS8, Line 262: 						reg = <0x87e0 0x5003800 0x0 0x30>;
line over 80 characters


https://review.coreboot.org/#/c/28024/8/src/mainboard/opencellular/elgon/gbcv2.dts@266
PS8, Line 266: 						compatible = "cavium,thunder-8890-mdio";
DT compatible string "cavium,thunder-8890-mdio" appears un-documented -- check util/lint/checkpatch.pl/Documentation/devicetree/bindings/


https://review.coreboot.org/#/c/28024/8/src/mainboard/opencellular/elgon/gbcv2.dts@266
PS8, Line 266: 						compatible = "cavium,thunder-8890-mdio";
DT compatible string vendor "cavium" appears un-documented -- check util/lint/checkpatch.pl/Documentation/devicetree/bindings/vendor-prefixes.txt


https://review.coreboot.org/#/c/28024/8/src/mainboard/opencellular/elgon/gbcv2.dts@266
PS8, Line 266: 						compatible = "cavium,thunder-8890-mdio";
line over 80 characters


https://review.coreboot.org/#/c/28024/8/src/mainboard/opencellular/elgon/gbcv2.dts@269
PS8, Line 269: 						reg = <0x87e0 0x5003880 0x0 0x30>;
line over 80 characters


https://review.coreboot.org/#/c/28024/8/src/mainboard/opencellular/elgon/gbcv2.dts@274
PS8, Line 274: 					compatible = "cavium,thunder-8890-mmc";
DT compatible string "cavium,thunder-8890-mmc" appears un-documented -- check util/lint/checkpatch.pl/Documentation/devicetree/bindings/


https://review.coreboot.org/#/c/28024/8/src/mainboard/opencellular/elgon/gbcv2.dts@274
PS8, Line 274: 					compatible = "cavium,thunder-8890-mmc";
DT compatible string vendor "cavium" appears un-documented -- check util/lint/checkpatch.pl/Documentation/devicetree/bindings/vendor-prefixes.txt


https://review.coreboot.org/#/c/28024/8/src/mainboard/opencellular/elgon/gbcv2.dts@281
PS8, Line 281: 						compatible = "mmc-slot";
DT compatible string "mmc-slot" appears un-documented -- check util/lint/checkpatch.pl/Documentation/devicetree/bindings/


https://review.coreboot.org/#/c/28024/8/src/mainboard/opencellular/elgon/gbcv2.dts@301
PS8, Line 301: 					compatible = "cavium,thunder-8890-twsi";
DT compatible string "cavium,thunder-8890-twsi" appears un-documented -- check util/lint/checkpatch.pl/Documentation/devicetree/bindings/


https://review.coreboot.org/#/c/28024/8/src/mainboard/opencellular/elgon/gbcv2.dts@301
PS8, Line 301: 					compatible = "cavium,thunder-8890-twsi";
DT compatible string vendor "cavium" appears un-documented -- check util/lint/checkpatch.pl/Documentation/devicetree/bindings/vendor-prefixes.txt


https://review.coreboot.org/#/c/28024/8/src/mainboard/opencellular/elgon/gbcv2.dts@307
PS8, Line 307: 						compatible = "idt,82p33914-1";
DT compatible string "idt,82p33914-1" appears un-documented -- check util/lint/checkpatch.pl/Documentation/devicetree/bindings/


https://review.coreboot.org/#/c/28024/8/src/mainboard/opencellular/elgon/gbcv2.dts@307
PS8, Line 307: 						compatible = "idt,82p33914-1";
DT compatible string vendor "idt" appears un-documented -- check util/lint/checkpatch.pl/Documentation/devicetree/bindings/vendor-prefixes.txt


https://review.coreboot.org/#/c/28024/8/src/mainboard/opencellular/elgon/gbcv2.dts@315
PS8, Line 315: 					compatible = "cavium,thunder-8890-twsi";
DT compatible string "cavium,thunder-8890-twsi" appears un-documented -- check util/lint/checkpatch.pl/Documentation/devicetree/bindings/


https://review.coreboot.org/#/c/28024/8/src/mainboard/opencellular/elgon/gbcv2.dts@315
PS8, Line 315: 					compatible = "cavium,thunder-8890-twsi";
DT compatible string vendor "cavium" appears un-documented -- check util/lint/checkpatch.pl/Documentation/devicetree/bindings/vendor-prefixes.txt


https://review.coreboot.org/#/c/28024/8/src/mainboard/opencellular/elgon/gbcv2.dts@321
PS8, Line 321: 						compatible = "atmel,24c256";
DT compatible string "atmel,24c256" appears un-documented -- check util/lint/checkpatch.pl/Documentation/devicetree/bindings/


https://review.coreboot.org/#/c/28024/8/src/mainboard/opencellular/elgon/gbcv2.dts@321
PS8, Line 321: 						compatible = "atmel,24c256";
DT compatible string vendor "atmel" appears un-documented -- check util/lint/checkpatch.pl/Documentation/devicetree/bindings/vendor-prefixes.txt


https://review.coreboot.org/#/c/28024/8/src/mainboard/opencellular/elgon/gbcv2.dts@327
PS8, Line 327: 						compatible = "isil,isl12057";
DT compatible string "isil,isl12057" appears un-documented -- check util/lint/checkpatch.pl/Documentation/devicetree/bindings/


https://review.coreboot.org/#/c/28024/8/src/mainboard/opencellular/elgon/gbcv2.dts@327
PS8, Line 327: 						compatible = "isil,isl12057";
DT compatible string vendor "isil" appears un-documented -- check util/lint/checkpatch.pl/Documentation/devicetree/bindings/vendor-prefixes.txt


https://review.coreboot.org/#/c/28024/8/src/mainboard/opencellular/elgon/gbcv2.dts@332
PS8, Line 332: 						compatible = "infineon,slb9645tt";
DT compatible string "infineon,slb9645tt" appears un-documented -- check util/lint/checkpatch.pl/Documentation/devicetree/bindings/


https://review.coreboot.org/#/c/28024/8/src/mainboard/opencellular/elgon/gbcv2.dts@332
PS8, Line 332: 						compatible = "infineon,slb9645tt";
DT compatible string vendor "infineon" appears un-documented -- check util/lint/checkpatch.pl/Documentation/devicetree/bindings/vendor-prefixes.txt


https://review.coreboot.org/#/c/28024/8/src/mainboard/opencellular/elgon/gbcv2.dts@332
PS8, Line 332: 						compatible = "infineon,slb9645tt";
line over 80 characters


https://review.coreboot.org/#/c/28024/8/src/mainboard/opencellular/elgon/gbcv2.dts@340
PS8, Line 340: 					compatible = "cavium,thunder-8890-bgx";
DT compatible string "cavium,thunder-8890-bgx" appears un-documented -- check util/lint/checkpatch.pl/Documentation/devicetree/bindings/


https://review.coreboot.org/#/c/28024/8/src/mainboard/opencellular/elgon/gbcv2.dts@340
PS8, Line 340: 					compatible = "cavium,thunder-8890-bgx";
DT compatible string vendor "cavium" appears un-documented -- check util/lint/checkpatch.pl/Documentation/devicetree/bindings/vendor-prefixes.txt


https://review.coreboot.org/#/c/28024/8/src/mainboard/opencellular/elgon/gbcv2.dts@345
PS8, Line 345: 						local-mac-address = [00 00 00 00 00 00];
line over 80 characters


https://review.coreboot.org/#/c/28024/8/src/mainboard/opencellular/elgon/gbcv2.dts@352
PS8, Line 352: 					compatible = "cavium,thunder-8890-bgx";
DT compatible string "cavium,thunder-8890-bgx" appears un-documented -- check util/lint/checkpatch.pl/Documentation/devicetree/bindings/


https://review.coreboot.org/#/c/28024/8/src/mainboard/opencellular/elgon/gbcv2.dts@352
PS8, Line 352: 					compatible = "cavium,thunder-8890-bgx";
DT compatible string vendor "cavium" appears un-documented -- check util/lint/checkpatch.pl/Documentation/devicetree/bindings/vendor-prefixes.txt


https://review.coreboot.org/#/c/28024/8/src/mainboard/opencellular/elgon/gbcv2.dts@358
PS8, Line 358: 						local-mac-address = [00 00 00 00 00 00];
line over 80 characters


https://review.coreboot.org/#/c/28024/8/src/mainboard/opencellular/elgon/gbcv2.dts@366
PS8, Line 366: 					compatible = "cavium,thunder-8890-bgx";
DT compatible string "cavium,thunder-8890-bgx" appears un-documented -- check util/lint/checkpatch.pl/Documentation/devicetree/bindings/


https://review.coreboot.org/#/c/28024/8/src/mainboard/opencellular/elgon/gbcv2.dts@366
PS8, Line 366: 					compatible = "cavium,thunder-8890-bgx";
DT compatible string vendor "cavium" appears un-documented -- check util/lint/checkpatch.pl/Documentation/devicetree/bindings/vendor-prefixes.txt


https://review.coreboot.org/#/c/28024/8/src/mainboard/opencellular/elgon/gbcv2.dts@372
PS8, Line 372: 				compatible = "cavium,thunder-8190-spi";
DT compatible string "cavium,thunder-8190-spi" appears un-documented -- check util/lint/checkpatch.pl/Documentation/devicetree/bindings/


https://review.coreboot.org/#/c/28024/8/src/mainboard/opencellular/elgon/gbcv2.dts@372
PS8, Line 372: 				compatible = "cavium,thunder-8190-spi";
DT compatible string vendor "cavium" appears un-documented -- check util/lint/checkpatch.pl/Documentation/devicetree/bindings/vendor-prefixes.txt


https://review.coreboot.org/#/c/28024/8/src/mainboard/opencellular/elgon/gbcv2.dts@379
PS8, Line 379: 					compatible = "winbond,w25q128", "jedec,spi-nor";
DT compatible string "winbond,w25q128" appears un-documented -- check util/lint/checkpatch.pl/Documentation/devicetree/bindings/


https://review.coreboot.org/#/c/28024/8/src/mainboard/opencellular/elgon/gbcv2.dts@379
PS8, Line 379: 					compatible = "winbond,w25q128", "jedec,spi-nor";
DT compatible string vendor "winbond" appears un-documented -- check util/lint/checkpatch.pl/Documentation/devicetree/bindings/vendor-prefixes.txt


https://review.coreboot.org/#/c/28024/8/src/mainboard/opencellular/elgon/gbcv2.dts@379
PS8, Line 379: 					compatible = "winbond,w25q128", "jedec,spi-nor";
DT compatible string "jedec,spi-nor" appears un-documented -- check util/lint/checkpatch.pl/Documentation/devicetree/bindings/


https://review.coreboot.org/#/c/28024/8/src/mainboard/opencellular/elgon/gbcv2.dts@379
PS8, Line 379: 					compatible = "winbond,w25q128", "jedec,spi-nor";
DT compatible string vendor "jedec" appears un-documented -- check util/lint/checkpatch.pl/Documentation/devicetree/bindings/vendor-prefixes.txt


https://review.coreboot.org/#/c/28024/8/src/mainboard/opencellular/elgon/gbcv2.dts@379
PS8, Line 379: 					compatible = "winbond,w25q128", "jedec,spi-nor";
line over 80 characters


https://review.coreboot.org/#/c/28024/8/src/mainboard/opencellular/elgon/gbcv2.dts@389
PS8, Line 389: 				compatible = "cavium,thunder-8890-gpio";
DT compatible string "cavium,thunder-8890-gpio" appears un-documented -- check util/lint/checkpatch.pl/Documentation/devicetree/bindings/


https://review.coreboot.org/#/c/28024/8/src/mainboard/opencellular/elgon/gbcv2.dts@389
PS8, Line 389: 				compatible = "cavium,thunder-8890-gpio";
DT compatible string vendor "cavium" appears un-documented -- check util/lint/checkpatch.pl/Documentation/devicetree/bindings/vendor-prefixes.txt


https://review.coreboot.org/#/c/28024/8/src/mainboard/opencellular/elgon/gbcv2.dts@399
PS8, Line 399: 				compatible = "cavium,cn8130-nand";
DT compatible string "cavium,cn8130-nand" appears un-documented -- check util/lint/checkpatch.pl/Documentation/devicetree/bindings/


https://review.coreboot.org/#/c/28024/8/src/mainboard/opencellular/elgon/gbcv2.dts@399
PS8, Line 399: 				compatible = "cavium,cn8130-nand";
DT compatible string vendor "cavium" appears un-documented -- check util/lint/checkpatch.pl/Documentation/devicetree/bindings/vendor-prefixes.txt


https://review.coreboot.org/#/c/28024/8/src/mainboard/opencellular/elgon/gbcv2.dts@406
PS8, Line 406: 			compatible = "cavium,pci-host-thunder-pem";
DT compatible string "cavium,pci-host-thunder-pem" appears un-documented -- check util/lint/checkpatch.pl/Documentation/devicetree/bindings/


https://review.coreboot.org/#/c/28024/8/src/mainboard/opencellular/elgon/gbcv2.dts@406
PS8, Line 406: 			compatible = "cavium,pci-host-thunder-pem";
DT compatible string vendor "cavium" appears un-documented -- check util/lint/checkpatch.pl/Documentation/devicetree/bindings/vendor-prefixes.txt


https://review.coreboot.org/#/c/28024/8/src/mainboard/opencellular/elgon/gbcv2.dts@415
PS8, Line 415: 			reg = <0x8800 0x1f000000 0x0 0x39000000 0x87e0 0xc0000000 0x0 0x1000000>;
line over 80 characters


https://review.coreboot.org/#/c/28024/8/src/mainboard/opencellular/elgon/gbcv2.dts@416
PS8, Line 416: 			ranges = <0x1000000 0x0 0x0 0x8830 0x0 0x0 0x10000 0x3000000 0x0 0x10000000 0x8810 0x10000000 0xf 0xf0000000 0x43000000 0x10 0x0 0x8820 0x0 0x10 0x0 0x3000000 0x87e0 0xc0000000 0x87e0 0xc0000000 0x0 0x1000000>;
line over 80 characters


https://review.coreboot.org/#/c/28024/8/src/mainboard/opencellular/elgon/gbcv2.dts@419
PS8, Line 419: 			interrupt-map = <0x0 0x0 0x0 0x1 0x1 0x0 0x0 0x0 0x10 0x4 0x0 0x0 0x0 0x2 0x1 0x0 0x0 0x0 0x11 0x4 0x0 0x0 0x0 0x3 0x1 0x0 0x0 0x0 0x12 0x4 0x0 0x0 0x0 0x4 0x1 0x0 0x0 0x0 0x13 0x4>;
line over 80 characters


https://review.coreboot.org/#/c/28024/8/src/mainboard/opencellular/elgon/gbcv2.dts@425
PS8, Line 425: 			compatible = "cavium,pci-host-thunder-pem";
DT compatible string "cavium,pci-host-thunder-pem" appears un-documented -- check util/lint/checkpatch.pl/Documentation/devicetree/bindings/


https://review.coreboot.org/#/c/28024/8/src/mainboard/opencellular/elgon/gbcv2.dts@425
PS8, Line 425: 			compatible = "cavium,pci-host-thunder-pem";
DT compatible string vendor "cavium" appears un-documented -- check util/lint/checkpatch.pl/Documentation/devicetree/bindings/vendor-prefixes.txt


https://review.coreboot.org/#/c/28024/8/src/mainboard/opencellular/elgon/gbcv2.dts@434
PS8, Line 434: 			reg = <0x8840 0x57000000 0x0 0x39000000 0x87e0 0xc1000000 0x0 0x1000000>;
line over 80 characters


https://review.coreboot.org/#/c/28024/8/src/mainboard/opencellular/elgon/gbcv2.dts@435
PS8, Line 435: 			ranges = <0x1000000 0x0 0x10000 0x8870 0x10000 0x0 0x10000 0x3000000 0x0 0x10000000 0x8850 0x10000000 0xf 0xf0000000 0x43000000 0x10 0x0 0x8860 0x0 0x10 0x0 0x3000000 0x87e0 0xc1000000 0x87e0 0xc1000000 0x0 0x1000000>;
line over 80 characters


https://review.coreboot.org/#/c/28024/8/src/mainboard/opencellular/elgon/gbcv2.dts@438
PS8, Line 438: 			interrupt-map = <0x0 0x0 0x0 0x1 0x1 0x0 0x0 0x0 0x14 0x4 0x0 0x0 0x0 0x2 0x1 0x0 0x0 0x0 0x15 0x4 0x0 0x0 0x0 0x3 0x1 0x0 0x0 0x0 0x16 0x4 0x0 0x0 0x0 0x4 0x1 0x0 0x0 0x0 0x17 0x4>;
line over 80 characters


https://review.coreboot.org/#/c/28024/8/src/mainboard/opencellular/elgon/gbcv2.dts@444
PS8, Line 444: 			compatible = "cavium,pci-host-thunder-pem";
DT compatible string "cavium,pci-host-thunder-pem" appears un-documented -- check util/lint/checkpatch.pl/Documentation/devicetree/bindings/


https://review.coreboot.org/#/c/28024/8/src/mainboard/opencellular/elgon/gbcv2.dts@444
PS8, Line 444: 			compatible = "cavium,pci-host-thunder-pem";
DT compatible string vendor "cavium" appears un-documented -- check util/lint/checkpatch.pl/Documentation/devicetree/bindings/vendor-prefixes.txt


https://review.coreboot.org/#/c/28024/8/src/mainboard/opencellular/elgon/gbcv2.dts@453
PS8, Line 453: 			reg = <0x8880 0x8f000000 0x0 0x39000000 0x87e0 0xc2000000 0x0 0x1000000>;
line over 80 characters


https://review.coreboot.org/#/c/28024/8/src/mainboard/opencellular/elgon/gbcv2.dts@454
PS8, Line 454: 			ranges = <0x1000000 0x0 0x20000 0x88b0 0x20000 0x0 0x10000 0x3000000 0x0 0x10000000 0x8890 0x10000000 0xf 0xf0000000 0x43000000 0x10 0x0 0x88a0 0x0 0x10 0x0 0x3000000 0x87e0 0xc2000000 0x87e0 0xc2000000 0x0 0x1000000>;
line over 80 characters


https://review.coreboot.org/#/c/28024/8/src/mainboard/opencellular/elgon/gbcv2.dts@457
PS8, Line 457: 			interrupt-map = <0x0 0x0 0x0 0x1 0x1 0x0 0x0 0x0 0x18 0x4 0x0 0x0 0x0 0x2 0x1 0x0 0x0 0x0 0x19 0x4 0x0 0x0 0x0 0x3 0x1 0x0 0x0 0x0 0x1a 0x4 0x0 0x0 0x0 0x4 0x1 0x0 0x0 0x0 0x1b 0x4>;
line over 80 characters


https://review.coreboot.org/#/c/28024/8/src/mainboard/opencellular/elgon/gbcv2.dts@465
PS8, Line 465: 			compatible = "cavium,thunder-8190-tdm";
DT compatible string "cavium,thunder-8190-tdm" appears un-documented -- check util/lint/checkpatch.pl/Documentation/devicetree/bindings/


https://review.coreboot.org/#/c/28024/8/src/mainboard/opencellular/elgon/gbcv2.dts@465
PS8, Line 465: 			compatible = "cavium,thunder-8190-tdm";
DT compatible string vendor "cavium" appears un-documented -- check util/lint/checkpatch.pl/Documentation/devicetree/bindings/vendor-prefixes.txt



-- 
To view, visit https://review.coreboot.org/28024
To unsubscribe, or for help writing mail filters, visit https://review.coreboot.org/settings

Gerrit-Project: coreboot
Gerrit-Branch: master
Gerrit-MessageType: comment
Gerrit-Change-Id: I9b809306cc48facbade5dc63846c4532b397e0b5
Gerrit-Change-Number: 28024
Gerrit-PatchSet: 8
Gerrit-Owner: Philipp Deppenwiese <zaolin.daisuki at gmail.com>
Gerrit-Reviewer: Paul Menzel <paulepanter at users.sourceforge.net>
Gerrit-Reviewer: build bot (Jenkins) <no-reply at coreboot.org>
Gerrit-CC: Patrick Rudolph <siro at das-labor.org>
Gerrit-Comment-Date: Fri, 24 Aug 2018 12:37:48 +0000
Gerrit-HasComments: Yes
Gerrit-HasLabels: No
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://mail.coreboot.org/pipermail/coreboot-gerrit/attachments/20180824/7f157401/attachment.html>


More information about the coreboot-gerrit mailing list