* [PATCH] fpga: xilinx-pr-decoupler: Fix unused xlnx_pr_decoupler_of_match warning for !CONFIG_OF
@ 2023-10-12 19:21 Rob Herring
2023-10-13 9:36 ` Xu Yilun
2023-10-20 0:37 ` kernel test robot
0 siblings, 2 replies; 6+ messages in thread
From: Rob Herring @ 2023-10-12 19:21 UTC (permalink / raw)
To: Moritz Fischer, Wu Hao, Xu Yilun, Tom Rix, Michal Simek
Cc: kernel test robot, linux-fpga, linux-arm-kernel, linux-kernel
Commit 8c966aadcc02 ("fpga: Use device_get_match_data()") dropped the
unconditional use of xlnx_pr_decoupler_of_match resulting in this
warning:
drivers/fpga/xilinx-pr-decoupler.c:94:34: warning: unused variable 'xlnx_pr_decoupler_of_match' [-Wunused-const-variable]
The fix is to drop of_match_ptr() which is not necessary because DT is
always used for this driver.
Reported-by: kernel test robot <lkp@intel.com>
Closes: https://lore.kernel.org/oe-kbuild-all/202310100247.Y7BFcalX-lkp@intel.com/
Signed-off-by: Rob Herring <robh@kernel.org>
---
drivers/fpga/xilinx-pr-decoupler.c | 2 +-
1 file changed, 1 insertion(+), 1 deletion(-)
diff --git a/drivers/fpga/xilinx-pr-decoupler.c b/drivers/fpga/xilinx-pr-decoupler.c
index 0c012d0f616d..68835896f180 100644
--- a/drivers/fpga/xilinx-pr-decoupler.c
+++ b/drivers/fpga/xilinx-pr-decoupler.c
@@ -167,7 +167,7 @@ static struct platform_driver xlnx_pr_decoupler_driver = {
.remove = xlnx_pr_decoupler_remove,
.driver = {
.name = "xlnx_pr_decoupler",
- .of_match_table = of_match_ptr(xlnx_pr_decoupler_of_match),
+ .of_match_table = xlnx_pr_decoupler_of_match,
},
};
--
2.42.0
^ permalink raw reply related [flat|nested] 6+ messages in thread
* Re: [PATCH] fpga: xilinx-pr-decoupler: Fix unused xlnx_pr_decoupler_of_match warning for !CONFIG_OF
2023-10-12 19:21 [PATCH] fpga: xilinx-pr-decoupler: Fix unused xlnx_pr_decoupler_of_match warning for !CONFIG_OF Rob Herring
@ 2023-10-13 9:36 ` Xu Yilun
2023-10-13 13:15 ` Rob Herring
2023-10-20 0:37 ` kernel test robot
1 sibling, 1 reply; 6+ messages in thread
From: Xu Yilun @ 2023-10-13 9:36 UTC (permalink / raw)
To: Rob Herring
Cc: Moritz Fischer, Wu Hao, Xu Yilun, Tom Rix, Michal Simek,
kernel test robot, linux-fpga, linux-arm-kernel, linux-kernel
On Thu, Oct 12, 2023 at 02:21:48PM -0500, Rob Herring wrote:
> Commit 8c966aadcc02 ("fpga: Use device_get_match_data()") dropped the
Hi rob:
Unfortunately I re-applied Commit 8c966aadcc02 and the previous commit
id is lost.
Since the 2 patches are not upstreamed yet, could I just merge them into
one?
Thanks,
Yilun
> unconditional use of xlnx_pr_decoupler_of_match resulting in this
> warning:
>
> drivers/fpga/xilinx-pr-decoupler.c:94:34: warning: unused variable 'xlnx_pr_decoupler_of_match' [-Wunused-const-variable]
>
> The fix is to drop of_match_ptr() which is not necessary because DT is
> always used for this driver.
>
> Reported-by: kernel test robot <lkp@intel.com>
> Closes: https://lore.kernel.org/oe-kbuild-all/202310100247.Y7BFcalX-lkp@intel.com/
> Signed-off-by: Rob Herring <robh@kernel.org>
> ---
> drivers/fpga/xilinx-pr-decoupler.c | 2 +-
> 1 file changed, 1 insertion(+), 1 deletion(-)
>
> diff --git a/drivers/fpga/xilinx-pr-decoupler.c b/drivers/fpga/xilinx-pr-decoupler.c
> index 0c012d0f616d..68835896f180 100644
> --- a/drivers/fpga/xilinx-pr-decoupler.c
> +++ b/drivers/fpga/xilinx-pr-decoupler.c
> @@ -167,7 +167,7 @@ static struct platform_driver xlnx_pr_decoupler_driver = {
> .remove = xlnx_pr_decoupler_remove,
> .driver = {
> .name = "xlnx_pr_decoupler",
> - .of_match_table = of_match_ptr(xlnx_pr_decoupler_of_match),
> + .of_match_table = xlnx_pr_decoupler_of_match,
> },
> };
>
> --
> 2.42.0
>
^ permalink raw reply [flat|nested] 6+ messages in thread
* Re: [PATCH] fpga: xilinx-pr-decoupler: Fix unused xlnx_pr_decoupler_of_match warning for !CONFIG_OF
2023-10-13 9:36 ` Xu Yilun
@ 2023-10-13 13:15 ` Rob Herring
2023-10-13 18:36 ` Rob Herring
0 siblings, 1 reply; 6+ messages in thread
From: Rob Herring @ 2023-10-13 13:15 UTC (permalink / raw)
To: Xu Yilun
Cc: Moritz Fischer, Wu Hao, Xu Yilun, Tom Rix, Michal Simek,
kernel test robot, linux-fpga, linux-arm-kernel, linux-kernel
On Fri, Oct 13, 2023 at 4:37 AM Xu Yilun <yilun.xu@linux.intel.com> wrote:
>
> On Thu, Oct 12, 2023 at 02:21:48PM -0500, Rob Herring wrote:
> > Commit 8c966aadcc02 ("fpga: Use device_get_match_data()") dropped the
>
> Hi rob:
>
> Unfortunately I re-applied Commit 8c966aadcc02 and the previous commit
> id is lost.
>
> Since the 2 patches are not upstreamed yet, could I just merge them into
> one?
Yes, that's fine.
Rob
>
> Thanks,
> Yilun
>
> > unconditional use of xlnx_pr_decoupler_of_match resulting in this
> > warning:
> >
> > drivers/fpga/xilinx-pr-decoupler.c:94:34: warning: unused variable 'xlnx_pr_decoupler_of_match' [-Wunused-const-variable]
> >
> > The fix is to drop of_match_ptr() which is not necessary because DT is
> > always used for this driver.
> >
> > Reported-by: kernel test robot <lkp@intel.com>
> > Closes: https://lore.kernel.org/oe-kbuild-all/202310100247.Y7BFcalX-lkp@intel.com/
> > Signed-off-by: Rob Herring <robh@kernel.org>
> > ---
> > drivers/fpga/xilinx-pr-decoupler.c | 2 +-
> > 1 file changed, 1 insertion(+), 1 deletion(-)
> >
> > diff --git a/drivers/fpga/xilinx-pr-decoupler.c b/drivers/fpga/xilinx-pr-decoupler.c
> > index 0c012d0f616d..68835896f180 100644
> > --- a/drivers/fpga/xilinx-pr-decoupler.c
> > +++ b/drivers/fpga/xilinx-pr-decoupler.c
> > @@ -167,7 +167,7 @@ static struct platform_driver xlnx_pr_decoupler_driver = {
> > .remove = xlnx_pr_decoupler_remove,
> > .driver = {
> > .name = "xlnx_pr_decoupler",
> > - .of_match_table = of_match_ptr(xlnx_pr_decoupler_of_match),
> > + .of_match_table = xlnx_pr_decoupler_of_match,
> > },
> > };
> >
> > --
> > 2.42.0
> >
^ permalink raw reply [flat|nested] 6+ messages in thread
* Re: [PATCH] fpga: xilinx-pr-decoupler: Fix unused xlnx_pr_decoupler_of_match warning for !CONFIG_OF
2023-10-13 13:15 ` Rob Herring
@ 2023-10-13 18:36 ` Rob Herring
2023-10-16 3:17 ` Xu Yilun
0 siblings, 1 reply; 6+ messages in thread
From: Rob Herring @ 2023-10-13 18:36 UTC (permalink / raw)
To: Xu Yilun
Cc: Moritz Fischer, Wu Hao, Xu Yilun, Tom Rix, Michal Simek,
kernel test robot, linux-fpga, linux-arm-kernel, linux-kernel
On Fri, Oct 13, 2023 at 8:15 AM Rob Herring <robh@kernel.org> wrote:
>
> On Fri, Oct 13, 2023 at 4:37 AM Xu Yilun <yilun.xu@linux.intel.com> wrote:
> >
> > On Thu, Oct 12, 2023 at 02:21:48PM -0500, Rob Herring wrote:
> > > Commit 8c966aadcc02 ("fpga: Use device_get_match_data()") dropped the
> >
> > Hi rob:
> >
> > Unfortunately I re-applied Commit 8c966aadcc02 and the previous commit
> > id is lost.
> >
> > Since the 2 patches are not upstreamed yet, could I just merge them into
> > one?
>
> Yes, that's fine.
Looks like altera-ps-spi.c also needs the same fix. Do you mind making
the same change there?
Rob
^ permalink raw reply [flat|nested] 6+ messages in thread
* Re: [PATCH] fpga: xilinx-pr-decoupler: Fix unused xlnx_pr_decoupler_of_match warning for !CONFIG_OF
2023-10-13 18:36 ` Rob Herring
@ 2023-10-16 3:17 ` Xu Yilun
0 siblings, 0 replies; 6+ messages in thread
From: Xu Yilun @ 2023-10-16 3:17 UTC (permalink / raw)
To: Rob Herring
Cc: Moritz Fischer, Wu Hao, Xu Yilun, Tom Rix, Michal Simek,
kernel test robot, linux-fpga, linux-arm-kernel, linux-kernel
On Fri, Oct 13, 2023 at 01:36:28PM -0500, Rob Herring wrote:
> On Fri, Oct 13, 2023 at 8:15 AM Rob Herring <robh@kernel.org> wrote:
> >
> > On Fri, Oct 13, 2023 at 4:37 AM Xu Yilun <yilun.xu@linux.intel.com> wrote:
> > >
> > > On Thu, Oct 12, 2023 at 02:21:48PM -0500, Rob Herring wrote:
> > > > Commit 8c966aadcc02 ("fpga: Use device_get_match_data()") dropped the
> > >
> > > Hi rob:
> > >
> > > Unfortunately I re-applied Commit 8c966aadcc02 and the previous commit
> > > id is lost.
> > >
> > > Since the 2 patches are not upstreamed yet, could I just merge them into
> > > one?
> >
> > Yes, that's fine.
>
> Looks like altera-ps-spi.c also needs the same fix. Do you mind making
> the same change there?
Fixed & applied.
Thanks,
Yilun
>
> Rob
^ permalink raw reply [flat|nested] 6+ messages in thread
* Re: [PATCH] fpga: xilinx-pr-decoupler: Fix unused xlnx_pr_decoupler_of_match warning for !CONFIG_OF
2023-10-12 19:21 [PATCH] fpga: xilinx-pr-decoupler: Fix unused xlnx_pr_decoupler_of_match warning for !CONFIG_OF Rob Herring
2023-10-13 9:36 ` Xu Yilun
@ 2023-10-20 0:37 ` kernel test robot
1 sibling, 0 replies; 6+ messages in thread
From: kernel test robot @ 2023-10-20 0:37 UTC (permalink / raw)
To: Rob Herring, Moritz Fischer, Wu Hao, Xu Yilun, Tom Rix, Michal Simek
Cc: oe-kbuild-all, kernel test robot, linux-fpga, linux-arm-kernel,
linux-kernel
Hi Rob,
kernel test robot noticed the following build errors:
[auto build test ERROR on xilinx-xlnx/master]
[also build test ERROR on linus/master v6.6-rc6]
[cannot apply to next-20231019]
[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#_base_tree_information]
url: https://github.com/intel-lab-lkp/linux/commits/Rob-Herring/fpga-xilinx-pr-decoupler-Fix-unused-xlnx_pr_decoupler_of_match-warning-for-CONFIG_OF/20231017-112523
base: https://github.com/Xilinx/linux-xlnx master
patch link: https://lore.kernel.org/r/20231012192149.1546368-1-robh%40kernel.org
patch subject: [PATCH] fpga: xilinx-pr-decoupler: Fix unused xlnx_pr_decoupler_of_match warning for !CONFIG_OF
config: parisc-randconfig-002-20231019 (https://download.01.org/0day-ci/archive/20231020/202310200826.9spStghB-lkp@intel.com/config)
compiler: hppa-linux-gcc (GCC) 13.2.0
reproduce (this is a W=1 build): (https://download.01.org/0day-ci/archive/20231020/202310200826.9spStghB-lkp@intel.com/reproduce)
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://lore.kernel.org/oe-kbuild-all/202310200826.9spStghB-lkp@intel.com/
All errors (new ones prefixed by >>):
>> drivers/fpga/xilinx-pr-decoupler.c:179:35: error: 'xlnx_pr_decoupler_of_match' undeclared here (not in a function); did you mean 'xlnx_pr_decoupler_data'?
179 | .of_match_table = xlnx_pr_decoupler_of_match,
| ^~~~~~~~~~~~~~~~~~~~~~~~~~
| xlnx_pr_decoupler_data
vim +179 drivers/fpga/xilinx-pr-decoupler.c
173
174 static struct platform_driver xlnx_pr_decoupler_driver = {
175 .probe = xlnx_pr_decoupler_probe,
176 .remove = xlnx_pr_decoupler_remove,
177 .driver = {
178 .name = "xlnx_pr_decoupler",
> 179 .of_match_table = xlnx_pr_decoupler_of_match,
180 },
181 };
182
--
0-DAY CI Kernel Test Service
https://github.com/intel/lkp-tests/wiki
^ permalink raw reply [flat|nested] 6+ messages in thread
end of thread, other threads:[~2023-10-20 0:37 UTC | newest]
Thread overview: 6+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2023-10-12 19:21 [PATCH] fpga: xilinx-pr-decoupler: Fix unused xlnx_pr_decoupler_of_match warning for !CONFIG_OF Rob Herring
2023-10-13 9:36 ` Xu Yilun
2023-10-13 13:15 ` Rob Herring
2023-10-13 18:36 ` Rob Herring
2023-10-16 3:17 ` Xu Yilun
2023-10-20 0:37 ` kernel test robot
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for NNTP newsgroup(s).