Re: [PATCH] sound/usb: Relax urb data alignment restriciton for HVR-950Q only

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

 



Hello John,

On Thu, Dec 10, 2009 at 8:38 PM, John S Gruber <johnsgruber@xxxxxxxxx> wrote:
> I think you found something in the specification I haven't found. What did you
> see that indicated how to deal with equipment misbehaving in this way?

I'm referring to section 2.3.2.3 of "Universal Serial Bus Device Class
Definition for Audio Data Formats"

> I found the following list of USB ID's. Just double checking, but is
> the 850 enough like the
> 950 line for me to include it?
>
>        case 72000: /* WinTV-HVR950q (Retail, IR, ATSC/QAM */
>        case 72001: /* WinTV-HVR950q (Retail, IR, ATSC/QAM and analog video */
>        case 72211: /* WinTV-HVR950q (OEM, IR, ATSC/QAM and analog video */
>        case 72221: /* WinTV-HVR950q (OEM, IR, ATSC/QAM and analog video */
>        case 72231: /* WinTV-HVR950q (OEM, IR, ATSC/QAM and analog video */
>        case 72241: /* WinTV-HVR950q (OEM, No IR, ATSC/QAM and analog video */
>        case 72251: /* WinTV-HVR950q (Retail, IR, ATSC/QAM and analog video */
> -->   case 72301: /* WinTV-HVR850 (Retail, IR, ATSC and analog video */
>        case 72500: /* WinTV-HVR950q (OEM, No IR, ATSC/QAM */

Yes, the HVR-850 should be included in the list.

> I'd add that being the beginner at making kernel changes your review
> is particularly
> helpful to me (and to my confidence).

You are likely to receive a more thorough/critical review when you
send to the alsa-devel mailing list, as they have considerably more
expertise in this area than I do.

Devin

-- 
Devin J. Heitmueller - Kernel Labs
http://www.kernellabs.com
--
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