On Mon, Oct 01, 2018 at 02:49:05PM -0700, Stephen Boyd wrote: > This code needs to select function #0, which is the first int in the > array of functions, not the number 0 which may or may not be the > function for "GPIO mode" per the enum mapping. We were getting lucky on > SDM845, where this was tested, because the function 0 matched the enum > value for "GPIO mode". On other platforms, e.g. MSM8996, the gpio enum > value is the last one in the list so this code doesn't work and we see a > warning at boot. Fix it by grabbing the first element out of the array > of functions. > > Cc: Doug Anderson <diand...@chromium.org> > Cc: Bjorn Andersson <bjorn.anders...@linaro.org> > Cc: Niklas Cassel <niklas.cas...@linaro.org> > Reported-by: Niklas Cassel <niklas.cas...@linaro.org> > Fixes: 1de7ddb3a15c ("pinctrl: msm: Mux out gpio function with > gpio_request()") > Signed-off-by: Stephen Boyd <swb...@chromium.org>
Tested-by: Niklas Cassel <niklas.cas...@linaro.org>