On Fri, 9 Jun 2017 09:37:55 +0200 Linus Walleij <linus.walleij@xxxxxxxxxx> wrote: > On Thu, Jun 1, 2017 at 2:18 PM, Richard Genoud > <richard.genoud@xxxxxxxxx> wrote: > > > The blink counter A was always selected because 0 was forced in the > > blink select counter register. > > The variable 'set' was obviously there to be used as the register > > value, selecting the B counter when id==1 and A counter when id==0. > > > > Tested on clearfog-pro (Marvell 88F6828) > > > > Fixes: 757642f9a584 ("gpio: mvebu: Add limited PWM support") > > Reviewed-by: Gregory CLEMENT <gregory.clement@xxxxxxxxxxxxxxxxxx> > > Reviewed-by: Ralph Sennhauser <ralph.sennhauser@xxxxxxxxx> > > Signed-off-by: Richard Genoud <richard.genoud@xxxxxxxxx> > > Patch applied for fixes. > > It appears this will clash with patches on the development branch :( Hi Linus, The commit 2233bf7a92e7 ("gpio: mvebu: switch to regmap for register access") which you likely mean breaks gpio-keys, reported about an hour ago, so you might just want to drop that one for now instead as it needs mor work anyway. Ralph -- To unsubscribe from this list: send the line "unsubscribe linux-gpio" in the body of a message to majordomo@xxxxxxxxxxxxxxx More majordomo info at http://vger.kernel.org/majordomo-info.html