diff mbox series

[1/2] media: bcm2835-unicam: Drop usage of of_match_ptr()

Message ID 20240430213633.23767-1-laurent.pinchart@ideasonboard.com (mailing list archive)
State New, archived
Headers show
Series media: bcm2835-unicam: Fix compilation errors and warnings | expand

Commit Message

Laurent Pinchart April 30, 2024, 9:36 p.m. UTC
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(-)

Comments

Ricardo Ribalda Delgado May 1, 2024, 11:03 a.m. UTC | #1
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 mbox series

Patch

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,
 	},
 };