diff mbox

ASoC: fsi: Remove obsolete sh_fsi2 platform_device_id entry

Message ID 1436875174-13960-1-git-send-email-geert+renesas@glider.be (mailing list archive)
State Accepted
Delegated to: Geert Uytterhoeven
Headers show

Commit Message

Geert Uytterhoeven July 14, 2015, 11:59 a.m. UTC
Since the removal of the r8a7740 legacy board code in commit
1fa59bda21c7fa36 ("ARM: shmobile: Remove legacy board code for
Armadillo-800 EVA"), all former users of the "sh_fsi2" platform device
name are only supported in generic DT-only ARM multi-platform builds.
The driver doesn't need to match platform devices by name anymore, hence
remove the corresponding platform_device_id entry.

Signed-off-by: Geert Uytterhoeven <geert+renesas@glider.be>
---
The commit mentioned above is in -next, destined for v4.3.
---
 sound/soc/sh/fsi.c | 1 -
 1 file changed, 1 deletion(-)

Comments

Simon Horman July 15, 2015, 12:09 a.m. UTC | #1
On Tue, Jul 14, 2015 at 01:59:34PM +0200, Geert Uytterhoeven wrote:
> Since the removal of the r8a7740 legacy board code in commit
> 1fa59bda21c7fa36 ("ARM: shmobile: Remove legacy board code for
> Armadillo-800 EVA"), all former users of the "sh_fsi2" platform device
> name are only supported in generic DT-only ARM multi-platform builds.
> The driver doesn't need to match platform devices by name anymore, hence
> remove the corresponding platform_device_id entry.
> 
> Signed-off-by: Geert Uytterhoeven <geert+renesas@glider.be>

Acked-by: Simon Horman <horms+renesas@verge.net.au>

> ---
> The commit mentioned above is in -next, destined for v4.3.
> ---
>  sound/soc/sh/fsi.c | 1 -
>  1 file changed, 1 deletion(-)
> 
> diff --git a/sound/soc/sh/fsi.c b/sound/soc/sh/fsi.c
> index 142c066eaee2ecb1..0215c78cbddf5491 100644
> --- a/sound/soc/sh/fsi.c
> +++ b/sound/soc/sh/fsi.c
> @@ -1911,7 +1911,6 @@ MODULE_DEVICE_TABLE(of, fsi_of_match);
>  
>  static const struct platform_device_id fsi_id_table[] = {
>  	{ "sh_fsi",	(kernel_ulong_t)&fsi1_core },
> -	{ "sh_fsi2",	(kernel_ulong_t)&fsi2_core },
>  	{},
>  };
>  MODULE_DEVICE_TABLE(platform, fsi_id_table);
> -- 
> 1.9.1
> 
> --
> To unsubscribe from this list: send the line "unsubscribe linux-sh" in
> the body of a message to majordomo@vger.kernel.org
> More majordomo info at  http://vger.kernel.org/majordomo-info.html
> 
--
To unsubscribe from this list: send the line "unsubscribe linux-sh" in
the body of a message to majordomo@vger.kernel.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html
Kuninori Morimoto July 15, 2015, 1:02 a.m. UTC | #2
Hi

> Since the removal of the r8a7740 legacy board code in commit
> 1fa59bda21c7fa36 ("ARM: shmobile: Remove legacy board code for
> Armadillo-800 EVA"), all former users of the "sh_fsi2" platform device
> name are only supported in generic DT-only ARM multi-platform builds.
> The driver doesn't need to match platform devices by name anymore, hence
> remove the corresponding platform_device_id entry.
> 
> Signed-off-by: Geert Uytterhoeven <geert+renesas@glider.be>
> ---

Acked-by: Kuninori Morimoto <kuninori.morimoto.gx@renesas.com>
--
To unsubscribe from this list: send the line "unsubscribe linux-sh" in
the body of a message to majordomo@vger.kernel.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html
Geert Uytterhoeven July 16, 2015, 7:55 a.m. UTC | #3
Hi Mark,

On Tue, Jul 14, 2015 at 1:59 PM, Geert Uytterhoeven
<geert+renesas@glider.be> wrote:
> Since the removal of the r8a7740 legacy board code in commit
> 1fa59bda21c7fa36 ("ARM: shmobile: Remove legacy board code for
> Armadillo-800 EVA"), all former users of the "sh_fsi2" platform device
> name are only supported in generic DT-only ARM multi-platform builds.
> The driver doesn't need to match platform devices by name anymore, hence
> remove the corresponding platform_device_id entry.
>
> Signed-off-by: Geert Uytterhoeven <geert+renesas@glider.be>

Looks like you applied this to spi/for-next instead of sound-asoc/for-next?

Gr{oetje,eeting}s,

                        Geert

--
Geert Uytterhoeven -- There's lots of Linux beyond ia32 -- geert@linux-m68k.org

In personal conversations with technical people, I call myself a hacker. But
when I'm talking to journalists I just say "programmer" or something like that.
                                -- Linus Torvalds
--
To unsubscribe from this list: send the line "unsubscribe linux-sh" in
the body of a message to majordomo@vger.kernel.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html
diff mbox

Patch

diff --git a/sound/soc/sh/fsi.c b/sound/soc/sh/fsi.c
index 142c066eaee2ecb1..0215c78cbddf5491 100644
--- a/sound/soc/sh/fsi.c
+++ b/sound/soc/sh/fsi.c
@@ -1911,7 +1911,6 @@  MODULE_DEVICE_TABLE(of, fsi_of_match);
 
 static const struct platform_device_id fsi_id_table[] = {
 	{ "sh_fsi",	(kernel_ulong_t)&fsi1_core },
-	{ "sh_fsi2",	(kernel_ulong_t)&fsi2_core },
 	{},
 };
 MODULE_DEVICE_TABLE(platform, fsi_id_table);