ARM: at91: pm: fix imbalanced reference counter for ethernet devices
authorClaudiu Beznea <claudiu.beznea@microchip.com>
Thu, 18 May 2023 06:25:11 +0000 (09:25 +0300)
committerClaudiu Beznea <claudiu.beznea@microchip.com>
Wed, 24 May 2023 08:45:31 +0000 (11:45 +0300)
commitccd4923d18d5698a5910d516646ce125b9155d47
tree242a436bff6064f4f55d7898a89c53a3000ce1e4
parentac9a78681b921877518763ba0e89202254349d1b
ARM: at91: pm: fix imbalanced reference counter for ethernet devices

The of_find_device_by_node() function is returning a struct platform_device
object with the embedded struct device member's reference counter
incremented. This needs to be dropped when done with the platform device
returned by of_find_device_by_node().

at91_pm_eth_quirk_is_valid() calls of_find_device_by_node() on
suspend and resume path. On suspend it calls of_find_device_by_node() and
on resume and failure paths it drops the counter of
struct platform_device::dev.

In case ethernet device may not wakeup there is a put_device() on
at91_pm_eth_quirk_is_valid() which is wrong as it colides with
put_device() on resume path leading to the reference counter of struct
device embedded in struct platform_device to be messed, stack trace to be
displayed (after 5 consecutive suspend/resume cycles) and execution to
hang.

Along with this the error path of at91_pm_config_quirks() had been also
adapted to decrement propertly the reference counter of struct device
embedded in struct platform_device.

Fixes: b7fc72c63399 ("ARM: at91: pm: add quirks for pm")
Signed-off-by: Claudiu Beznea <claudiu.beznea@microchip.com>
Acked-by: Nicolas Ferre <nicolas.ferre@microchip.com>
Link: https://lore.kernel.org/r/20230518062511.2988500-1-claudiu.beznea@microchip.com
arch/arm/mach-at91/pm.c