Re: [PATCH] gspca_stv06xx: enable button found on some Quickcam Express variant

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 



On Tue, 28 Oct 2014 15:39:41 +0100
Antonio Ospite <ao2@xxxxxx> wrote:

> On Mon, 14 Jul 2014 12:27:57 +0200
> Hans de Goede <hdegoede@xxxxxxxxxx> wrote:
> 
> > Hi,
> > 
> > On 07/11/2014 02:56 PM, Antonio Ospite wrote:
> > > Signed-off-by: Antonio Ospite <ao2@xxxxxx>
> > 
> > Thanks, I've added this to my tree and send a pull-req for it
> > to Mauro.
> >
> 
> Hi Hans, I still don't see the change in 3.18-rc2, maybe it got lost.
> 
> Here is the patchwork link in case you want to pick the change for 3.19:
> https://patchwork.linuxtv.org/patch/24732/
> 

Ping.

Still missing in 3.19-rc3.
Can we have it for 3.20?

Thanks,
   Antonio

> > Regards,
> > 
> > Hans
> > 
> > > ---
> > >  drivers/media/usb/gspca/stv06xx/stv06xx.c | 4 ++--
> > >  1 file changed, 2 insertions(+), 2 deletions(-)
> > > 
> > > diff --git a/drivers/media/usb/gspca/stv06xx/stv06xx.c b/drivers/media/usb/gspca/stv06xx/stv06xx.c
> > > index 49d209b..6ac93d8 100644
> > > --- a/drivers/media/usb/gspca/stv06xx/stv06xx.c
> > > +++ b/drivers/media/usb/gspca/stv06xx/stv06xx.c
> > > @@ -505,13 +505,13 @@ static int sd_int_pkt_scan(struct gspca_dev *gspca_dev,
> > >  {
> > >  	int ret = -EINVAL;
> > >  
> > > -	if (len == 1 && data[0] == 0x80) {
> > > +	if (len == 1 && (data[0] == 0x80 || data[0] == 0x10)) {
> > >  		input_report_key(gspca_dev->input_dev, KEY_CAMERA, 1);
> > >  		input_sync(gspca_dev->input_dev);
> > >  		ret = 0;
> > >  	}
> > >  
> > > -	if (len == 1 && data[0] == 0x88) {
> > > +	if (len == 1 && (data[0] == 0x88 || data[0] == 0x11)) {
> > >  		input_report_key(gspca_dev->input_dev, KEY_CAMERA, 0);
> > >  		input_sync(gspca_dev->input_dev);
> > >  		ret = 0;
> > > 
> 
> 
> -- 
> Antonio Ospite
> http://ao2.it
> 
> A: Because it messes up the order in which people normally read text.
>    See http://en.wikipedia.org/wiki/Posting_style
> Q: Why is top-posting such a bad thing?
> --
> To unsubscribe from this list: send the line "unsubscribe linux-media" in
> the body of a message to majordomo@xxxxxxxxxxxxxxx
> More majordomo info at  http://vger.kernel.org/majordomo-info.html


-- 
Antonio Ospite
http://ao2.it

A: Because it messes up the order in which people normally read text.
   See http://en.wikipedia.org/wiki/Posting_style
Q: Why is top-posting such a bad thing?
--
To unsubscribe from this list: send the line "unsubscribe linux-media" in
the body of a message to majordomo@xxxxxxxxxxxxxxx
More majordomo info at  http://vger.kernel.org/majordomo-info.html



[Index of Archives]     [Linux Input]     [Video for Linux]     [Gstreamer Embedded]     [Mplayer Users]     [Linux USB Devel]     [Linux Audio Users]     [Linux Kernel]     [Linux SCSI]     [Yosemite Backpacking]
  Powered by Linux