| Message ID | 20240430213633.23767-1-laurent.pinchart@ideasonboard.com (mailing list archive) |
|---|---|
| State | New |
| Headers | show |
| Series | media: bcm2835-unicam: Fix compilation errors and warnings | expand |
Hi Laurent This is a dupe of: https://patchwork.linuxtv.org/project/linux-media/patch/20240430-fix-ipu6-v1-1-9b31fbbce6e4@chromium.org/ regards! On Tue, Apr 30, 2024 at 11:39 PM Laurent Pinchart <laurent.pinchart@ideasonboard.com> wrote: > > Using of_match_ptr() to set the .of_match_table field of the device > driver results in the unicam_of_match table being unused on non-OF > platforms, causing a compilation warning. Fix it by dropping usage of > of_match_ptr(), which can be done because the .of_match_table field is > part of the device_driver structure regardless of whether or not > CONFIG_OF is selected. > > Signed-off-by: Laurent Pinchart <laurent.pinchart@ideasonboard.com> > Reported-by: kernel test robot <lkp@intel.com> > Closes: https://lore.kernel.org/oe-kbuild-all/202404302324.8aTC84kE-lkp@intel.com/ > --- > drivers/media/platform/broadcom/bcm2835-unicam.c | 2 +- > 1 file changed, 1 insertion(+), 1 deletion(-) > > diff --git a/drivers/media/platform/broadcom/bcm2835-unicam.c b/drivers/media/platform/broadcom/bcm2835-unicam.c > index bd2bbb53070e..c590e26fe2cf 100644 > --- a/drivers/media/platform/broadcom/bcm2835-unicam.c > +++ b/drivers/media/platform/broadcom/bcm2835-unicam.c > @@ -2733,7 +2733,7 @@ static struct platform_driver unicam_driver = { > .driver = { > .name = UNICAM_MODULE_NAME, > .pm = pm_ptr(&unicam_pm_ops), > - .of_match_table = of_match_ptr(unicam_of_match), > + .of_match_table = unicam_of_match, > }, > }; > > -- > Regards, > > Laurent Pinchart > >
diff --git a/drivers/media/platform/broadcom/bcm2835-unicam.c b/drivers/media/platform/broadcom/bcm2835-unicam.c index bd2bbb53070e..c590e26fe2cf 100644 --- a/drivers/media/platform/broadcom/bcm2835-unicam.c +++ b/drivers/media/platform/broadcom/bcm2835-unicam.c @@ -2733,7 +2733,7 @@ static struct platform_driver unicam_driver = { .driver = { .name = UNICAM_MODULE_NAME, .pm = pm_ptr(&unicam_pm_ops), - .of_match_table = of_match_ptr(unicam_of_match), + .of_match_table = unicam_of_match, }, };
Using of_match_ptr() to set the .of_match_table field of the device driver results in the unicam_of_match table being unused on non-OF platforms, causing a compilation warning. Fix it by dropping usage of of_match_ptr(), which can be done because the .of_match_table field is part of the device_driver structure regardless of whether or not CONFIG_OF is selected. Signed-off-by: Laurent Pinchart <laurent.pinchart@ideasonboard.com> Reported-by: kernel test robot <lkp@intel.com> Closes: https://lore.kernel.org/oe-kbuild-all/202404302324.8aTC84kE-lkp@intel.com/ --- drivers/media/platform/broadcom/bcm2835-unicam.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-)