On Mon, Oct 1, 2018 at 11:49 PM Stephen Boyd <[email protected]> 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 <[email protected]> > Cc: Bjorn Andersson <[email protected]> > Cc: Niklas Cassel <[email protected]> > Reported-by: Niklas Cassel <[email protected]> > Fixes: 1de7ddb3a15c ("pinctrl: msm: Mux out gpio function with > gpio_request()") > Signed-off-by: Stephen Boyd <[email protected]> Patch applied with all the tags. Yours, Linus Walleij
