mirror of https://lore.kernel.org/lkml/
 help / color / mirror / Atom feed
From: Thinh Nguyen <Thinh.Nguyen@synopsys.com>
To: Krishna Kurapati PSSNV <krishna.kurapati@oss.qualcomm.com>
Cc: kernel test robot <lkp@intel.com>,
	Thinh Nguyen <Thinh.Nguyen@synopsys.com>,
	Greg Kroah-Hartman <gregkh@linuxfoundation.org>,
	"oe-kbuild-all@lists.linux.dev" <oe-kbuild-all@lists.linux.dev>,
	"linux-arm-msm@vger.kernel.org" <linux-arm-msm@vger.kernel.org>,
	"linux-usb@vger.kernel.org" <linux-usb@vger.kernel.org>,
	"linux-kernel@vger.kernel.org" <linux-kernel@vger.kernel.org>,
	"dmitry.baryshkov@oss.qualcomm.com"
	<dmitry.baryshkov@oss.qualcomm.com>,
	"konrad.dybcio@oss.qualcomm.com" <konrad.dybcio@oss.qualcomm.com>
Subject: Re: [PATCH v5 2/2] usb: dwc3: qcom: Implement glue callbacks to facilitate runtime suspend
Date: Thu, 16 Oct 2025 22:35:51 +0000	[thread overview]
Message-ID: <20251016223548.7nkkkew5gbarsk4m@synopsys.com> (raw)
In-Reply-To: <8d8fe2c8-6b56-43a6-8a28-5150514da691@oss.qualcomm.com>

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@intel.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

      reply	other threads:[~2025-10-16 22:36 UTC|newest]

Thread overview: 6+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2025-09-07 18:14 [PATCH v5 0/2] usb: dwc3: Modify role-switching QC drd usb controllers Krishna Kurapati
2025-09-07 18:14 ` [PATCH v5 1/2] usb: dwc3: core: Introduce glue callbacks for flattened implementations Krishna Kurapati
2025-09-07 18:14 ` [PATCH v5 2/2] usb: dwc3: qcom: Implement glue callbacks to facilitate runtime suspend Krishna Kurapati
2025-09-08 18:25   ` kernel test robot
2025-09-09  4:21     ` Krishna Kurapati PSSNV
2025-10-16 22:35       ` Thinh Nguyen [this message]

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=20251016223548.7nkkkew5gbarsk4m@synopsys.com \
    --to=thinh.nguyen@synopsys.com \
    --cc=dmitry.baryshkov@oss.qualcomm.com \
    --cc=gregkh@linuxfoundation.org \
    --cc=konrad.dybcio@oss.qualcomm.com \
    --cc=krishna.kurapati@oss.qualcomm.com \
    --cc=linux-arm-msm@vger.kernel.org \
    --cc=linux-kernel@vger.kernel.org \
    --cc=linux-usb@vger.kernel.org \
    --cc=lkp@intel.com \
    --cc=oe-kbuild-all@lists.linux.dev \
    /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 a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox

all inboxes | Powered by JetHome®