[v5,2/3] media: s5p-mfc: Add variant data for MFC v7 hardware for Exynos 3250 SoC

Message ID 20221114115024.69591-3-aakarsh.jain@samsung.com
State New
Headers
Series Add new compatible for Exynos3250 |

Commit Message

Aakarsh Jain Nov. 14, 2022, 11:50 a.m. UTC
  Commit 5441e9dafdfc6dc40 ("[media] s5p-mfc: Core support for MFC v7")
which adds mfc v7 support for Exynos3250 and use the same compatible
string as used by Exynos5240 but both the IPs are a bit different in
terms of IP clock.
Add variant driver data based on the new compatible string 
"samsung,exynos3250-mfc" for Exynos3250 SoC.

Suggested-by: Alim Akhtar <alim.akhtar@samsung.com>
Fixes: 5441e9dafdfc ("[media] s5p-mfc: Core support for MFC v7")
Signed-off-by: Aakarsh Jain <aakarsh.jain@samsung.com>
---
 .../media/platform/samsung/s5p-mfc/s5p_mfc.c    | 17 +++++++++++++++--
 1 file changed, 15 insertions(+), 2 deletions(-)
  

Comments

Alim Akhtar Nov. 24, 2022, 11:43 a.m. UTC | #1
>-----Original Message-----
>From: Aakarsh Jain [mailto:aakarsh.jain@samsung.com]
>Sent: Monday, November 14, 2022 5:20 PM
>To: linux-arm-kernel@lists.infradead.org; linux-media@vger.kernel.org; linux-
>kernel@vger.kernel.org; devicetree@vger.kernel.org
>Cc: m.szyprowski@samsung.com; andrzej.hajda@intel.com;
>mchehab@kernel.org; hverkuil-cisco@xs4all.nl;
>ezequiel@vanguardiasur.com.ar; jernej.skrabec@gmail.com;
>benjamin.gaignard@collabora.com; krzysztof.kozlowski+dt@linaro.org;
>stanimir.varbanov@linaro.org; dillon.minfei@gmail.com;
>david.plowman@raspberrypi.com; mark.rutland@arm.com;
>robh+dt@kernel.org; krzk+dt@kernel.org; andi@etezian.org;
>alim.akhtar@samsung.com; aswani.reddy@samsung.com;
>pankaj.dubey@samsung.com; smitha.t@samsung.com;
>aakarsh.jain@samsung.com
>Subject: [Patch v5 2/3] media: s5p-mfc: Add variant data for MFC v7 hardware
>for Exynos 3250 SoC
>
>Commit 5441e9dafdfc6dc40 ("[media] s5p-mfc: Core support for MFC v7")
>which adds mfc v7 support for Exynos3250 and use the same compatible string
>as used by Exynos5240 but both the IPs are a bit different in terms of IP clock.
>Add variant driver data based on the new compatible string
>"samsung,exynos3250-mfc" for Exynos3250 SoC.
>
>Suggested-by: Alim Akhtar <alim.akhtar@samsung.com>
>Fixes: 5441e9dafdfc ("[media] s5p-mfc: Core support for MFC v7")
>Signed-off-by: Aakarsh Jain <aakarsh.jain@samsung.com>
>---
Thanks!

Reviewed-by: Alim Akhtar <alim.akhtar@samsung.com>

> .../media/platform/samsung/s5p-mfc/s5p_mfc.c    | 17 +++++++++++++++--
> 1 file changed, 15 insertions(+), 2 deletions(-)
>
>diff --git a/drivers/media/platform/samsung/s5p-mfc/s5p_mfc.c
>b/drivers/media/platform/samsung/s5p-mfc/s5p_mfc.c
>index fca5c6405eec..007c7dbee037 100644
>--- a/drivers/media/platform/samsung/s5p-mfc/s5p_mfc.c
>+++ b/drivers/media/platform/samsung/s5p-mfc/s5p_mfc.c
>@@ -1576,8 +1576,18 @@ static struct s5p_mfc_variant mfc_drvdata_v7 = {
> 	.port_num	= MFC_NUM_PORTS_V7,
> 	.buf_size	= &buf_size_v7,
> 	.fw_name[0]     = "s5p-mfc-v7.fw",
>-	.clk_names	= {"mfc", "sclk_mfc"},
>-	.num_clocks	= 2,
>+	.clk_names	= {"mfc"},
>+	.num_clocks	= 1,
>+};
>+
>+static struct s5p_mfc_variant mfc_drvdata_v7_3250 = {
>+	.version        = MFC_VERSION_V7,
>+	.version_bit    = MFC_V7_BIT,
>+	.port_num       = MFC_NUM_PORTS_V7,
>+	.buf_size       = &buf_size_v7,
>+	.fw_name[0]     = "s5p-mfc-v7.fw",
>+	.clk_names      = {"mfc", "sclk_mfc"},
>+	.num_clocks     = 2,
> };
>
> static struct s5p_mfc_buf_size_v6 mfc_buf_size_v8 = { @@ -1647,6 +1657,9
>@@ static const struct of_device_id exynos_mfc_match[] = {
> 	}, {
> 		.compatible = "samsung,mfc-v7",
> 		.data = &mfc_drvdata_v7,
>+	}, {
>+		.compatible = "samsung,exynos3250-mfc",
>+		.data = &mfc_drvdata_v7_3250,
> 	}, {
> 		.compatible = "samsung,mfc-v8",
> 		.data = &mfc_drvdata_v8,
>--
>2.17.1
  

Patch

diff --git a/drivers/media/platform/samsung/s5p-mfc/s5p_mfc.c b/drivers/media/platform/samsung/s5p-mfc/s5p_mfc.c
index fca5c6405eec..007c7dbee037 100644
--- a/drivers/media/platform/samsung/s5p-mfc/s5p_mfc.c
+++ b/drivers/media/platform/samsung/s5p-mfc/s5p_mfc.c
@@ -1576,8 +1576,18 @@  static struct s5p_mfc_variant mfc_drvdata_v7 = {
 	.port_num	= MFC_NUM_PORTS_V7,
 	.buf_size	= &buf_size_v7,
 	.fw_name[0]     = "s5p-mfc-v7.fw",
-	.clk_names	= {"mfc", "sclk_mfc"},
-	.num_clocks	= 2,
+	.clk_names	= {"mfc"},
+	.num_clocks	= 1,
+};
+
+static struct s5p_mfc_variant mfc_drvdata_v7_3250 = {
+	.version        = MFC_VERSION_V7,
+	.version_bit    = MFC_V7_BIT,
+	.port_num       = MFC_NUM_PORTS_V7,
+	.buf_size       = &buf_size_v7,
+	.fw_name[0]     = "s5p-mfc-v7.fw",
+	.clk_names      = {"mfc", "sclk_mfc"},
+	.num_clocks     = 2,
 };
 
 static struct s5p_mfc_buf_size_v6 mfc_buf_size_v8 = {
@@ -1647,6 +1657,9 @@  static const struct of_device_id exynos_mfc_match[] = {
 	}, {
 		.compatible = "samsung,mfc-v7",
 		.data = &mfc_drvdata_v7,
+	}, {
+		.compatible = "samsung,exynos3250-mfc",
+		.data = &mfc_drvdata_v7_3250,
 	}, {
 		.compatible = "samsung,mfc-v8",
 		.data = &mfc_drvdata_v8,