Message ID | cover.1531531808.git.collinsd@codeaurora.org |
---|---|
Headers | show |
Series | regulator: add QCOM RPMh regulator driver | expand |
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?
Hi, On Fri, Aug 10, 2018 at 9:29 AM, Mark Brown <broonie@kernel.org> 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@chromium.org/> addresses this? Thanks! -Doug