Re: [PATCH v8 3/7] i2c: Add Nuvoton NCT6694 I2C support

From: Ming Yu
Date: Tue Mar 25 2025 - 22:46:46 EST


Dear Andi,

Thank you for reviewing,

Andi Shyti <andi.shyti@xxxxxxxxxx> 於 2025年3月20日 週四 上午7:58寫道:
>
...
> > +enum i2c_baudrate {
> > + I2C_BR_25K = 0,
> > + I2C_BR_50K,
> > + I2C_BR_100K,
> > + I2C_BR_200K,
> > + I2C_BR_400K,
> > + I2C_BR_800K,
> > + I2C_BR_1M
> > +};
>
> do we need all these frequencies? I don't see them use anywhere.
>

Originally, I used module parameters to configure these I2C's baud
rate, Do you think this approach iis suitable?

> Besides, can you please use a proper prefix? I2C_BR_* prefix
> doesn't belong to this driver.
>

Okay, I will fix these macros in the next patch.


Best regards,
Ming