On 27/05/2019 19:57, Martin Blumenstingl wrote: > Hi Neil, > > On Mon, May 27, 2019 at 2:28 PM Neil Armstrong <narmstr...@baylibre.com> > wrote: > [...] >> This looks a little over-engineered, but it's correct : > my main motivation was to "not copy the 20 line switch/case statement > from meson_pwm_enable() to meson_pwm_get_state()" > I extended the idea that already existed for the "mux_reg_shifts" > array and made it work for "more than one value" > > please speak up if you have another idea in mind, maybe that makes the > result even better
Sorry if I was misunderstood, your solution is correct, and I don't have any simpler ideas in mind ! Neil > > > Martin >