RE: [net-next PATCH v5] net: macb: Fix several edge cases in validate

Hi Sean,

Thanks for this improvement.

+	if (!macb_is_gem(bp) ||
+	    (bp->caps & MACB_CAPS_GIGABIT_MODE_AVAILABLE)) {
+		have_1g = true;
+		if (bp->caps & MACB_CAPS_PCS)
+			have_sgmii = true;
+		if (bp->caps & MACB_CAPS_HIGH_SPEED)
+			have_10g = true;

As I understand, MACB_CAPS_GIGABIT_MODE_AVAILABLE is used as a quirk in configs
to prevent giga bit operation support, Nicolas should have more information about this.

macb_is_gem() tells whether giga bit operations is supported by HW, MACB_CAPS_PCS indicate
whether PCS is included in the design (needed for SGMII and 10G operation), MACB_CAPS_HIGH_SPEED
indicate if design supports 10G operation.

I believe this should be

+	if (macb_is_gem(bp) &&
+	    (bp->caps & MACB_CAPS_GIGABIT_MODE_AVAILABLE)) {
+		have_1g = true;
+		if (bp->caps & MACB_CAPS_PCS)
+			have_sgmii = true;
+		if (bp->caps & MACB_CAPS_HIGH_SPEED)
+			have_10g = true;

Regards,
Parshuram Thombare