[PATCH] i2c: rtl9300: add support for 50 kHz and 2.5 MHz bus speeds

[PATCH] i2c: rtl9300: add support for 50 kHz and 2.5 MHz bus speeds

[Jan Kantert]

Some SFP modules on certain switches (for example the ONTi ONT-S508CL-8S and
XikeStor SKS8300-8X) exhibit unreliable I2C communication at the currently
supported speeds. Add support for 50 kHz and 2.5 MHz I2C bus modes on the
RTL9300 to improve compatibility with these devices.

Signed-off-by: Jan Kantert <jan-kernel@kantert.net>
---
 drivers/i2c/busses/i2c-rtl9300.c | 16 ++++++++++++++--
 1 file changed, 14 insertions(+), 2 deletions(-)

diff --git a/drivers/i2c/busses/i2c-rtl9300.c b/drivers/i2c/busses/i2c-rtl9300.c
index 672cb978066d..67a5c4228fc9 100644
--- a/drivers/i2c/busses/i2c-rtl9300.c
+++ b/drivers/i2c/busses/i2c-rtl9300.c
@@ -11,10 +11,16 @@
 #include <linux/unaligned.h>
 
 enum rtl9300_bus_freq {
-	RTL9300_I2C_STD_FREQ,
-	RTL9300_I2C_FAST_FREQ,
+	RTL9300_I2C_STD_FREQ,			// 100kHz
+	RTL9300_I2C_FAST_FREQ,			// 400kHz
+	RTL9300_I2C_SUPER_FAST_FREQ,		// 2.5MHz
+	RTL9300_I2C_SLOW_FREQ,			// 50kHz
 };
 
+#define RTL9300_I2C_MAX_SUPER_FAST_FREQ		2500000
+#define RTL9300_I2C_MAX_SLOW_FREQ		50000
+
+
 struct rtl9300_i2c;
 
 struct rtl9300_i2c_chan {
@@ -433,6 +439,12 @@ static int rtl9300_i2c_probe(struct platform_device *pdev)
 		case I2C_MAX_FAST_MODE_FREQ:
 			chan->bus_freq = RTL9300_I2C_FAST_FREQ;
 			break;
+		case RTL9300_I2C_MAX_SUPER_FAST_FREQ:
+			chan->bus_freq = RTL9300_I2C_SUPER_FAST_FREQ;
+			break;
+		case RTL9300_I2C_MAX_SLOW_FREQ:
+			chan->bus_freq = RTL9300_I2C_SLOW_FREQ;
+			break;
 		default:
 			dev_warn(i2c->dev, "SDA%d clock-frequency %d not supported using default\n",
 				 sda_num, clock_freq);
-- 
2.34.1

Re: [PATCH] i2c: rtl9300: add support for 50 kHz and 2.5 MHz bus speeds

[Chris Packham]

Hi Jan,

On 28/02/2026 00:11, Jan Kantert wrote:
> Some SFP modules on certain switches (for example the ONTi ONT-S508CL-8S and
> XikeStor SKS8300-8X) exhibit unreliable I2C communication at the currently
> supported speeds. Add support for 50 kHz and 2.5 MHz I2C bus modes on the
> RTL9300 to improve compatibility with these devices.
>
> Signed-off-by: Jan Kantert <jan-kernel@kantert.net>

Reviewed-by: Chris Packham <chris.packham@alliedtelesis.co.nz>

Thanks,

> ---
>   drivers/i2c/busses/i2c-rtl9300.c | 16 ++++++++++++++--
>   1 file changed, 14 insertions(+), 2 deletions(-)
>
> diff --git a/drivers/i2c/busses/i2c-rtl9300.c b/drivers/i2c/busses/i2c-rtl9300.c
> index 672cb978066d..67a5c4228fc9 100644
> --- a/drivers/i2c/busses/i2c-rtl9300.c
> +++ b/drivers/i2c/busses/i2c-rtl9300.c
> @@ -11,10 +11,16 @@
>   #include <linux/unaligned.h>
>   
>   enum rtl9300_bus_freq {
> -	RTL9300_I2C_STD_FREQ,
> -	RTL9300_I2C_FAST_FREQ,
> +	RTL9300_I2C_STD_FREQ,			// 100kHz
> +	RTL9300_I2C_FAST_FREQ,			// 400kHz
> +	RTL9300_I2C_SUPER_FAST_FREQ,		// 2.5MHz
> +	RTL9300_I2C_SLOW_FREQ,			// 50kHz
>   };
>   
> +#define RTL9300_I2C_MAX_SUPER_FAST_FREQ		2500000
> +#define RTL9300_I2C_MAX_SLOW_FREQ		50000
> +
> +
>   struct rtl9300_i2c;
>   
>   struct rtl9300_i2c_chan {
> @@ -433,6 +439,12 @@ static int rtl9300_i2c_probe(struct platform_device *pdev)
>   		case I2C_MAX_FAST_MODE_FREQ:
>   			chan->bus_freq = RTL9300_I2C_FAST_FREQ;
>   			break;
> +		case RTL9300_I2C_MAX_SUPER_FAST_FREQ:
> +			chan->bus_freq = RTL9300_I2C_SUPER_FAST_FREQ;
> +			break;
> +		case RTL9300_I2C_MAX_SLOW_FREQ:
> +			chan->bus_freq = RTL9300_I2C_SLOW_FREQ;
> +			break;
>   		default:
>   			dev_warn(i2c->dev, "SDA%d clock-frequency %d not supported using default\n",
>   				 sda_num, clock_freq);

Re: [PATCH] i2c: rtl9300: add support for 50 kHz and 2.5 MHz bus speeds

[Andi Shyti]

Hi Jan,

On Fri, Feb 27, 2026 at 12:11:34PM +0100, Jan Kantert wrote:
> Some SFP modules on certain switches (for example the ONTi ONT-S508CL-8S and
> XikeStor SKS8300-8X) exhibit unreliable I2C communication at the currently
> supported speeds. Add support for 50 kHz and 2.5 MHz I2C bus modes on the
> RTL9300 to improve compatibility with these devices.
> 
> Signed-off-by: Jan Kantert <jan-kernel@kantert.net>

merged to i2c/i2c-host.

Thanks,
Andi