From: kbuild test robot <lkp@intel.com>
To: Jorge Sanjuan <jorge.sanjuan@codethink.co.uk>
Cc: kbuild-all@01.org, tiwai@suse.com, gregkh@linuxfoundation.org,
alsa-devel@alsa-project.org, linux-kernel@vger.kernel.org,
Jorge Sanjuan <jorge.sanjuan@codethink.co.uk>
Subject: Re: [alsa-devel] [PATCH 3/4] ALSA: usb-audio: Use Class Specific EP for UAC3 devices.
Date: Mon, 23 Apr 2018 04:30:00 +0800 [thread overview]
Message-ID: <201804230447.jd3wJQ96%fengguang.wu@intel.com> (raw)
In-Reply-To: <20180420170327.31569-4-jorge.sanjuan@codethink.co.uk>
Hi Jorge,
Thank you for the patch! Perhaps something to improve:
[auto build test WARNING on sound/for-next]
[also build test WARNING on v4.17-rc1 next-20180420]
[if your patch is applied to the wrong git tree, please drop us a note to help improve the system]
url: https://github.com/0day-ci/linux/commits/Jorge-Sanjuan/ALSA-usb-UAC3-new-features/20180423-015726
base: https://git.kernel.org/pub/scm/linux/kernel/git/tiwai/sound.git for-next
reproduce:
# apt-get install sparse
make ARCH=x86_64 allmodconfig
make C=1 CF=-D__CHECK_ENDIAN__
sparse warnings: (new ones prefixed by >>)
>> sound/usb/stream.c:597:26: sparse: restricted __le32 degrades to integer
vim +597 sound/usb/stream.c
544
545 static int parse_uac_endpoint_attributes(struct snd_usb_audio *chip,
546 struct usb_host_interface *alts,
547 int protocol, int iface_no)
548 {
549 /* parsed with a v1 header here. that's ok as we only look at the
550 * header first which is the same for both versions */
551 struct uac_iso_endpoint_descriptor *csep;
552 struct usb_interface_descriptor *altsd = get_iface_desc(alts);
553 int attributes = 0;
554
555 csep = snd_usb_find_desc(alts->endpoint[0].extra, alts->endpoint[0].extralen, NULL, USB_DT_CS_ENDPOINT);
556
557 /* Creamware Noah has this descriptor after the 2nd endpoint */
558 if (!csep && altsd->bNumEndpoints >= 2)
559 csep = snd_usb_find_desc(alts->endpoint[1].extra, alts->endpoint[1].extralen, NULL, USB_DT_CS_ENDPOINT);
560
561 /*
562 * If we can't locate the USB_DT_CS_ENDPOINT descriptor in the extra
563 * bytes after the first endpoint, go search the entire interface.
564 * Some devices have it directly *before* the standard endpoint.
565 */
566 if (!csep)
567 csep = snd_usb_find_desc(alts->extra, alts->extralen, NULL, USB_DT_CS_ENDPOINT);
568
569 if (!csep || csep->bLength < 7 ||
570 csep->bDescriptorSubtype != UAC_EP_GENERAL) {
571 usb_audio_warn(chip,
572 "%u:%d : no or invalid class specific endpoint descriptor\n",
573 iface_no, altsd->bAlternateSetting);
574 return 0;
575 }
576
577 switch (protocol) {
578 case UAC_VERSION_1:
579 attributes = csep->bmAttributes;
580 break;
581 case UAC_VERSION_2: {
582 struct uac2_iso_endpoint_descriptor *csep2 =
583 (struct uac2_iso_endpoint_descriptor *) csep;
584
585 attributes = csep->bmAttributes & UAC_EP_CS_ATTR_FILL_MAX;
586
587 /* emulate the endpoint attributes of a v1 device */
588 if (csep2->bmControls & UAC2_CONTROL_PITCH)
589 attributes |= UAC_EP_CS_ATTR_PITCH_CONTROL;
590 break;
591 }
592 case UAC_VERSION_3: {
593 struct uac3_iso_endpoint_descriptor *csep3 =
594 (struct uac3_iso_endpoint_descriptor *) csep;
595
596 /* emulate the endpoint attributes of a v1 device */
> 597 if (csep3->bmControls & UAC2_CONTROL_PITCH)
598 attributes |= UAC_EP_CS_ATTR_PITCH_CONTROL;
599 break;
600 }
601 }
602
603 return attributes;
604 }
605
---
0-DAY kernel test infrastructure Open Source Technology Center
https://lists.01.org/pipermail/kbuild-all Intel Corporation
WARNING: multiple messages have this Message-ID (diff)
From: kbuild test robot <lkp@intel.com>
Cc: kbuild-all@01.org, tiwai@suse.com, gregkh@linuxfoundation.org,
alsa-devel@alsa-project.org, linux-kernel@vger.kernel.org,
Jorge Sanjuan <jorge.sanjuan@codethink.co.uk>
Subject: Re: [alsa-devel] [PATCH 3/4] ALSA: usb-audio: Use Class Specific EP for UAC3 devices.
Date: Mon, 23 Apr 2018 04:30:00 +0800 [thread overview]
Message-ID: <201804230447.jd3wJQ96%fengguang.wu@intel.com> (raw)
In-Reply-To: <20180420170327.31569-4-jorge.sanjuan@codethink.co.uk>
Hi Jorge,
Thank you for the patch! Perhaps something to improve:
[auto build test WARNING on sound/for-next]
[also build test WARNING on v4.17-rc1 next-20180420]
[if your patch is applied to the wrong git tree, please drop us a note to help improve the system]
url: https://github.com/0day-ci/linux/commits/Jorge-Sanjuan/ALSA-usb-UAC3-new-features/20180423-015726
base: https://git.kernel.org/pub/scm/linux/kernel/git/tiwai/sound.git for-next
reproduce:
# apt-get install sparse
make ARCH=x86_64 allmodconfig
make C=1 CF=-D__CHECK_ENDIAN__
sparse warnings: (new ones prefixed by >>)
>> sound/usb/stream.c:597:26: sparse: restricted __le32 degrades to integer
vim +597 sound/usb/stream.c
544
545 static int parse_uac_endpoint_attributes(struct snd_usb_audio *chip,
546 struct usb_host_interface *alts,
547 int protocol, int iface_no)
548 {
549 /* parsed with a v1 header here. that's ok as we only look at the
550 * header first which is the same for both versions */
551 struct uac_iso_endpoint_descriptor *csep;
552 struct usb_interface_descriptor *altsd = get_iface_desc(alts);
553 int attributes = 0;
554
555 csep = snd_usb_find_desc(alts->endpoint[0].extra, alts->endpoint[0].extralen, NULL, USB_DT_CS_ENDPOINT);
556
557 /* Creamware Noah has this descriptor after the 2nd endpoint */
558 if (!csep && altsd->bNumEndpoints >= 2)
559 csep = snd_usb_find_desc(alts->endpoint[1].extra, alts->endpoint[1].extralen, NULL, USB_DT_CS_ENDPOINT);
560
561 /*
562 * If we can't locate the USB_DT_CS_ENDPOINT descriptor in the extra
563 * bytes after the first endpoint, go search the entire interface.
564 * Some devices have it directly *before* the standard endpoint.
565 */
566 if (!csep)
567 csep = snd_usb_find_desc(alts->extra, alts->extralen, NULL, USB_DT_CS_ENDPOINT);
568
569 if (!csep || csep->bLength < 7 ||
570 csep->bDescriptorSubtype != UAC_EP_GENERAL) {
571 usb_audio_warn(chip,
572 "%u:%d : no or invalid class specific endpoint descriptor\n",
573 iface_no, altsd->bAlternateSetting);
574 return 0;
575 }
576
577 switch (protocol) {
578 case UAC_VERSION_1:
579 attributes = csep->bmAttributes;
580 break;
581 case UAC_VERSION_2: {
582 struct uac2_iso_endpoint_descriptor *csep2 =
583 (struct uac2_iso_endpoint_descriptor *) csep;
584
585 attributes = csep->bmAttributes & UAC_EP_CS_ATTR_FILL_MAX;
586
587 /* emulate the endpoint attributes of a v1 device */
588 if (csep2->bmControls & UAC2_CONTROL_PITCH)
589 attributes |= UAC_EP_CS_ATTR_PITCH_CONTROL;
590 break;
591 }
592 case UAC_VERSION_3: {
593 struct uac3_iso_endpoint_descriptor *csep3 =
594 (struct uac3_iso_endpoint_descriptor *) csep;
595
596 /* emulate the endpoint attributes of a v1 device */
> 597 if (csep3->bmControls & UAC2_CONTROL_PITCH)
598 attributes |= UAC_EP_CS_ATTR_PITCH_CONTROL;
599 break;
600 }
601 }
602
603 return attributes;
604 }
605
---
0-DAY kernel test infrastructure Open Source Technology Center
https://lists.01.org/pipermail/kbuild-all Intel Corporation
next prev parent reply other threads:[~2018-04-22 20:30 UTC|newest]
Thread overview: 63+ messages / expand[flat|nested] mbox.gz Atom feed top
2018-04-20 17:03 [PATCH 0/4] ALSA: usb: UAC3 new features Jorge Sanjuan
2018-04-20 17:03 ` [PATCH 1/4] ALSA: usb-audio: UAC3. Add support for mixer unit Jorge Sanjuan
2018-04-20 17:03 ` Jorge Sanjuan
2018-04-23 11:03 ` Takashi Iwai
2018-04-23 11:03 ` Takashi Iwai
2018-04-20 17:03 ` [PATCH 2/4] ALSA: usb-audio: ADC3: Fix channel mapping conversion for ADC3 Jorge Sanjuan
2018-04-20 17:03 ` Jorge Sanjuan
2018-04-23 12:11 ` Takashi Iwai
2018-04-23 12:11 ` Takashi Iwai
2018-04-24 8:03 ` [alsa-devel] " Ruslan Bilovol
2018-04-24 8:03 ` Ruslan Bilovol
2018-04-20 17:03 ` [PATCH 3/4] ALSA: usb-audio: Use Class Specific EP for UAC3 devices Jorge Sanjuan
2018-04-20 17:03 ` Jorge Sanjuan
2018-04-22 20:30 ` kbuild test robot [this message]
2018-04-22 20:30 ` [alsa-devel] " kbuild test robot
2018-04-20 17:03 ` [PATCH 4/4] ALSA: usb-audio: UAC3 Add support for connector insertion Jorge Sanjuan
2018-04-22 20:55 ` kbuild test robot
2018-04-22 20:55 ` kbuild test robot
2018-04-23 12:19 ` Takashi Iwai
2018-04-23 12:19 ` Takashi Iwai
2018-04-23 16:06 ` Jorge
2018-04-24 17:24 ` [PATCH v2 0/4] ALSA: usb: UAC3 new features Jorge Sanjuan
2018-04-24 17:24 ` Jorge Sanjuan
2018-04-24 17:24 ` [PATCH v2 1/4] ALSA: usb-audio: UAC3. Add support for mixer unit Jorge Sanjuan
2018-04-25 22:35 ` [alsa-devel] " Ruslan Bilovol
2018-04-26 16:56 ` Jorge
2018-04-26 16:56 ` Jorge
2018-04-27 17:06 ` [PATCH v3 " Jorge Sanjuan
2018-04-27 17:06 ` Jorge Sanjuan
2018-05-04 0:57 ` Ruslan Bilovol
2018-05-08 9:43 ` Jorge
2018-05-09 22:11 ` Ruslan Bilovol
2018-04-24 17:24 ` [PATCH v2 2/4] ALSA: usb-audio: ADC3: Fix channel mapping conversion for ADC3 Jorge Sanjuan
2018-04-24 17:55 ` Takashi Iwai
2018-04-24 17:55 ` Takashi Iwai
2018-04-24 17:24 ` [PATCH v2 3/4] ALSA: usb-audio: Use Class Specific EP for UAC3 devices Jorge Sanjuan
2018-04-24 17:24 ` Jorge Sanjuan
2018-04-25 22:53 ` [alsa-devel] " Ruslan Bilovol
2018-04-24 17:24 ` [PATCH v2 4/4] ALSA: usb-audio: UAC3 Add support for connector insertion Jorge Sanjuan
2018-04-24 17:24 ` Jorge Sanjuan
2018-04-24 18:02 ` [PATCH v2 0/4] ALSA: usb: UAC3 new features Takashi Iwai
2018-04-24 18:02 ` Takashi Iwai
2018-04-26 9:26 ` [alsa-devel] " Ruslan Bilovol
2018-04-26 9:26 ` Ruslan Bilovol
2018-04-26 17:13 ` [alsa-devel] " Jorge
2018-05-11 15:25 ` [PATCH v4 " Jorge Sanjuan
2018-05-11 15:25 ` Jorge Sanjuan
2018-05-11 15:25 ` [PATCH v4 1/4] ALSA: usb-audio: UAC3. Add support for mixer unit Jorge Sanjuan
2018-05-11 15:25 ` Jorge Sanjuan
2018-05-14 20:54 ` Ruslan Bilovol
2018-05-11 15:25 ` [PATCH v4 2/4] ALSA: usb-audio: Use Class Specific EP for UAC3 devices Jorge Sanjuan
2018-05-14 21:00 ` Ruslan Bilovol
2018-05-11 15:25 ` [PATCH v4 3/4] ALSA: usb-audio: UAC3 Add support for connector insertion Jorge Sanjuan
2018-05-11 15:25 ` [PATCH v4 4/4] ALSA: usb-audio: UAC3: Parse Input Terminal number of channels Jorge Sanjuan
2018-05-11 15:25 ` Jorge Sanjuan
2018-05-14 8:54 ` Jorge
2018-05-14 9:36 ` Ruslan Bilovol
2018-05-14 11:03 ` [RESEND PATCH " Jorge Sanjuan
2018-05-14 11:03 ` Jorge Sanjuan
2018-05-14 21:05 ` Ruslan Bilovol
2018-05-14 21:05 ` Ruslan Bilovol
2018-05-15 5:38 ` [PATCH v4 0/4] ALSA: usb: UAC3 new features Takashi Iwai
2018-05-15 5:38 ` Takashi Iwai
Reply instructions:
You may reply publicly to this message via plain-text email
using any one of the following methods:
* Save the following mbox file, import it into your mail client,
and reply-to-all from there: mbox
Avoid top-posting and favor interleaved quoting:
https://en.wikipedia.org/wiki/Posting_style#Interleaved_style
* Reply using the --to, --cc, and --in-reply-to
switches of git-send-email(1):
git send-email \
--in-reply-to=201804230447.jd3wJQ96%fengguang.wu@intel.com \
--to=lkp@intel.com \
--cc=alsa-devel@alsa-project.org \
--cc=gregkh@linuxfoundation.org \
--cc=jorge.sanjuan@codethink.co.uk \
--cc=kbuild-all@01.org \
--cc=linux-kernel@vger.kernel.org \
--cc=tiwai@suse.com \
/path/to/YOUR_REPLY
https://kernel.org/pub/software/scm/git/docs/git-send-email.html
* If your mail client supports setting the In-Reply-To header
via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line
before the message body.
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.