mbox series

[net,v2,0/5] ptp: vmclock: bugfixes and cleanups for error handling

Message ID 20250207-vmclock-probe-v2-0-bc2fce0bdf07@linutronix.de (mailing list archive)
Headers show
Series ptp: vmclock: bugfixes and cleanups for error handling | expand

Message

Thomas Weißschuh Feb. 7, 2025, 9:39 a.m. UTC
Some error handling issues I noticed while looking at the code.

Only compile-tested.

Signed-off-by: Thomas Weißschuh <thomas.weissschuh@linutronix.de>
---
Changes in v2:
- Fix typo in commit message: vmclock_ptp_register()
- Retarget against net tree
- Include patch from David. It's at the start of the series so that all
  bugfixes are at the beginning and the logical ordering of my patches
  is not disrupted.
- Pick up tags from LKML
- Link to v1: https://lore.kernel.org/r/20250206-vmclock-probe-v1-0-17a3ea07be34@linutronix.de

---
David Woodhouse (1):
      ptp: vmclock: Add .owner to vmclock_miscdev_fops

Thomas Weißschuh (4):
      ptp: vmclock: Set driver data before its usage
      ptp: vmclock: Don't unregister misc device if it was not registered
      ptp: vmclock: Clean up miscdev and ptp clock through devres
      ptp: vmclock: Remove goto-based cleanup logic

 drivers/ptp/ptp_vmclock.c | 47 +++++++++++++++++++++--------------------------
 1 file changed, 21 insertions(+), 26 deletions(-)
---
base-commit: 2014c95afecee3e76ca4a56956a936e23283f05b
change-id: 20250206-vmclock-probe-57cbcb770925

Best regards,

Comments

patchwork-bot+netdevbpf@kernel.org Feb. 11, 2025, 9:40 a.m. UTC | #1
Hello:

This series was applied to netdev/net.git (main)
by Paolo Abeni <pabeni@redhat.com>:

On Fri, 07 Feb 2025 10:39:01 +0100 you wrote:
> Some error handling issues I noticed while looking at the code.
> 
> Only compile-tested.
> 
> Signed-off-by: Thomas Weißschuh <thomas.weissschuh@linutronix.de>
> ---
> Changes in v2:
> - Fix typo in commit message: vmclock_ptp_register()
> - Retarget against net tree
> - Include patch from David. It's at the start of the series so that all
>   bugfixes are at the beginning and the logical ordering of my patches
>   is not disrupted.
> - Pick up tags from LKML
> - Link to v1: https://lore.kernel.org/r/20250206-vmclock-probe-v1-0-17a3ea07be34@linutronix.de
> 
> [...]

Here is the summary with links:
  - [net,v2,1/5] ptp: vmclock: Add .owner to vmclock_miscdev_fops
    https://git.kernel.org/netdev/net/c/7b07b040257c
  - [net,v2,2/5] ptp: vmclock: Set driver data before its usage
    https://git.kernel.org/netdev/net/c/f7d07cd4f77d
  - [net,v2,3/5] ptp: vmclock: Don't unregister misc device if it was not registered
    https://git.kernel.org/netdev/net/c/39e926c3a21b
  - [net,v2,4/5] ptp: vmclock: Clean up miscdev and ptp clock through devres
    https://git.kernel.org/netdev/net/c/9a884c3800b2
  - [net,v2,5/5] ptp: vmclock: Remove goto-based cleanup logic
    https://git.kernel.org/netdev/net/c/b4c1fde5ced9

You are awesome, thank you!