[PATCH v1] s390/ap: Drop unused member from ap_device_id

Uwe Kleine-König (The Capable Hub) posted 1 patch 1 month, 3 weeks ago
There is a newer version of this series
include/linux/device-id/ap.h | 1 -
1 file changed, 1 deletion(-)
[PATCH v1] s390/ap: Drop unused member from ap_device_id
Posted by Uwe Kleine-König (The Capable Hub) 1 month, 3 weeks ago
ap_device_id::device_info is not used in the kernel. The structure is
also not part of API/ABI, so the unused member can just be dropped.

Signed-off-by: Uwe Kleine-König (The Capable Hub) <u.kleine-koenig@baylibre.com>
---
 include/linux/device-id/ap.h | 1 -
 1 file changed, 1 deletion(-)

diff --git a/include/linux/device-id/ap.h b/include/linux/device-id/ap.h
index 0992333a34db..011af33c0bcf 100644
--- a/include/linux/device-id/ap.h
+++ b/include/linux/device-id/ap.h
@@ -14,7 +14,6 @@ typedef unsigned long kernel_ulong_t;
 struct ap_device_id {
 	__u16 match_flags;	/* which fields to match against */
 	__u8 dev_type;		/* device type */
-	kernel_ulong_t driver_info;
 };
 
 #endif /* ifndef LINUX_DEVICE_ID_AP_H */

base-commit: 1701fda2f58e345c050f4309971bdc07cd6146ba
-- 
2.55.0.11.g153666a7d9bb

Re: [PATCH v1] s390/ap: Drop unused member from ap_device_id
Posted by Harald Freudenberger 1 month, 2 weeks ago
On 2026-08-06 17:57, Uwe Kleine-König wrote:
> ap_device_id::device_info is not used in the kernel. The structure is
> also not part of API/ABI, so the unused member can just be dropped.
> 
> Signed-off-by: Uwe Kleine-König (The Capable Hub) 
> <u.kleine-koenig@baylibre.com>
> ---
>  include/linux/device-id/ap.h | 1 -
>  1 file changed, 1 deletion(-)
> 
> diff --git a/include/linux/device-id/ap.h 
> b/include/linux/device-id/ap.h
> index 0992333a34db..011af33c0bcf 100644
> --- a/include/linux/device-id/ap.h
> +++ b/include/linux/device-id/ap.h
> @@ -14,7 +14,6 @@ typedef unsigned long kernel_ulong_t;
>  struct ap_device_id {
>  	__u16 match_flags;	/* which fields to match against */
>  	__u8 dev_type;		/* device type */
> -	kernel_ulong_t driver_info;
>  };
> 
>  #endif /* ifndef LINUX_DEVICE_ID_AP_H */
> 
> base-commit: 1701fda2f58e345c050f4309971bdc07cd6146ba

Reviewed-by: Harald Freudenberger <freude@linux.ibm.com>
[PATCH v2] s390/ap: Drop unused member from ap_device_id
Posted by Uwe Kleine-König (The Capable Hub) 1 month, 3 weeks ago
ap_device_id::driver_info is not used in the kernel. The structure is
also not part of API/ABI, so the unused member can just be dropped.

Signed-off-by: Uwe Kleine-König (The Capable Hub) <u.kleine-koenig@baylibre.com>
---
Hello,

Sashiko was quick to identify two issues in my patch. Before someone
gets around to review v1 by hand, here is a v2 addressing Sashiko's
concerns:

 - remove the typedef for kernel_ulong_t
 - s/device_info/driver_info/ in the commit log

Thanks
Uwe

 include/linux/device-id/ap.h | 2 --
 1 file changed, 2 deletions(-)

diff --git a/include/linux/device-id/ap.h b/include/linux/device-id/ap.h
index 0992333a34db..e050abebbf3d 100644
--- a/include/linux/device-id/ap.h
+++ b/include/linux/device-id/ap.h
@@ -4,7 +4,6 @@
 
 #ifdef __KERNEL__
 #include <linux/types.h>
-typedef unsigned long kernel_ulong_t;
 #endif
 
 #define AP_DEVICE_ID_MATCH_CARD_TYPE		0x01
@@ -14,7 +13,6 @@ typedef unsigned long kernel_ulong_t;
 struct ap_device_id {
 	__u16 match_flags;	/* which fields to match against */
 	__u8 dev_type;		/* device type */
-	kernel_ulong_t driver_info;
 };
 
 #endif /* ifndef LINUX_DEVICE_ID_AP_H */

base-commit: 1701fda2f58e345c050f4309971bdc07cd6146ba
-- 
2.55.0.11.g153666a7d9bb

Re: [PATCH v2] s390/ap: Drop unused member from ap_device_id
Posted by Vasily Gorbik 1 month, 1 week ago
On Thu, Aug 06, 2026 at 06:21:19PM +0200, Uwe Kleine-König (The Capable Hub) wrote:
> ap_device_id::driver_info is not used in the kernel. The structure is
> also not part of API/ABI, so the unused member can just be dropped.
> 
> Signed-off-by: Uwe Kleine-König (The Capable Hub) <u.kleine-koenig@baylibre.com>
> ---
> Hello,
> 
> Sashiko was quick to identify two issues in my patch. Before someone
> gets around to review v1 by hand, here is a v2 addressing Sashiko's
> concerns:
> 
>  - remove the typedef for kernel_ulong_t
>  - s/device_info/driver_info/ in the commit log
> 
> Thanks
> Uwe
> 
>  include/linux/device-id/ap.h | 2 --
>  1 file changed, 2 deletions(-)

Applied, thank you!
Re: [PATCH v2] s390/ap: Drop unused member from ap_device_id
Posted by Holger Dengler 1 month, 3 weeks ago
On 8/6/26 18:21, Uwe Kleine-König (The Capable Hub) wrote:
> ap_device_id::driver_info is not used in the kernel. The structure is
> also not part of API/ABI, so the unused member can just be dropped.
> 
> Signed-off-by: Uwe Kleine-König (The Capable Hub) <u.kleine-koenig@baylibre.com>

Acked-by: Holger Dengler <dengler@linux.ibm.com>

-- 
Mit freundlichen Grüßen / Kind regards
Holger Dengler