All of lore.kernel.org
 help / color / mirror / Atom feed
From: kernel test robot <lkp@intel.com>
To: Souradeep Chowdhury <schowdhu@codeaurora.org>,
	linux-arm-msm@vger.kernel.org, linux-usb@vger.kernel.org,
	devicetree@vger.kernel.org,
	Bryan O'Donoghue <pure.logic@nexus-software.ie>,
	Bjorn Andersson <bjorn.andersson@linaro.org>,
	Greg KH <greg@kroah.com>
Cc: kbuild-all@lists.01.org, linux-kernel@vger.kernel.org,
	ckadabi@codeaurora.org, tsoni@codeaurora.org,
	bryanh@codeaurora.org
Subject: Re: [PATCH V0 3/7] usb: dwc3: drd: Register the eud connector child node for dwc3
Date: Tue, 5 Oct 2021 20:32:01 +0800	[thread overview]
Message-ID: <202110052047.E8BoSwkq-lkp@intel.com> (raw)
In-Reply-To: <2006cf6c5662f70ca3348b683551bae4a8e4a0e2.1633343547.git.schowdhu@codeaurora.org>

[-- Attachment #1: Type: text/plain, Size: 2651 bytes --]

Hi Souradeep,

Thank you for the patch! Yet something to improve:

[auto build test ERROR on usb/usb-testing]
[also build test ERROR on robh/for-next linus/master v5.15-rc3 next-20210922]
[cannot apply to balbi-usb/testing/next]
[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/Souradeep-Chowdhury/Add-Embedded-USB-Debugger-EUD-driver/20211004-191901
base:   https://git.kernel.org/pub/scm/linux/kernel/git/gregkh/usb.git usb-testing
config: arc-allyesconfig (attached as .config)
compiler: arceb-elf-gcc (GCC) 11.2.0
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/4f53df1d9b6786f951384f59e3ffa7fed1817a2d
        git remote add linux-review https://github.com/0day-ci/linux
        git fetch --no-tags linux-review Souradeep-Chowdhury/Add-Embedded-USB-Debugger-EUD-driver/20211004-191901
        git checkout 4f53df1d9b6786f951384f59e3ffa7fed1817a2d
        # save the attached .config to linux build tree
        COMPILER_INSTALL_PATH=$HOME/0day COMPILER=gcc-11.2.0 make.cross ARCH=arc 

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/dwc3/drd.c: In function 'dwc3_register_eud':
>> drivers/usb/dwc3/drd.c:171:54: error: variable 'con_np' set but not used [-Werror=unused-but-set-variable]
     171 |         struct device_node      *np = dev->of_node, *con_np;
         |                                                      ^~~~~~
   cc1: all warnings being treated as errors


vim +/con_np +171 drivers/usb/dwc3/drd.c

   167	
   168	static int dwc3_register_eud(struct dwc3 *dwc)
   169	{
   170		struct device           *dev = dwc->dev;
 > 171		struct device_node      *np = dev->of_node, *con_np;
   172		int                     ret;
   173	
   174		con_np = of_get_child_by_name(np, "eud_usb_connector");
   175		if (!np) {
   176			dev_dbg(dev, "no usb_connector child node specified\n");
   177			return 0;
   178		}
   179	
   180		ret = of_platform_populate(np, NULL, NULL, dev);
   181		if (ret) {
   182			dev_err(dev, "failed to register usb_connector - %d\n", ret);
   183			return ret;
   184		}
   185	
   186		return 0;
   187	}
   188	

---
0-DAY CI Kernel Test Service, Intel Corporation
https://lists.01.org/hyperkitty/list/kbuild-all@lists.01.org

[-- Attachment #2: .config.gz --]
[-- Type: application/gzip, Size: 69189 bytes --]

WARNING: multiple messages have this Message-ID (diff)
From: kernel test robot <lkp@intel.com>
To: kbuild-all@lists.01.org
Subject: Re: [PATCH V0 3/7] usb: dwc3: drd: Register the eud connector child node for dwc3
Date: Tue, 05 Oct 2021 20:32:01 +0800	[thread overview]
Message-ID: <202110052047.E8BoSwkq-lkp@intel.com> (raw)
In-Reply-To: <2006cf6c5662f70ca3348b683551bae4a8e4a0e2.1633343547.git.schowdhu@codeaurora.org>

[-- Attachment #1: Type: text/plain, Size: 2718 bytes --]

Hi Souradeep,

Thank you for the patch! Yet something to improve:

[auto build test ERROR on usb/usb-testing]
[also build test ERROR on robh/for-next linus/master v5.15-rc3 next-20210922]
[cannot apply to balbi-usb/testing/next]
[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/Souradeep-Chowdhury/Add-Embedded-USB-Debugger-EUD-driver/20211004-191901
base:   https://git.kernel.org/pub/scm/linux/kernel/git/gregkh/usb.git usb-testing
config: arc-allyesconfig (attached as .config)
compiler: arceb-elf-gcc (GCC) 11.2.0
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/4f53df1d9b6786f951384f59e3ffa7fed1817a2d
        git remote add linux-review https://github.com/0day-ci/linux
        git fetch --no-tags linux-review Souradeep-Chowdhury/Add-Embedded-USB-Debugger-EUD-driver/20211004-191901
        git checkout 4f53df1d9b6786f951384f59e3ffa7fed1817a2d
        # save the attached .config to linux build tree
        COMPILER_INSTALL_PATH=$HOME/0day COMPILER=gcc-11.2.0 make.cross ARCH=arc 

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/dwc3/drd.c: In function 'dwc3_register_eud':
>> drivers/usb/dwc3/drd.c:171:54: error: variable 'con_np' set but not used [-Werror=unused-but-set-variable]
     171 |         struct device_node      *np = dev->of_node, *con_np;
         |                                                      ^~~~~~
   cc1: all warnings being treated as errors


vim +/con_np +171 drivers/usb/dwc3/drd.c

   167	
   168	static int dwc3_register_eud(struct dwc3 *dwc)
   169	{
   170		struct device           *dev = dwc->dev;
 > 171		struct device_node      *np = dev->of_node, *con_np;
   172		int                     ret;
   173	
   174		con_np = of_get_child_by_name(np, "eud_usb_connector");
   175		if (!np) {
   176			dev_dbg(dev, "no usb_connector child node specified\n");
   177			return 0;
   178		}
   179	
   180		ret = of_platform_populate(np, NULL, NULL, dev);
   181		if (ret) {
   182			dev_err(dev, "failed to register usb_connector - %d\n", ret);
   183			return ret;
   184		}
   185	
   186		return 0;
   187	}
   188	

---
0-DAY CI Kernel Test Service, Intel Corporation
https://lists.01.org/hyperkitty/list/kbuild-all(a)lists.01.org

[-- Attachment #2: config.gz --]
[-- Type: application/gzip, Size: 69189 bytes --]

  parent reply	other threads:[~2021-10-05 12:32 UTC|newest]

Thread overview: 28+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2021-10-04 11:16 [PATCH V0 0/7] Add Embedded USB Debugger (EUD) driver Souradeep Chowdhury
2021-10-04 11:16 ` [PATCH V0 1/7] dt-bindings: connector: Add property for eud type c connector Souradeep Chowdhury
2021-10-04 16:37   ` Rob Herring
2021-10-05 13:11     ` schowdhu
2021-10-05 16:37       ` Bjorn Andersson
2021-10-07  9:25         ` schowdhu
2021-10-12  3:22           ` Bjorn Andersson
2021-11-02  9:20             ` schowdhu
2021-10-04 11:16 ` [PATCH V0 2/7] dt-bindings: usb: dwc3: Update dwc3 properties for EUD connector Souradeep Chowdhury
2021-10-04 15:50   ` Bjorn Andersson
2021-10-05 12:57     ` schowdhu
2021-10-04 11:16 ` [PATCH V0 3/7] usb: dwc3: drd: Register the eud connector child node for dwc3 Souradeep Chowdhury
2021-10-05  4:42   ` kernel test robot
2021-10-05  4:42     ` kernel test robot
2021-10-05 12:32   ` kernel test robot [this message]
2021-10-05 12:32     ` kernel test robot
2021-10-04 11:16 ` [PATCH V0 4/7] usb: common: eud: Added the driver support for Embedded USB Debugger(EUD) Souradeep Chowdhury
2021-10-04 11:31   ` Greg KH
2021-10-05 12:54     ` schowdhu
2021-10-04 16:42   ` Bjorn Andersson
2021-10-05  7:14     ` Joe Perches
2021-10-05 13:58       ` schowdhu
2021-10-05 13:01     ` schowdhu
2021-10-04 20:08   ` Randy Dunlap
2021-10-05 13:57     ` schowdhu
2021-10-04 11:16 ` [PATCH V0 5/7] arm64: dts: qcom: sc7280: Add EUD connector node Souradeep Chowdhury
2021-10-04 11:16 ` [PATCH V0 6/7] arm64: dts: qcom: sc7280: Set the default dr_mode for usb2 Souradeep Chowdhury
2021-10-04 11:16 ` [PATCH V0 7/7] MAINTAINERS: Add maintainer entry for EUD Souradeep Chowdhury

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=202110052047.E8BoSwkq-lkp@intel.com \
    --to=lkp@intel.com \
    --cc=bjorn.andersson@linaro.org \
    --cc=bryanh@codeaurora.org \
    --cc=ckadabi@codeaurora.org \
    --cc=devicetree@vger.kernel.org \
    --cc=greg@kroah.com \
    --cc=kbuild-all@lists.01.org \
    --cc=linux-arm-msm@vger.kernel.org \
    --cc=linux-kernel@vger.kernel.org \
    --cc=linux-usb@vger.kernel.org \
    --cc=pure.logic@nexus-software.ie \
    --cc=schowdhu@codeaurora.org \
    --cc=tsoni@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.