On 06/12/2018 03:54 PM, Rob Herring wrote:Correct - all board designs that include this dtsi file follow such commonality (ie. design with SATA0 first, etc).Â By having common board designs it allows for commonality in dts files rather than duplicating information everywhere.Â If somebody designs a bizarro board they are free to create their own dts file of course.
On Thu, Jun 7, 2018 at 12:53 PM, Scott Branden
Hi Rob,This only works if ports are contiguously enabled (0-N). You might not
Could you please kindly comment on change below.
It allows board variants to be added easily via a simple define for
different number of SATA ports.
On 18-06-04 09:22 AM, Florian Fainelli wrote:
On 05/18/2018 11:34 AM, Scott Branden wrote:
Move remaining sata configuration to stingray-sata.dtsi and enableRob could you review this and let us know if this approach is okay or
ports based on NUM_SATA defined.
Now, all that needs to be done is define NUM_SATA per board.
not? Thank you!
Signed-off-by: Scott Branden <scott.branden@xxxxxxxxxxxx>
diff --git a/arch/arm64/boot/dts/broadcom/stingray/stingray-sata.dtsi
index 8c68e0c..7f6d176 100644
@@ -43,7 +43,11 @@
interrupts = <GIC_SPI 321 IRQ_TYPE_LEVEL_HIGH>;
#address-cells = <1>;
#size-cells = <0>;
+#if (NUM_SATA > 0)
+ status = "okay";
status = "disabled";
care, but it is not a pattern that works in general.
The use of a define to specify the number of SATA ports in the board design meets our requirements of being able to maintain many boards.Â We need a method to specify the number of ports in the board design rather than copying and pasting the information in many dts files.Â If you have an alternative upstreamable mechanism to manage the configuration of many boards without copy and paste that would be ideal?And I'm not a fan
of C preprocessing in DT files in general beyond just defines for
Should we interpret this as a formal NAK?