diff mbox series

ASoC: imx-audmux: Silence a clang warning

Message ID 20220526010543.1164793-1-festevam@gmail.com (mailing list archive)
State Accepted
Commit 2f4a8171da06609bb6a063630ed546ee3d93dad7
Headers show
Series ASoC: imx-audmux: Silence a clang warning | expand

Commit Message

Fabio Estevam May 26, 2022, 1:05 a.m. UTC
Change the of_device_get_match_data() cast to (uintptr_t)
to silence the following clang warning:

sound/soc/fsl/imx-audmux.c:301:16: warning: cast to smaller integer type 'enum imx_audmux_type' from 'const void *' [-Wvoid-pointer-to-enum-cast]

Reported-by: kernel test robot <lkp@intel.com>
Fixes: 6a8b8b582db1 ("ASoC: imx-audmux: Remove unused .id_table")
Signed-off-by: Fabio Estevam <festevam@gmail.com>
---
 sound/soc/fsl/imx-audmux.c | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

Comments

Shengjiu Wang May 27, 2022, 8:15 a.m. UTC | #1
On Thu, May 26, 2022 at 9:05 AM Fabio Estevam <festevam@gmail.com> wrote:

> Change the of_device_get_match_data() cast to (uintptr_t)
> to silence the following clang warning:
>
> sound/soc/fsl/imx-audmux.c:301:16: warning: cast to smaller integer type
> 'enum imx_audmux_type' from 'const void *' [-Wvoid-pointer-to-enum-cast]
>
> Reported-by: kernel test robot <lkp@intel.com>
> Fixes: 6a8b8b582db1 ("ASoC: imx-audmux: Remove unused .id_table")
> Signed-off-by: Fabio Estevam <festevam@gmail.com>
>

Acked-by: Shengjiu Wang <shengjiu.wang@gmail.com>

best regards
wang shengjiu


> ---
>  sound/soc/fsl/imx-audmux.c | 2 +-
>  1 file changed, 1 insertion(+), 1 deletion(-)
>
> diff --git a/sound/soc/fsl/imx-audmux.c b/sound/soc/fsl/imx-audmux.c
> index dfa05d40b276..a8e5e0f57faf 100644
> --- a/sound/soc/fsl/imx-audmux.c
> +++ b/sound/soc/fsl/imx-audmux.c
> @@ -298,7 +298,7 @@ static int imx_audmux_probe(struct platform_device
> *pdev)
>                 audmux_clk = NULL;
>         }
>
> -       audmux_type = (enum
> imx_audmux_type)of_device_get_match_data(&pdev->dev);
> +       audmux_type = (uintptr_t)of_device_get_match_data(&pdev->dev);
>
>         switch (audmux_type) {
>         case IMX31_AUDMUX:
> --
> 2.25.1
>
>
Mark Brown June 7, 2022, 10:45 a.m. UTC | #2
On Wed, 25 May 2022 22:05:43 -0300, Fabio Estevam wrote:
> Change the of_device_get_match_data() cast to (uintptr_t)
> to silence the following clang warning:
> 
> sound/soc/fsl/imx-audmux.c:301:16: warning: cast to smaller integer type 'enum imx_audmux_type' from 'const void *' [-Wvoid-pointer-to-enum-cast]
> 
> 

Applied to

   https://git.kernel.org/pub/scm/linux/kernel/git/broonie/sound.git for-next

Thanks!

[1/1] ASoC: imx-audmux: Silence a clang warning
      commit: 2f4a8171da06609bb6a063630ed546ee3d93dad7

All being well this means that it will be integrated into the linux-next
tree (usually sometime in the next 24 hours) and sent to Linus during
the next merge window (or sooner if it is a bug fix), however if
problems are discovered then the patch may be dropped or reverted.

You may get further e-mails resulting from automated or manual testing
and review of the tree, please engage with people reporting problems and
send followup patches addressing any issues that are reported if needed.

If any updates are required or you are submitting further changes they
should be sent as incremental updates against current git, existing
patches will not be replaced.

Please add any relevant lists and maintainers to the CCs when replying
to this mail.

Thanks,
Mark
diff mbox series

Patch

diff --git a/sound/soc/fsl/imx-audmux.c b/sound/soc/fsl/imx-audmux.c
index dfa05d40b276..a8e5e0f57faf 100644
--- a/sound/soc/fsl/imx-audmux.c
+++ b/sound/soc/fsl/imx-audmux.c
@@ -298,7 +298,7 @@  static int imx_audmux_probe(struct platform_device *pdev)
 		audmux_clk = NULL;
 	}
 
-	audmux_type = (enum imx_audmux_type)of_device_get_match_data(&pdev->dev);
+	audmux_type = (uintptr_t)of_device_get_match_data(&pdev->dev);
 
 	switch (audmux_type) {
 	case IMX31_AUDMUX: