Claude review: platform/surface: gpe: use platform_device_register_full()

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

Patch Review

The init path conversion is clean — replacing manual `fwnode_create_software_node()` + `platform_device_alloc()` + `platform_device_add()` with a single `platform_device_register_full()` call via `.properties` is a nice simplification.

**Bug: `surface_gpe_exit()` is not updated and becomes a use-after-free.**

The current exit function (unchanged by this patch) does:
```c
static void __exit surface_gpe_exit(void)
{
    struct fwnode_handle *fwnode = surface_gpe_device->dev.fwnode;
    platform_device_unregister(surface_gpe_device);
    platform_driver_unregister(&surface_gpe_driver);
    fwnode_remove_software_node(fwnode);
}
```

After this patch, `.properties` in `pdevinfo` causes `platform_device_register_full()` to call `device_create_managed_software_node()`. A managed swnode is automatically freed during `software_node_notify_remove()` (called from `device_del()` inside `platform_device_unregister()`). At that point, `fwnode` is a dangling pointer, and the subsequent `fwnode_remove_software_node(fwnode)` is a use-after-free.

The exit function must be updated to remove the manual `fwnode_remove_software_node()` call, since the managed node is cleaned up automatically by the unregister path. Something like:

```c
static void __exit surface_gpe_exit(void)
{
    platform_device_unregister(surface_gpe_device);
    platform_driver_unregister(&surface_gpe_driver);
}
```

---
Generated by Claude Code Patch Reviewer