Hi, On Fri, May 20, 2022 at 2:27 PM Kuogee Hsieh <quic_khsieh@xxxxxxxxxxx> wrote: > > @@ -101,11 +101,6 @@ struct dp_reg_entry { > int disable_load; > }; > > -struct dp_regulator_cfg { > - int num; > - struct dp_reg_entry regs[DP_DEV_REGULATOR_MAX]; You forgot to delete the #define of DP_DEV_REGULATOR_MAX earlier in this file. Other than that, this looks good to me. Reviewed-by: Douglas Anderson <dianders@xxxxxxxxxxxx>