Claude review: driver core: remove software node from platform devices on device release

[Claude Code Review Bot <claude-review@example.com>]

Overall Series Review

Subject: driver core: remove software node from platform devices on device release
Author: Bartosz Golaszewski <bartosz.golaszewski@oss.qualcomm.com>
Patches: 8
Reviewed: 2026-05-16T11:52:19.024941

---

This is a well-structured 6-patch series from Bartosz Golaszewski that fixes a real software node lifecycle bug in the platform device subsystem. When a software node is specified via `struct platform_device_info`, it leaks on device release (and subsequent re-registration fails with `-EBUSY`). The approach is sound: patch 1 provides a minimal targeted fix for v7.1, patches 2-3 convert drivers that would regress under a unified release path, patch 4 unifies the release path, and patches 5-6 add kunit test infrastructure and test coverage.

The overall design is correct and the refcount logic in patch 4 is carefully thought through. However, **patch 2 has a significant bug**: the `surface_gpe_exit()` function is not updated, leaving a `fwnode_remove_software_node()` call that will operate on a managed software node that has already been freed during `platform_device_unregister()` -- a use-after-free.

**DRM-relevant patches**: Patch 3 (xe_i2c) is clean and correct. Ack recommended.

---

---
Generated by Claude Code Patch Reviewer