* [PATCH] media: v4l2-ctrls: Enable support for u64 value
@ 2025-08-31 8:06 Sunny Patel
2025-08-31 9:06 ` Hans Verkuil
0 siblings, 1 reply; 2+ messages in thread
From: Sunny Patel @ 2025-08-31 8:06 UTC (permalink / raw)
To: Sakari Ailus
Cc: Hans Verkuil, Laurent Pinchart, Ricardo Ribalda, linux-media,
linux-kernel, Sunny Patel
Added Support for unsigned 64 bit value
Signed-off-by: Sunny Patel <nueralspacetech@gmail.com>
---
include/media/v4l2-ctrls.h | 2 ++
1 file changed, 2 insertions(+)
diff --git a/include/media/v4l2-ctrls.h b/include/media/v4l2-ctrls.h
index c32c46286441..7b2bc459f378 100644
--- a/include/media/v4l2-ctrls.h
+++ b/include/media/v4l2-ctrls.h
@@ -250,6 +250,7 @@ typedef void (*v4l2_ctrl_notify_fnc)(struct v4l2_ctrl *ctrl, void *priv);
* @cur.val: The control's current value, if the @type is represented via
* a u32 integer (see &enum v4l2_ctrl_type).
* @val: The control's new s32 value.
+ * @val64: The control's new u64 value.
* @p_def: The control's default value represented via a union which
* provides a standard way of accessing control types
* through a pointer (for compound controls only).
@@ -313,6 +314,7 @@ struct v4l2_ctrl {
void *p_array;
u32 p_array_alloc_elems;
s32 val;
+ u64 val64;
struct {
s32 val;
} cur;
--
2.43.0
^ permalink raw reply related [flat|nested] 2+ messages in thread
* Re: [PATCH] media: v4l2-ctrls: Enable support for u64 value
2025-08-31 8:06 [PATCH] media: v4l2-ctrls: Enable support for u64 value Sunny Patel
@ 2025-08-31 9:06 ` Hans Verkuil
0 siblings, 0 replies; 2+ messages in thread
From: Hans Verkuil @ 2025-08-31 9:06 UTC (permalink / raw)
To: Sunny Patel, Sakari Ailus
Cc: Hans Verkuil, Laurent Pinchart, Ricardo Ribalda, linux-media,
linux-kernel
On 31/08/2025 10:06, Sunny Patel wrote:
> Added Support for unsigned 64 bit value
>
> Signed-off-by: Sunny Patel <nueralspacetech@gmail.com>
> ---
> include/media/v4l2-ctrls.h | 2 ++
> 1 file changed, 2 insertions(+)
>
> diff --git a/include/media/v4l2-ctrls.h b/include/media/v4l2-ctrls.h
> index c32c46286441..7b2bc459f378 100644
> --- a/include/media/v4l2-ctrls.h
> +++ b/include/media/v4l2-ctrls.h
> @@ -250,6 +250,7 @@ typedef void (*v4l2_ctrl_notify_fnc)(struct v4l2_ctrl *ctrl, void *priv);
> * @cur.val: The control's current value, if the @type is represented via
> * a u32 integer (see &enum v4l2_ctrl_type).
> * @val: The control's new s32 value.
> + * @val64: The control's new u64 value.
> * @p_def: The control's default value represented via a union which
> * provides a standard way of accessing control types
> * through a pointer (for compound controls only).
> @@ -313,6 +314,7 @@ struct v4l2_ctrl {
> void *p_array;
> u32 p_array_alloc_elems;
> s32 val;
> + u64 val64;
> struct {
> s32 val;
> } cur;
????
This patch makes no sense whatsoever. Rejecting.
Regards,
Hans
^ permalink raw reply [flat|nested] 2+ messages in thread
end of thread, other threads:[~2025-08-31 9:06 UTC | newest]
Thread overview: 2+ messages (download: mbox.gz follow: Atom feed
-- links below jump to the message on this page --
2025-08-31 8:06 [PATCH] media: v4l2-ctrls: Enable support for u64 value Sunny Patel
2025-08-31 9:06 ` Hans Verkuil
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for NNTP newsgroup(s).