From: Jack Pham <jackp@codeaurora.org>
To: kernel test robot <lkp@intel.com>
Cc: Greg Kroah-Hartman <gregkh@linuxfoundation.org>,
Felipe Balbi <balbi@kernel.org>,
Thinh Nguyen <Thinh.Nguyen@synopsys.com>,
Ruslan Bilovol <ruslan.bilovol@gmail.com>,
Jerome Brunet <jbrunet@baylibre.com>,
Pavel Hofman <pavel.hofman@ivitera.com>,
llvm@lists.linux.dev, kbuild-all@lists.01.org,
linux-usb@vger.kernel.org, Ferry Toth <fntoth@gmail.com>,
Wesley Cheng <wcheng@codeaurora.org>,
Pawel Laszczak <pawell@cadence.com>
Subject: Re: [PATCH v2 2/2] usb: gadget: f_uac2: Populate SS descriptors' wBytesPerInterval
Date: Thu, 9 Sep 2021 10:40:22 -0700 [thread overview]
Message-ID: <20210909174007.GC20111@jackp-linux.qualcomm.com> (raw)
In-Reply-To: <202109100030.4qMPkRBj-lkp@intel.com>
On Fri, Sep 10, 2021 at 01:08:02AM +0800, kernel test robot wrote:
> Hi Jack,
>
> Thank you for the patch! Yet something to improve:
>
> [auto build test ERROR on usb/usb-testing]
> [also build test ERROR on peter.chen-usb/for-usb-next linus/master next-20210909]
> [cannot apply to v5.14]
> [If your patch is applied to the wrong git tree, kindly drop us a note.
> And when submitting patch, we suggest to use '--base' as documented in
> https://git-scm.com/docs/git-format-patch]
>
> url: https://github.com/0day-ci/linux/commits/Jack-Pham/usb-gadget-f_uac2-Fixes-for-SuperSpeed/20210909-162955
> base: https://git.kernel.org/pub/scm/linux/kernel/git/gregkh/usb.git usb-testing
> config: i386-randconfig-a016-20210908 (attached as .config)
> compiler: clang version 14.0.0 (https://github.com/llvm/llvm-project 261cbe98c38f8c1ee1a482fe76511110e790f58a)
> reproduce (this is a W=1 build):
> wget https://raw.githubusercontent.com/intel/lkp-tests/master/sbin/make.cross -O ~/bin/make.cross
> chmod +x ~/bin/make.cross
> # https://github.com/0day-ci/linux/commit/f262014b234c389fc10439b7c11aa10ea84270f6
> git remote add linux-review https://github.com/0day-ci/linux
> git fetch --no-tags linux-review Jack-Pham/usb-gadget-f_uac2-Fixes-for-SuperSpeed/20210909-162955
> git checkout f262014b234c389fc10439b7c11aa10ea84270f6
> # save the attached .config to linux build tree
> mkdir build_dir
> COMPILER_INSTALL_PATH=$HOME/0day COMPILER=clang make.cross O=build_dir ARCH=i386 SHELL=/bin/bash
>
> If you fix the issue, kindly add following tag as appropriate
> Reported-by: kernel test robot <lkp@intel.com>
>
> All errors (new ones prefixed by >>):
>
> >> drivers/usb/gadget/function/f_uac2.c:1181:19: error: member reference type 'struct usb_ss_ep_comp_descriptor' is not a pointer; did you mean to use '.'?
> ss_epin_desc_comp->wBytesPerInterval = ss_epin_desc->wMaxPacketSize;
> ~~~~~~~~~~~~~~~~~^~
> .
> >> drivers/usb/gadget/function/f_uac2.c:1181:53: error: member reference type 'struct usb_endpoint_descriptor' is not a pointer; did you mean to use '.'?
> ss_epin_desc_comp->wBytesPerInterval = ss_epin_desc->wMaxPacketSize;
> ~~~~~~~~~~~~^~
> .
> >> drivers/usb/gadget/function/f_uac2.c:1181:39: error: expression is not assignable
> ss_epin_desc_comp->wBytesPerInterval = ss_epin_desc->wMaxPacketSize;
> ~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~ ^
> drivers/usb/gadget/function/f_uac2.c:1182:20: error: member reference type 'struct usb_ss_ep_comp_descriptor' is not a pointer; did you mean to use '.'?
> ss_epout_desc_comp->wBytesPerInterval = ss_epout_desc->wMaxPacketSize;
> ~~~~~~~~~~~~~~~~~~^~
> .
> drivers/usb/gadget/function/f_uac2.c:1182:55: error: member reference type 'struct usb_endpoint_descriptor' is not a pointer; did you mean to use '.'?
> ss_epout_desc_comp->wBytesPerInterval = ss_epout_desc->wMaxPacketSize;
> ~~~~~~~~~~~~~^~
> .
> drivers/usb/gadget/function/f_uac2.c:1182:40: error: expression is not assignable
> ss_epout_desc_comp->wBytesPerInterval = ss_epout_desc->wMaxPacketSize;
> ~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~ ^
> 6 errors generated.
DOH! Was wondering why this was working for me but failed to realize I
was still loading an older build that had previous changes.
Sorry for the noise!! V3 coming very soon.
Jack
--
The Qualcomm Innovation Center, Inc. is a member of Code Aurora Forum,
a Linux Foundation Collaborative Project
WARNING: multiple messages have this Message-ID (diff)
From: Jack Pham <jackp@codeaurora.org>
To: kbuild-all@lists.01.org
Subject: Re: [PATCH v2 2/2] usb: gadget: f_uac2: Populate SS descriptors' wBytesPerInterval
Date: Thu, 09 Sep 2021 10:40:22 -0700 [thread overview]
Message-ID: <20210909174007.GC20111@jackp-linux.qualcomm.com> (raw)
In-Reply-To: <202109100030.4qMPkRBj-lkp@intel.com>
[-- Attachment #1: Type: text/plain, Size: 3924 bytes --]
On Fri, Sep 10, 2021 at 01:08:02AM +0800, kernel test robot wrote:
> Hi Jack,
>
> Thank you for the patch! Yet something to improve:
>
> [auto build test ERROR on usb/usb-testing]
> [also build test ERROR on peter.chen-usb/for-usb-next linus/master next-20210909]
> [cannot apply to v5.14]
> [If your patch is applied to the wrong git tree, kindly drop us a note.
> And when submitting patch, we suggest to use '--base' as documented in
> https://git-scm.com/docs/git-format-patch]
>
> url: https://github.com/0day-ci/linux/commits/Jack-Pham/usb-gadget-f_uac2-Fixes-for-SuperSpeed/20210909-162955
> base: https://git.kernel.org/pub/scm/linux/kernel/git/gregkh/usb.git usb-testing
> config: i386-randconfig-a016-20210908 (attached as .config)
> compiler: clang version 14.0.0 (https://github.com/llvm/llvm-project 261cbe98c38f8c1ee1a482fe76511110e790f58a)
> reproduce (this is a W=1 build):
> wget https://raw.githubusercontent.com/intel/lkp-tests/master/sbin/make.cross -O ~/bin/make.cross
> chmod +x ~/bin/make.cross
> # https://github.com/0day-ci/linux/commit/f262014b234c389fc10439b7c11aa10ea84270f6
> git remote add linux-review https://github.com/0day-ci/linux
> git fetch --no-tags linux-review Jack-Pham/usb-gadget-f_uac2-Fixes-for-SuperSpeed/20210909-162955
> git checkout f262014b234c389fc10439b7c11aa10ea84270f6
> # save the attached .config to linux build tree
> mkdir build_dir
> COMPILER_INSTALL_PATH=$HOME/0day COMPILER=clang make.cross O=build_dir ARCH=i386 SHELL=/bin/bash
>
> If you fix the issue, kindly add following tag as appropriate
> Reported-by: kernel test robot <lkp@intel.com>
>
> All errors (new ones prefixed by >>):
>
> >> drivers/usb/gadget/function/f_uac2.c:1181:19: error: member reference type 'struct usb_ss_ep_comp_descriptor' is not a pointer; did you mean to use '.'?
> ss_epin_desc_comp->wBytesPerInterval = ss_epin_desc->wMaxPacketSize;
> ~~~~~~~~~~~~~~~~~^~
> .
> >> drivers/usb/gadget/function/f_uac2.c:1181:53: error: member reference type 'struct usb_endpoint_descriptor' is not a pointer; did you mean to use '.'?
> ss_epin_desc_comp->wBytesPerInterval = ss_epin_desc->wMaxPacketSize;
> ~~~~~~~~~~~~^~
> .
> >> drivers/usb/gadget/function/f_uac2.c:1181:39: error: expression is not assignable
> ss_epin_desc_comp->wBytesPerInterval = ss_epin_desc->wMaxPacketSize;
> ~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~ ^
> drivers/usb/gadget/function/f_uac2.c:1182:20: error: member reference type 'struct usb_ss_ep_comp_descriptor' is not a pointer; did you mean to use '.'?
> ss_epout_desc_comp->wBytesPerInterval = ss_epout_desc->wMaxPacketSize;
> ~~~~~~~~~~~~~~~~~~^~
> .
> drivers/usb/gadget/function/f_uac2.c:1182:55: error: member reference type 'struct usb_endpoint_descriptor' is not a pointer; did you mean to use '.'?
> ss_epout_desc_comp->wBytesPerInterval = ss_epout_desc->wMaxPacketSize;
> ~~~~~~~~~~~~~^~
> .
> drivers/usb/gadget/function/f_uac2.c:1182:40: error: expression is not assignable
> ss_epout_desc_comp->wBytesPerInterval = ss_epout_desc->wMaxPacketSize;
> ~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~ ^
> 6 errors generated.
DOH! Was wondering why this was working for me but failed to realize I
was still loading an older build that had previous changes.
Sorry for the noise!! V3 coming very soon.
Jack
--
The Qualcomm Innovation Center, Inc. is a member of Code Aurora Forum,
a Linux Foundation Collaborative Project
next prev parent reply other threads:[~2021-09-09 17:40 UTC|newest]
Thread overview: 7+ messages / expand[flat|nested] mbox.gz Atom feed top
2021-09-09 8:26 [PATCH v2 0/2] usb: gadget: f_uac2: Fixes for SuperSpeed Jack Pham
2021-09-09 8:26 ` [PATCH v2 1/2] usb: gadget: f_uac2: Add missing companion descriptor for feedback EP Jack Pham
2021-09-09 8:26 ` [PATCH v2 2/2] usb: gadget: f_uac2: Populate SS descriptors' wBytesPerInterval Jack Pham
2021-09-09 17:08 ` kernel test robot
2021-09-09 17:08 ` kernel test robot
2021-09-09 17:40 ` Jack Pham [this message]
2021-09-09 17:40 ` Jack Pham
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=20210909174007.GC20111@jackp-linux.qualcomm.com \
--to=jackp@codeaurora.org \
--cc=Thinh.Nguyen@synopsys.com \
--cc=balbi@kernel.org \
--cc=fntoth@gmail.com \
--cc=gregkh@linuxfoundation.org \
--cc=jbrunet@baylibre.com \
--cc=kbuild-all@lists.01.org \
--cc=linux-usb@vger.kernel.org \
--cc=lkp@intel.com \
--cc=llvm@lists.linux.dev \
--cc=pavel.hofman@ivitera.com \
--cc=pawell@cadence.com \
--cc=ruslan.bilovol@gmail.com \
--cc=wcheng@codeaurora.org \
/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.