On 30-05-24, 11:11, Jia Jie Ho wrote: > + > +struct dw_axi_peripheral_config { > +#define DWAXIDMAC_STARFIVE_SM_ALGO BIT(0) what does this quirk mean? > + u32 quirks; Can you explain why you need this to be exposed. I would prefer we use existing interfaces and not define a new one... -- ~Vinod