Hi, On Fri, Aug 10, 2018 at 9:29 AM, Mark Brown <broonie@xxxxxxxxxx> wrote: > On Fri, Jul 13, 2018 at 06:50:59PM -0700, David Collins wrote: > >> + switch (rpmh_mode) { > >> + default: >> + mode = REGULATOR_MODE_INVALID; >> + } > > I'm not sure why the break statements are being omitted in default > cases, but I do find myself stopping and trying to figure it out? Hopefully <https://lore.kernel.org/lkml/20180810200531.88409-1-dianders@xxxxxxxxxxxx/> addresses this? Thanks! -Doug