[<prev] [next>] [<thread-prev] [day] [month] [year] [list]
Message-ID: <20251016223548.7nkkkew5gbarsk4m@synopsys.com>
Date: Thu, 16 Oct 2025 22:35:51 +0000
From: Thinh Nguyen <Thinh.Nguyen@...opsys.com>
To: Krishna Kurapati PSSNV <krishna.kurapati@....qualcomm.com>
CC: kernel test robot <lkp@...el.com>,
Thinh Nguyen <Thinh.Nguyen@...opsys.com>,
Greg Kroah-Hartman <gregkh@...uxfoundation.org>,
"oe-kbuild-all@...ts.linux.dev" <oe-kbuild-all@...ts.linux.dev>,
"linux-arm-msm@...r.kernel.org" <linux-arm-msm@...r.kernel.org>,
"linux-usb@...r.kernel.org" <linux-usb@...r.kernel.org>,
"linux-kernel@...r.kernel.org" <linux-kernel@...r.kernel.org>,
"dmitry.baryshkov@....qualcomm.com" <dmitry.baryshkov@....qualcomm.com>,
"konrad.dybcio@....qualcomm.com" <konrad.dybcio@....qualcomm.com>
Subject: Re: [PATCH v5 2/2] usb: dwc3: qcom: Implement glue callbacks to
facilitate runtime suspend
Hi Krishna,
On Tue, Sep 09, 2025, Krishna Kurapati PSSNV wrote:
>
>
> On 9/8/2025 11:55 PM, kernel test robot wrote:
> > Hi Krishna,
> >
> > kernel test robot noticed the following build warnings:
> >
> > [auto build test WARNING on usb/usb-testing]
> > [also build test WARNING on usb/usb-next next-20250908]
> > [cannot apply to usb/usb-linus linus/master v6.17-rc5]
> > [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://urldefense.com/v3/__https://git-scm.com/docs/git-format-patch*_base_tree_information__;Iw!!A4F2R9G_pg!ZsymjyqT3rb5VegxXoBFsTXfzChL0PJ6FFSYeo9ZESr4QEZXhFEanRAKzU86TMQFKs5dr0rLwM0RxWzIqx6HtUYhdKILvuCUs069TQ$ ]
> >
> > url: https://urldefense.com/v3/__https://github.com/intel-lab-lkp/linux/commits/Krishna-Kurapati/usb-dwc3-core-Introduce-glue-callbacks-for-flattened-implementations/20250908-021710__;!!A4F2R9G_pg!ZsymjyqT3rb5VegxXoBFsTXfzChL0PJ6FFSYeo9ZESr4QEZXhFEanRAKzU86TMQFKs5dr0rLwM0RxWzIqx6HtUYhdKILvuBH_vZCtA$
> > base: https://urldefense.com/v3/__https://git.kernel.org/pub/scm/linux/kernel/git/gregkh/usb.git__;!!A4F2R9G_pg!ZsymjyqT3rb5VegxXoBFsTXfzChL0PJ6FFSYeo9ZESr4QEZXhFEanRAKzU86TMQFKs5dr0rLwM0RxWzIqx6HtUYhdKILvuBJOue9Yg$
> > usb-testing
> > patch link: https://urldefense.com/v3/__https://lore.kernel.org/r/20250907181412.2174616-3-krishna.kurapati*40oss.qualcomm.com__;JQ!!A4F2R9G_pg!ZsymjyqT3rb5VegxXoBFsTXfzChL0PJ6FFSYeo9ZESr4QEZXhFEanRAKzU86TMQFKs5dr0rLwM0RxWzIqx6HtUYhdKILvuAC-pVfPg$
> > patch subject: [PATCH v5 2/2] usb: dwc3: qcom: Implement glue callbacks
> > to facilitate runtime suspend
> > config: hexagon-randconfig-r132-20250908 (https://urldefense.com/v3/__https://download.01.org/0day-ci/archive/20250909/202509090115.dVhd78BE-lkp@intel.com/config__;!!A4F2R9G_pg!ZsymjyqT3rb5VegxXoBFsTXfzChL0PJ6FFSYeo9ZESr4QEZXhFEanRAKzU86TMQFKs5dr0rLwM0RxWzIqx6HtUYhdKILvuCR0LOHOQ$ )
> > compiler: clang version 20.1.8 (https://urldefense.com/v3/__https://github.com/llvm/llvm-project__;!!A4F2R9G_pg!ZsymjyqT3rb5VegxXoBFsTXfzChL0PJ6FFSYeo9ZESr4QEZXhFEanRAKzU86TMQFKs5dr0rLwM0RxWzIqx6HtUYhdKILvuAfav-YqA$ 87f0227cb60147a26a1eeb4fb06e3b505e9c7261)
> > reproduce: (https://urldefense.com/v3/__https://download.01.org/0day-ci/archive/20250909/202509090115.dVhd78BE-lkp@intel.com/reproduce__;!!A4F2R9G_pg!ZsymjyqT3rb5VegxXoBFsTXfzChL0PJ6FFSYeo9ZESr4QEZXhFEanRAKzU86TMQFKs5dr0rLwM0RxWzIqx6HtUYhdKILvuBCFwA9Uw$ )
> >
> > If you fix the issue in a separate patch/commit (i.e. not just a new version of
> > the same patch/commit), kindly add following tags
> > | Reported-by: kernel test robot <lkp@...el.com>
> > | Closes: https://urldefense.com/v3/__https://lore.kernel.org/oe-kbuild-all/202509090115.dVhd78BE-lkp@intel.com/__;!!A4F2R9G_pg!ZsymjyqT3rb5VegxXoBFsTXfzChL0PJ6FFSYeo9ZESr4QEZXhFEanRAKzU86TMQFKs5dr0rLwM0RxWzIqx6HtUYhdKILvuAf2fxkAA$
> >
> > sparse warnings: (new ones prefixed by >>)
> > > > drivers/usb/dwc3/dwc3-qcom.c:605:22: sparse: sparse: symbol 'dwc3_qcom_glue_ops' was not declared. Should it be static?
> >
> > vim +/dwc3_qcom_glue_ops +605 drivers/usb/dwc3/dwc3-qcom.c
> >
> > 604
> > > 605 struct dwc3_glue_ops dwc3_qcom_glue_ops = {
> > 606 .pre_set_role = dwc3_qcom_set_role_notifier,
> > 607 .pre_run_stop = dwc3_qcom_run_stop_notifier,
> > 608 };
> > 609
> >
>
> I see that the usb-linus is missing the following patch:
>
> https://urldefense.com/v3/__https://lore.kernel.org/all/20250729092708.3628187-1-krishna.kurapati@oss.qualcomm.com/__;!!A4F2R9G_pg!ZsymjyqT3rb5VegxXoBFsTXfzChL0PJ6FFSYeo9ZESr4QEZXhFEanRAKzU86TMQFKs5dr0rLwM0RxWzIqx6HtUYhdKILvuCb-WGjNQ$
>
Just checking, as noted by the test bot, make sure to specify the base
commit for your patch creation so to help the bot with these kinds of
issue. You may need to resubmit this. Please rebase against Greg's
usb-testing branch.
Thanks,
Thinh
Powered by blists - more mailing lists