On Thu Feb 13, 2020 at 11:59 AM, Jaedon Shin wrote: > @@ -173,8 +174,79 @@ struct brcm_pcie { > int gen; > u64 msi_target_addr; > struct brcm_msi *msi; > +#ifdef CONFIG_REGULATOR Correct me if I'm wrong, but I don't think these #ifdefs are necessary (same below). The regulator code defines empty functions and relevant structures even when not enabled. Regards, Nicolas