On Thu, Jun 22, 2017 at 11:32:15AM +0200, Greg Kurz wrote:
> On Wed, 21 Jun 2017 17:18:45 +0800
> David Gibson wrote:
>
> > This function has two unused parameters - remove them.
> >
>
> It's ok for the d argument but I'm not sure about errp... Indeed it isn't used
On Wed, 21 Jun 2017 17:18:45 +0800
David Gibson wrote:
> This function has two unused parameters - remove them.
>
It's ok for the d argument but I'm not sure about errp... Indeed it isn't used
in the current code, but looking at the paths below spapr_drc_detach(),
This function has two unused parameters - remove them.
It also sets awaiting_release on all paths, except one. On that path
setting it is harmless, since it will be immediately cleared by
spapr_drc_release(). So factor it out of the if statements.
Signed-off-by: David Gibson