* [PATCH RESEND] media: uvcvideo: Add support for Apple T2-attached FaceTime HD Camera
@ 2022-01-26 10:37 Aditya Garg
2022-01-31 9:12 ` Aditya Garg
2022-02-02 6:37 ` Laurent Pinchart
0 siblings, 2 replies; 7+ messages in thread
From: Aditya Garg @ 2022-01-26 10:37 UTC (permalink / raw)
To: laurent.pinchart, laurent.pinchart+renesas, mchehab,
mchehab+huawei, hverkuil, sergey.senozhatsky, ribalda,
Linux Kernel Mailing List, linux-media
Cc: Orlando Chamberlain, paul, Aun-Ali Zaidi
From: Paul Pawlowski <paul@mrarm.io>
Adds the requisite device id to support detection of the Apple FaceTime
HD webcam exposed over the T2 BCE VHCI interface.
Tested-by: Aun-Ali Zaidi <admin@kodeit.net>
Signed-off-by: Paul Pawlowski <paul@mrarm.io>
Signed-off-by: Aun-Ali Zaidi <admin@kodeit.net>
Signed-off-by: Aditya Garg <gargaditya08@live.com>
---
drivers/media/usb/uvc/uvc_driver.c | 9 +++++++++
1 file changed, 9 insertions(+)
diff --git a/drivers/media/usb/uvc/uvc_driver.c b/drivers/media/usb/uvc/uvc_driver.c
index 7c007426e..88dc9e7aa 100644
--- a/drivers/media/usb/uvc/uvc_driver.c
+++ b/drivers/media/usb/uvc/uvc_driver.c
@@ -2848,6 +2848,15 @@ static const struct usb_device_id uvc_ids[] = {
.bInterfaceProtocol = 0,
.driver_info = UVC_INFO_QUIRK(UVC_QUIRK_PROBE_MINMAX
| UVC_QUIRK_BUILTIN_ISIGHT) },
+ /* Apple FaceTime HD Camera (Built-In) */
+ { .match_flags = USB_DEVICE_ID_MATCH_DEVICE
+ | USB_DEVICE_ID_MATCH_INT_INFO,
+ .idVendor = 0x05ac,
+ .idProduct = 0x8514,
+ .bInterfaceClass = USB_CLASS_VIDEO,
+ .bInterfaceSubClass = 1,
+ .bInterfaceProtocol = 0,
+ .driver_info = (kernel_ulong_t)&uvc_quirk_probe_def },
/* Apple Built-In iSight via iBridge */
{ .match_flags = USB_DEVICE_ID_MATCH_DEVICE
| USB_DEVICE_ID_MATCH_INT_INFO,
--
2.25.1
^ permalink raw reply related [flat|nested] 7+ messages in thread
* Re: [PATCH RESEND] media: uvcvideo: Add support for Apple T2-attached FaceTime HD Camera
2022-01-26 10:37 [PATCH RESEND] media: uvcvideo: Add support for Apple T2-attached FaceTime HD Camera Aditya Garg
@ 2022-01-31 9:12 ` Aditya Garg
2022-02-02 3:41 ` Sergey Senozhatsky
2022-02-02 6:37 ` Laurent Pinchart
1 sibling, 1 reply; 7+ messages in thread
From: Aditya Garg @ 2022-01-31 9:12 UTC (permalink / raw)
To: laurent.pinchart, laurent.pinchart+renesas, mchehab,
mchehab+huawei, hverkuil, sergey.senozhatsky, ribalda,
Linux Kernel Mailing List, linux-media
Cc: Orlando Chamberlain, paul, Aun-Ali Zaidi, Linus Torvalds
Hi Maintainers
I sent this patch on 12.01.22 and then resent it on 26.01.22. I still haven't received a reply yet. Maybe I am not sending the mail to the correct maintainer? I request someone to reply cause no reply seems to create a confusion.
Linus, I am sending this mail to you too, since you are the ultimate maintainer.
Regards
Aditya
> On 26-Jan-2022, at 4:07 PM, Aditya Garg <gargaditya08@live.com> wrote:
>
> From: Paul Pawlowski <paul@mrarm.io>
>
> Adds the requisite device id to support detection of the Apple FaceTime
> HD webcam exposed over the T2 BCE VHCI interface.
>
> Tested-by: Aun-Ali Zaidi <admin@kodeit.net>
> Signed-off-by: Paul Pawlowski <paul@mrarm.io>
> Signed-off-by: Aun-Ali Zaidi <admin@kodeit.net>
> Signed-off-by: Aditya Garg <gargaditya08@live.com>
> ---
> drivers/media/usb/uvc/uvc_driver.c | 9 +++++++++
> 1 file changed, 9 insertions(+)
>
> diff --git a/drivers/media/usb/uvc/uvc_driver.c b/drivers/media/usb/uvc/uvc_driver.c
> index 7c007426e..88dc9e7aa 100644
> --- a/drivers/media/usb/uvc/uvc_driver.c
> +++ b/drivers/media/usb/uvc/uvc_driver.c
> @@ -2848,6 +2848,15 @@ static const struct usb_device_id uvc_ids[] = {
> .bInterfaceProtocol = 0,
> .driver_info = UVC_INFO_QUIRK(UVC_QUIRK_PROBE_MINMAX
> | UVC_QUIRK_BUILTIN_ISIGHT) },
> + /* Apple FaceTime HD Camera (Built-In) */
> + { .match_flags = USB_DEVICE_ID_MATCH_DEVICE
> + | USB_DEVICE_ID_MATCH_INT_INFO,
> + .idVendor = 0x05ac,
> + .idProduct = 0x8514,
> + .bInterfaceClass = USB_CLASS_VIDEO,
> + .bInterfaceSubClass = 1,
> + .bInterfaceProtocol = 0,
> + .driver_info = (kernel_ulong_t)&uvc_quirk_probe_def },
> /* Apple Built-In iSight via iBridge */
> { .match_flags = USB_DEVICE_ID_MATCH_DEVICE
> | USB_DEVICE_ID_MATCH_INT_INFO,
> --
> 2.25.1
>
>
^ permalink raw reply [flat|nested] 7+ messages in thread
* Re: [PATCH RESEND] media: uvcvideo: Add support for Apple T2-attached FaceTime HD Camera
2022-01-31 9:12 ` Aditya Garg
@ 2022-02-02 3:41 ` Sergey Senozhatsky
0 siblings, 0 replies; 7+ messages in thread
From: Sergey Senozhatsky @ 2022-02-02 3:41 UTC (permalink / raw)
To: Aditya Garg
Cc: laurent.pinchart, laurent.pinchart+renesas, mchehab,
mchehab+huawei, hverkuil, sergey.senozhatsky, ribalda,
Linux Kernel Mailing List, linux-media, Orlando Chamberlain,
paul, Aun-Ali Zaidi, Linus Torvalds
On (22/01/31 09:12), Aditya Garg wrote:
> > On 26-Jan-2022, at 4:07 PM, Aditya Garg <gargaditya08@live.com> wrote:
> >
> > From: Paul Pawlowski <paul@mrarm.io>
> >
> > Adds the requisite device id to support detection of the Apple FaceTime
> > HD webcam exposed over the T2 BCE VHCI interface.
> >
> > Tested-by: Aun-Ali Zaidi <admin@kodeit.net>
> > Signed-off-by: Paul Pawlowski <paul@mrarm.io>
> > Signed-off-by: Aun-Ali Zaidi <admin@kodeit.net>
> > Signed-off-by: Aditya Garg <gargaditya08@live.com>
I'm not the maintainer, but the patch looks OK to me
FWIW
Reviewed-by: Sergey Senozhatsky <senozhatsky@chromium.org>
Laurent, Ricardo, any thoughts?
> > drivers/media/usb/uvc/uvc_driver.c | 9 +++++++++
> > 1 file changed, 9 insertions(+)
> >
> > diff --git a/drivers/media/usb/uvc/uvc_driver.c b/drivers/media/usb/uvc/uvc_driver.c
> > index 7c007426e..88dc9e7aa 100644
> > --- a/drivers/media/usb/uvc/uvc_driver.c
> > +++ b/drivers/media/usb/uvc/uvc_driver.c
> > @@ -2848,6 +2848,15 @@ static const struct usb_device_id uvc_ids[] = {
> > .bInterfaceProtocol = 0,
> > .driver_info = UVC_INFO_QUIRK(UVC_QUIRK_PROBE_MINMAX
> > | UVC_QUIRK_BUILTIN_ISIGHT) },
> > + /* Apple FaceTime HD Camera (Built-In) */
> > + { .match_flags = USB_DEVICE_ID_MATCH_DEVICE
> > + | USB_DEVICE_ID_MATCH_INT_INFO,
> > + .idVendor = 0x05ac,
> > + .idProduct = 0x8514,
> > + .bInterfaceClass = USB_CLASS_VIDEO,
> > + .bInterfaceSubClass = 1,
> > + .bInterfaceProtocol = 0,
> > + .driver_info = (kernel_ulong_t)&uvc_quirk_probe_def },
> > /* Apple Built-In iSight via iBridge */
> > { .match_flags = USB_DEVICE_ID_MATCH_DEVICE
> > | USB_DEVICE_ID_MATCH_INT_INFO,
^ permalink raw reply [flat|nested] 7+ messages in thread
* Re: [PATCH RESEND] media: uvcvideo: Add support for Apple T2-attached FaceTime HD Camera
2022-01-26 10:37 [PATCH RESEND] media: uvcvideo: Add support for Apple T2-attached FaceTime HD Camera Aditya Garg
2022-01-31 9:12 ` Aditya Garg
@ 2022-02-02 6:37 ` Laurent Pinchart
2022-02-02 7:08 ` Aditya Garg
1 sibling, 1 reply; 7+ messages in thread
From: Laurent Pinchart @ 2022-02-02 6:37 UTC (permalink / raw)
To: Aditya Garg
Cc: mchehab, mchehab+huawei, hverkuil, sergey.senozhatsky, ribalda,
Linux Kernel Mailing List, linux-media, Orlando Chamberlain,
paul, Aun-Ali Zaidi
Hi Aditya,
Thank you for the patch, and sorry for the late reply.
On Wed, Jan 26, 2022 at 10:37:13AM +0000, Aditya Garg wrote:
> From: Paul Pawlowski <paul@mrarm.io>
>
> Adds the requisite device id to support detection of the Apple FaceTime
> HD webcam exposed over the T2 BCE VHCI interface.
>
> Tested-by: Aun-Ali Zaidi <admin@kodeit.net>
> Signed-off-by: Paul Pawlowski <paul@mrarm.io>
> Signed-off-by: Aun-Ali Zaidi <admin@kodeit.net>
> Signed-off-by: Aditya Garg <gargaditya08@live.com>
> ---
> drivers/media/usb/uvc/uvc_driver.c | 9 +++++++++
> 1 file changed, 9 insertions(+)
>
> diff --git a/drivers/media/usb/uvc/uvc_driver.c b/drivers/media/usb/uvc/uvc_driver.c
> index 7c007426e..88dc9e7aa 100644
> --- a/drivers/media/usb/uvc/uvc_driver.c
> +++ b/drivers/media/usb/uvc/uvc_driver.c
> @@ -2848,6 +2848,15 @@ static const struct usb_device_id uvc_ids[] = {
> .bInterfaceProtocol = 0,
> .driver_info = UVC_INFO_QUIRK(UVC_QUIRK_PROBE_MINMAX
> | UVC_QUIRK_BUILTIN_ISIGHT) },
> + /* Apple FaceTime HD Camera (Built-In) */
> + { .match_flags = USB_DEVICE_ID_MATCH_DEVICE
> + | USB_DEVICE_ID_MATCH_INT_INFO,
> + .idVendor = 0x05ac,
> + .idProduct = 0x8514,
> + .bInterfaceClass = USB_CLASS_VIDEO,
> + .bInterfaceSubClass = 1,
> + .bInterfaceProtocol = 0,
> + .driver_info = (kernel_ulong_t)&uvc_quirk_probe_def },
Have you tested that the device doesn't work without this quirk ?
> /* Apple Built-In iSight via iBridge */
> { .match_flags = USB_DEVICE_ID_MATCH_DEVICE
> | USB_DEVICE_ID_MATCH_INT_INFO,
--
Regards,
Laurent Pinchart
^ permalink raw reply [flat|nested] 7+ messages in thread
* Re: [PATCH RESEND] media: uvcvideo: Add support for Apple T2-attached FaceTime HD Camera
2022-02-02 6:37 ` Laurent Pinchart
@ 2022-02-02 7:08 ` Aditya Garg
2022-02-02 23:59 ` Laurent Pinchart
0 siblings, 1 reply; 7+ messages in thread
From: Aditya Garg @ 2022-02-02 7:08 UTC (permalink / raw)
To: Laurent Pinchart
Cc: mchehab, mchehab+huawei, hverkuil, sergey.senozhatsky, ribalda,
Linux Kernel Mailing List, linux-media, Orlando Chamberlain,
paul, Aun-Ali Zaidi
Hi Laurent
> On 02-Feb-2022, at 12:07 PM, Laurent Pinchart <laurent.pinchart@ideasonboard.com> wrote:
>
> Hi Aditya,
>
> Thank you for the patch, and sorry for the late reply.
>
> On Wed, Jan 26, 2022 at 10:37:13AM +0000, Aditya Garg wrote:
>> From: Paul Pawlowski <paul@mrarm.io>
>>
>> Adds the requisite device id to support detection of the Apple FaceTime
>> HD webcam exposed over the T2 BCE VHCI interface.
>>
>> Tested-by: Aun-Ali Zaidi <admin@kodeit.net>
>> Signed-off-by: Paul Pawlowski <paul@mrarm.io>
>> Signed-off-by: Aun-Ali Zaidi <admin@kodeit.net>
>> Signed-off-by: Aditya Garg <gargaditya08@live.com>
>> ---
>> drivers/media/usb/uvc/uvc_driver.c | 9 +++++++++
>> 1 file changed, 9 insertions(+)
>>
>> diff --git a/drivers/media/usb/uvc/uvc_driver.c b/drivers/media/usb/uvc/uvc_driver.c
>> index 7c007426e..88dc9e7aa 100644
>> --- a/drivers/media/usb/uvc/uvc_driver.c
>> +++ b/drivers/media/usb/uvc/uvc_driver.c
>> @@ -2848,6 +2848,15 @@ static const struct usb_device_id uvc_ids[] = {
>> .bInterfaceProtocol = 0,
>> .driver_info = UVC_INFO_QUIRK(UVC_QUIRK_PROBE_MINMAX
>> | UVC_QUIRK_BUILTIN_ISIGHT) },
>> + /* Apple FaceTime HD Camera (Built-In) */
>> + { .match_flags = USB_DEVICE_ID_MATCH_DEVICE
>> + | USB_DEVICE_ID_MATCH_INT_INFO,
>> + .idVendor = 0x05ac,
>> + .idProduct = 0x8514,
>> + .bInterfaceClass = USB_CLASS_VIDEO,
>> + .bInterfaceSubClass = 1,
>> + .bInterfaceProtocol = 0,
>> + .driver_info = (kernel_ulong_t)&uvc_quirk_probe_def },
>
> Have you tested that the device doesn't work without this quirk ?
uvcvideo 1-2:1.1: Failed to query (129) UVC probe control : -110 (exp. 48).
uvcvideo 1-2:1.1: Failed to initialize the device (-5).
These errors are encountered without this patch
>
>> /* Apple Built-In iSight via iBridge */
>> { .match_flags = USB_DEVICE_ID_MATCH_DEVICE
>> | USB_DEVICE_ID_MATCH_INT_INFO,
>
> --
> Regards,
>
> Laurent Pinchart
^ permalink raw reply [flat|nested] 7+ messages in thread
* Re: [PATCH RESEND] media: uvcvideo: Add support for Apple T2-attached FaceTime HD Camera
2022-02-02 7:08 ` Aditya Garg
@ 2022-02-02 23:59 ` Laurent Pinchart
2022-02-03 12:51 ` Aditya Garg
0 siblings, 1 reply; 7+ messages in thread
From: Laurent Pinchart @ 2022-02-02 23:59 UTC (permalink / raw)
To: Aditya Garg
Cc: mchehab, mchehab+huawei, hverkuil, sergey.senozhatsky, ribalda,
Linux Kernel Mailing List, linux-media, Orlando Chamberlain,
paul, Aun-Ali Zaidi
Hi Aditya,
On Wed, Feb 02, 2022 at 07:08:21AM +0000, Aditya Garg wrote:
> > On 02-Feb-2022, at 12:07 PM, Laurent Pinchart wrote:
> > On Wed, Jan 26, 2022 at 10:37:13AM +0000, Aditya Garg wrote:
> >> From: Paul Pawlowski <paul@mrarm.io>
> >>
> >> Adds the requisite device id to support detection of the Apple FaceTime
> >> HD webcam exposed over the T2 BCE VHCI interface.
> >>
> >> Tested-by: Aun-Ali Zaidi <admin@kodeit.net>
> >> Signed-off-by: Paul Pawlowski <paul@mrarm.io>
> >> Signed-off-by: Aun-Ali Zaidi <admin@kodeit.net>
> >> Signed-off-by: Aditya Garg <gargaditya08@live.com>
> >> ---
> >> drivers/media/usb/uvc/uvc_driver.c | 9 +++++++++
> >> 1 file changed, 9 insertions(+)
> >>
> >> diff --git a/drivers/media/usb/uvc/uvc_driver.c b/drivers/media/usb/uvc/uvc_driver.c
> >> index 7c007426e..88dc9e7aa 100644
> >> --- a/drivers/media/usb/uvc/uvc_driver.c
> >> +++ b/drivers/media/usb/uvc/uvc_driver.c
> >> @@ -2848,6 +2848,15 @@ static const struct usb_device_id uvc_ids[] = {
> >> .bInterfaceProtocol = 0,
> >> .driver_info = UVC_INFO_QUIRK(UVC_QUIRK_PROBE_MINMAX
> >> | UVC_QUIRK_BUILTIN_ISIGHT) },
> >> + /* Apple FaceTime HD Camera (Built-In) */
> >> + { .match_flags = USB_DEVICE_ID_MATCH_DEVICE
> >> + | USB_DEVICE_ID_MATCH_INT_INFO,
> >> + .idVendor = 0x05ac,
> >> + .idProduct = 0x8514,
> >> + .bInterfaceClass = USB_CLASS_VIDEO,
> >> + .bInterfaceSubClass = 1,
> >> + .bInterfaceProtocol = 0,
> >> + .driver_info = (kernel_ulong_t)&uvc_quirk_probe_def },
> >
> > Have you tested that the device doesn't work without this quirk ?
>
> uvcvideo 1-2:1.1: Failed to query (129) UVC probe control : -110 (exp. 48).
> uvcvideo 1-2:1.1: Failed to initialize the device (-5).
>
> These errors are encountered without this patch
Thank you for the confirmation.
Reviewed-by: Laurent Pinchart <laurent.pinchart@ideasonboard.com>
and applied to my tree.
> >> /* Apple Built-In iSight via iBridge */
> >> { .match_flags = USB_DEVICE_ID_MATCH_DEVICE
> >> | USB_DEVICE_ID_MATCH_INT_INFO,
--
Regards,
Laurent Pinchart
^ permalink raw reply [flat|nested] 7+ messages in thread
* Re: [PATCH RESEND] media: uvcvideo: Add support for Apple T2-attached FaceTime HD Camera
2022-02-02 23:59 ` Laurent Pinchart
@ 2022-02-03 12:51 ` Aditya Garg
0 siblings, 0 replies; 7+ messages in thread
From: Aditya Garg @ 2022-02-03 12:51 UTC (permalink / raw)
To: Laurent Pinchart
Cc: mchehab, mchehab+huawei, hverkuil, sergey.senozhatsky, ribalda,
Linux Kernel Mailing List, linux-media, Orlando Chamberlain,
paul, Aun-Ali Zaidi
> On 03-Feb-2022, at 5:29 AM, Laurent Pinchart <laurent.pinchart@ideasonboard.com> wrote:
>
> Hi Aditya,
>
> On Wed, Feb 02, 2022 at 07:08:21AM +0000, Aditya Garg wrote:
>>> On 02-Feb-2022, at 12:07 PM, Laurent Pinchart wrote:
>>> On Wed, Jan 26, 2022 at 10:37:13AM +0000, Aditya Garg wrote:
>>>> From: Paul Pawlowski <paul@mrarm.io>
>>>>
>>>> Adds the requisite device id to support detection of the Apple FaceTime
>>>> HD webcam exposed over the T2 BCE VHCI interface.
>>>>
>>>> Tested-by: Aun-Ali Zaidi <admin@kodeit.net>
>>>> Signed-off-by: Paul Pawlowski <paul@mrarm.io>
>>>> Signed-off-by: Aun-Ali Zaidi <admin@kodeit.net>
>>>> Signed-off-by: Aditya Garg <gargaditya08@live.com>
>>>> ---
>>>> drivers/media/usb/uvc/uvc_driver.c | 9 +++++++++
>>>> 1 file changed, 9 insertions(+)
>>>>
>>>> diff --git a/drivers/media/usb/uvc/uvc_driver.c b/drivers/media/usb/uvc/uvc_driver.c
>>>> index 7c007426e..88dc9e7aa 100644
>>>> --- a/drivers/media/usb/uvc/uvc_driver.c
>>>> +++ b/drivers/media/usb/uvc/uvc_driver.c
>>>> @@ -2848,6 +2848,15 @@ static const struct usb_device_id uvc_ids[] = {
>>>> .bInterfaceProtocol = 0,
>>>> .driver_info = UVC_INFO_QUIRK(UVC_QUIRK_PROBE_MINMAX
>>>> | UVC_QUIRK_BUILTIN_ISIGHT) },
>>>> + /* Apple FaceTime HD Camera (Built-In) */
>>>> + { .match_flags = USB_DEVICE_ID_MATCH_DEVICE
>>>> + | USB_DEVICE_ID_MATCH_INT_INFO,
>>>> + .idVendor = 0x05ac,
>>>> + .idProduct = 0x8514,
>>>> + .bInterfaceClass = USB_CLASS_VIDEO,
>>>> + .bInterfaceSubClass = 1,
>>>> + .bInterfaceProtocol = 0,
>>>> + .driver_info = (kernel_ulong_t)&uvc_quirk_probe_def },
>>>
>>> Have you tested that the device doesn't work without this quirk ?
>>
>> uvcvideo 1-2:1.1: Failed to query (129) UVC probe control : -110 (exp. 48).
>> uvcvideo 1-2:1.1: Failed to initialize the device (-5).
>>
>> These errors are encountered without this patch
>
> Thank you for the confirmation.
>
> Reviewed-by: Laurent Pinchart <laurent.pinchart@ideasonboard.com>
>
> and applied to my tree.
Thanks !
>
>>>> /* Apple Built-In iSight via iBridge */
>>>> { .match_flags = USB_DEVICE_ID_MATCH_DEVICE
>>>> | USB_DEVICE_ID_MATCH_INT_INFO,
>
> --
> Regards,
>
> Laurent Pinchart
^ permalink raw reply [flat|nested] 7+ messages in thread
end of thread, other threads:[~2022-02-03 12:51 UTC | newest]
Thread overview: 7+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2022-01-26 10:37 [PATCH RESEND] media: uvcvideo: Add support for Apple T2-attached FaceTime HD Camera Aditya Garg
2022-01-31 9:12 ` Aditya Garg
2022-02-02 3:41 ` Sergey Senozhatsky
2022-02-02 6:37 ` Laurent Pinchart
2022-02-02 7:08 ` Aditya Garg
2022-02-02 23:59 ` Laurent Pinchart
2022-02-03 12:51 ` Aditya Garg
This is an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.