diff options
author | Sam Bobroff <sam.bobroff@au1.ibm.com> | 2018-03-19 13:48:55 +1100 |
---|---|---|
committer | Michael Ellerman <mpe@ellerman.id.au> | 2018-03-27 23:44:59 +1100 |
commit | 5fd13460af36f5c6c958165d697950e676fad7c6 (patch) | |
tree | 34e339dd218bfcea856d1047bf1a071bc3f7de9a /lib/win_minmax.c | |
parent | cd95f804ac49c457c483309a58362acbc8654063 (diff) |
powerpc/eeh: Clarify arguments to eeh_reset_device()
It is currently difficult to understand the behaviour of
eeh_reset_device() due to the way it's parameters are used. In
particular, when 'bus' is NULL, it's value is still necessary so the
same value is looked up again locally under a different name
('frozen_bus') but behaviour is changed.
To clarify this, add a new parameter 'driver_eeh_aware', and have the
caller set it when it would have passed NULL for 'bus' and always pass
a value for 'bus'. Then change any test that was on 'bus' to one on
'!driver_eeh_aware' and replace uses of 'frozen_bus' with 'bus'.
Also update the function's comment.
This should not change behaviour.
Signed-off-by: Sam Bobroff <sam.bobroff@au1.ibm.com>
Reviewed-by: Alexey Kardashevskiy <aik@ozlabs.ru>
Signed-off-by: Michael Ellerman <mpe@ellerman.id.au>
Diffstat (limited to 'lib/win_minmax.c')
0 files changed, 0 insertions, 0 deletions