On 24-08-20, 18:10, Greg KH wrote: > On Mon, Aug 24, 2020 at 03:00:03PM +0530, Viresh Kumar wrote: > > From: Stephen Boyd <swboyd@xxxxxxxxxxxx> > > > > commit 8979ef70850eb469e1094279259d1ef393ffe85f upstream. > > > > We get the opp_table pointer at the top of the function and so we should > > put the pointer at the end of the function like all other exit paths > > from this function do. > > > > Cc: v5.7+ <stable@xxxxxxxxxxxxxxx> # v5.7+ > > Fixes: aca48b61f963 ("opp: Manage empty OPP tables with clk handle") > > Reviewed-by: Rajendra Nayak <rnayak@xxxxxxxxxxxxxx> > > Signed-off-by: Stephen Boyd <swboyd@xxxxxxxxxxxx> > > [ Viresh: Split the patch into two ] > > Signed-off-by: Viresh Kumar <viresh.kumar@xxxxxxxxxx> > > [ Viresh: Update the code for v5.7-stable ] > > Signed-off-by: Viresh Kumar <viresh.kumar@xxxxxxxxxx> > > --- > > drivers/opp/core.c | 2 +- > > 1 file changed, 1 insertion(+), 1 deletion(-) > > This is already in the 5.7-stable queue, why add it again? > > confused, Because I received an email from your bot that it failed to apply to 5.7 stable, rightly so as the patch was required to be modified. -- viresh