On 23.03.20 10:33, Baruch Siach wrote:
Hi Stefan,
On Mon, Mar 23 2020, Stefan Roese wrote:
On 27.01.20 21:01, Joel Johnson wrote:
While newer Linux kernels provide autoconfiguration of SFP, provide
an option for setting in U-Boot Kconfig for use prior to booting.
Signed-off-by: Joel Johnson <mrj...@lixil.net>
---
v2 changes:
- fixed help indentation
v3 changes:
- none
v4 changes:
- adjust static SerDes configuration at runtime instead of #ifdef
v5 changes:
- make independent of runtime detection
---
board/solidrun/clearfog/Kconfig | 7 +++++++
board/solidrun/clearfog/clearfog.c | 5 +++++
2 files changed, 12 insertions(+)
diff --git a/board/solidrun/clearfog/Kconfig b/board/solidrun/clearfog/Kconfig
index 936d5918f8..c910e17093 100644
--- a/board/solidrun/clearfog/Kconfig
+++ b/board/solidrun/clearfog/Kconfig
@@ -15,4 +15,11 @@ config TARGET_CLEARFOG_BASE
detection via additional EEPROM hardware. This option enables
selecting
the Base variant for older hardware revisions.
+config CLEARFOG_SFP_25GB
+ bool "Enable 2.5 Gbps mode for SFP"
+ help
+ Set the SFP module connection to support 2.5 Gbps transfer speed for
the
+ SGMII connection (requires a supporting SFP). By default, transfer
speed
+ of 1.25 Gbps is used, suitable for a more common 1 Gbps SFP module.
+
endmenu
diff --git a/board/solidrun/clearfog/clearfog.c
b/board/solidrun/clearfog/clearfog.c
index c873d00905..064ce4e520 100644
--- a/board/solidrun/clearfog/clearfog.c
+++ b/board/solidrun/clearfog/clearfog.c
@@ -63,6 +63,11 @@ int hws_board_topology_load(struct serdes_map
**serdes_map_array, u8 *count)
{
cf_read_tlv_data();
+ /* Apply build configuration options before runtime configuration */
+ if (IS_ENABLED(CONFIG_CLEARFOG_SFP_25GB))
+ board_serdes_map[5].serdes_speed = SERDES_SPEED_3_125_GBPS;
Just checking: Your option is for 2.5 gbps and you configure here to
3.125 gpbs. Is this intended?
This is intended. Serdes encoding is 8b/10b. So for 2.5Gbps you need
serdes rate of 3.125Gbps. For 1Gbps we already set 1.25Gpbs serdes rate
(SERDES_SPEED_1_25_GBPS) in board_serdes_map[].
Makes sense. Thanks.
Reviewed-by: Stefan Roese <s...@denx.de>
Thanks,
Stefan
