> +const char *i2c_freq_mode_string(u32 bus_freq_hz)
> +{
> +     switch (bus_freq_hz) {
> +     case I2C_MAX_STANDARD_MODE_FREQ:
> +             return "Standard Mode (100 kHz)";

Sorry, I just noticed just now. Shouldn't we also support lower
frequencies than the maximum one? I.e.

        if (bus_freq_hz <= I2C_MAX_STANDARD_MODE_FREQ)
                return "Standard Mode (max 100 kHz)";
        else if (bus_freq_hz <= ... )

?

Attachment: signature.asc
Description: PGP signature

Reply via email to