On 07/08/23 7:55 pm, Andrew Lunn wrote: >> @@ -210,6 +210,9 @@ void icssg_config_ipg(struct prueth_emac *emac) >> case SPEED_100: >> icssg_mii_update_ipg(prueth->mii_rt, slice, MII_RT_TX_IPG_100M); >> break; >> + case SPEED_10: >> + icssg_mii_update_ipg(prueth->mii_rt, slice, MII_RT_TX_IPG_100M); >> + break; > > Since that looks like a typO, you might want to add a comment. > > Adnrew Sure, Andrew. I'll add the below comment in 'case SPEED_10' so that it doesn't seem like a typo. case SPEED_10: /* IPG for 10M is same as 100M */ icssg_mii_update_ipg(prueth->mii_rt, slice, MII_RT_TX_IPG_100M); break; -- Thanks and Regards, Danish.