On Fri, Jul 21, 2023 at 07:47:41PM +0200, Christophe JAILLET wrote: > Since commit 235602146ec9 ("i2c-nomadik: turn the platform driver to an amba > driver"), there is no more request_mem_region() call in this driver. > > So remove the release_mem_region() call from the remove function which is > likely a left over. > > Fixes: 235602146ec9 ("i2c-nomadik: turn the platform driver to an amba driver") > Cc: <stable@xxxxxxxxxxxxxxx> # v3.6+ > Acked-by: Linus Walleij <linus.walleij@xxxxxxxxxx> > Reviewed-by: Andi Shyti <andi.shyti@xxxxxxxxxx> > Signed-off-by: Christophe JAILLET <christophe.jaillet@xxxxxxxxxx> > --- > The patch below that should fix a merge conflict related to commit > 9c7174db4cdd1 ("i2c: nomadik: Use devm_clk_get_enabled()") has been > HAND MODIFIED. I don't understand, that commit is not in the stable trees. What do you mean by "hand modified"? > I hope it is fine, but is provided as-is. Especially line numbers should be > wrong, but 'patch' should be able to deal with it. (sorry if it does not apply) > > I guess that it should also apply to all previous branches. > > I've left the commit description as it was. Not sure what to do with A-b and R-b > tags. Why isn't this needed in Linus's tree? confused, greg k-h