linux-sunxi.lists.linux.dev archive mirror
 help / color / mirror / Atom feed
From: Sakari Ailus <sakari.ailus@linux.intel.com>
To: linux-media@vger.kernel.org
Cc: "Krzysztof Kozlowski" <krzysztof.kozlowski@linaro.org>,
	"Joe Tessler" <jrt@google.com>,
	"Hans Verkuil" <hverkuil-cisco@xs4all.nl>,
	"Mauro Carvalho Chehab" <mchehab@kernel.org>,
	"Neil Armstrong" <neil.armstrong@linaro.org>,
	"Kevin Hilman" <khilman@baylibre.com>,
	"Jerome Brunet" <jbrunet@baylibre.com>,
	"Martin Blumenstingl" <martin.blumenstingl@googlemail.com>,
	"Thierry Reding" <thierry.reding@gmail.com>,
	"Jonathan Hunter" <jonathanh@nvidia.com>,
	"Manivannan Sadhasivam" <mani@kernel.org>,
	"Michael Tretter" <m.tretter@pengutronix.de>,
	"Pengutronix Kernel Team" <kernel@pengutronix.de>,
	"Jacopo Mondi" <jacopo+renesas@jmondi.org>,
	"Kieran Bingham" <kieran.bingham+renesas@ideasonboard.com>,
	"Laurent Pinchart" <laurent.pinchart+renesas@ideasonboard.com>,
	"Niklas Söderlund" <niklas.soderlund+renesas@ragnatech.se>,
	"Rui Miguel Silva" <rmfrfs@gmail.com>,
	"Wenyou Yang" <wenyou.yang@microchip.com>,
	"Bin Liu" <bin.liu@mediatek.com>,
	"Matthias Brugger" <matthias.bgg@gmail.com>,
	"AngeloGioacchino Del Regno"
	<angelogioacchino.delregno@collabora.com>,
	"Minghsiu Tsai" <minghsiu.tsai@mediatek.com>,
	"Houlong Wei" <houlong.wei@mediatek.com>,
	"Andrew-CT Chen" <andrew-ct.chen@mediatek.com>,
	"Andrzej Pietrasiewicz" <andrzejtp2010@gmail.com>,
	"Jacek Anaszewski" <jacek.anaszewski@gmail.com>,
	"Sylwester Nawrocki" <s.nawrocki@samsung.com>,
	"Patrice Chotard" <patrice.chotard@foss.st.com>,
	"Yong Deng" <yong.deng@magewell.com>,
	"Paul Kocialkowski" <paul.kocialkowski@bootlin.com>,
	"Chen-Yu Tsai" <wens@csie.org>,
	"Jernej Skrabec" <jernej.skrabec@gmail.com>,
	"Samuel Holland" <samuel@sholland.org>,
	"Lad, Prabhakar" <prabhakar.csengg@gmail.com>,
	"Ezequiel Garcia" <ezequiel@vanguardiasur.com.ar>,
	"Philipp Zabel" <p.zabel@pengutronix.de>,
	"Sean Young" <sean@mess.org>,
	linux-kernel@vger.kernel.org, linux-amlogic@lists.infradead.org,
	linux-arm-kernel@lists.infradead.org,
	linux-tegra@vger.kernel.org, linux-mediatek@lists.infradead.org,
	linux-sunxi@lists.linux.dev, linux-rockchip@lists.infradead.org
Subject: [PATCH v2 20/28] media: platform: jpeg: always reference OF data
Date: Wed, 21 Jun 2023 00:03:08 +0300	[thread overview]
Message-ID: <20230620210308.650986-1-sakari.ailus@linux.intel.com> (raw)
In-Reply-To: <20230312131318.351173-20-krzysztof.kozlowski@linaro.org>

From: Krzysztof Kozlowski <krzysztof.kozlowski@linaro.org>

The driver can match only via the DT table so the table should be always
used and the of_match_ptr does not have any sense (this also allows ACPI
matching via PRP0001, even though it might not be relevant here).

Signed-off-by: Krzysztof Kozlowski <krzysztof.kozlowski@linaro.org>
[Sakari Ailus: Rebased on media tree master, reword commit message,
	       drop additional #if defined(CONFIG_OF)/#endif's.]
Signed-off-by: Sakari Ailus <sakari.ailus@linux.intel.com>
---
 drivers/media/platform/mediatek/jpeg/mtk_jpeg_core.c   | 6 +-----
 drivers/media/platform/mediatek/jpeg/mtk_jpeg_dec_hw.c | 4 +---
 drivers/media/platform/mediatek/jpeg/mtk_jpeg_enc_hw.c | 4 +---
 3 files changed, 3 insertions(+), 11 deletions(-)

diff --git a/drivers/media/platform/mediatek/jpeg/mtk_jpeg_core.c b/drivers/media/platform/mediatek/jpeg/mtk_jpeg_core.c
index 4768156181c99..40cb3cb87ba17 100644
--- a/drivers/media/platform/mediatek/jpeg/mtk_jpeg_core.c
+++ b/drivers/media/platform/mediatek/jpeg/mtk_jpeg_core.c
@@ -28,7 +28,6 @@
 #include "mtk_jpeg_core.h"
 #include "mtk_jpeg_dec_parse.h"
 
-#if defined(CONFIG_OF)
 static struct mtk_jpeg_fmt mtk_jpeg_enc_formats[] = {
 	{
 		.fourcc		= V4L2_PIX_FMT_JPEG,
@@ -102,7 +101,6 @@ static struct mtk_jpeg_fmt mtk_jpeg_dec_formats[] = {
 		.flags		= MTK_JPEG_FMT_FLAG_CAPTURE,
 	},
 };
-#endif
 
 #define MTK_JPEG_ENC_NUM_FORMATS ARRAY_SIZE(mtk_jpeg_enc_formats)
 #define MTK_JPEG_DEC_NUM_FORMATS ARRAY_SIZE(mtk_jpeg_dec_formats)
@@ -1455,7 +1453,6 @@ static const struct dev_pm_ops mtk_jpeg_pm_ops = {
 	SET_RUNTIME_PM_OPS(mtk_jpeg_pm_suspend, mtk_jpeg_pm_resume, NULL)
 };
 
-#if defined(CONFIG_OF)
 static int mtk_jpegenc_get_hw(struct mtk_jpeg_ctx *ctx)
 {
 	struct mtk_jpegenc_comp_dev *comp_jpeg;
@@ -1951,14 +1948,13 @@ static const struct of_device_id mtk_jpeg_match[] = {
 };
 
 MODULE_DEVICE_TABLE(of, mtk_jpeg_match);
-#endif
 
 static struct platform_driver mtk_jpeg_driver = {
 	.probe = mtk_jpeg_probe,
 	.remove_new = mtk_jpeg_remove,
 	.driver = {
 		.name           = MTK_JPEG_NAME,
-		.of_match_table = of_match_ptr(mtk_jpeg_match),
+		.of_match_table = mtk_jpeg_match,
 		.pm             = &mtk_jpeg_pm_ops,
 	},
 };
diff --git a/drivers/media/platform/mediatek/jpeg/mtk_jpeg_dec_hw.c b/drivers/media/platform/mediatek/jpeg/mtk_jpeg_dec_hw.c
index 869068fac5e2f..baa7be58ce691 100644
--- a/drivers/media/platform/mediatek/jpeg/mtk_jpeg_dec_hw.c
+++ b/drivers/media/platform/mediatek/jpeg/mtk_jpeg_dec_hw.c
@@ -39,7 +39,6 @@ enum mtk_jpeg_color {
 	MTK_JPEG_COLOR_400		= 0x00110000
 };
 
-#if defined(CONFIG_OF)
 static const struct of_device_id mtk_jpegdec_hw_ids[] = {
 	{
 		.compatible = "mediatek,mt8195-jpgdec-hw",
@@ -47,7 +46,6 @@ static const struct of_device_id mtk_jpegdec_hw_ids[] = {
 	{},
 };
 MODULE_DEVICE_TABLE(of, mtk_jpegdec_hw_ids);
-#endif
 
 static inline int mtk_jpeg_verify_align(u32 val, int align, u32 reg)
 {
@@ -653,7 +651,7 @@ static struct platform_driver mtk_jpegdec_hw_driver = {
 	.probe = mtk_jpegdec_hw_probe,
 	.driver = {
 		.name = "mtk-jpegdec-hw",
-		.of_match_table = of_match_ptr(mtk_jpegdec_hw_ids),
+		.of_match_table = mtk_jpegdec_hw_ids,
 	},
 };
 
diff --git a/drivers/media/platform/mediatek/jpeg/mtk_jpeg_enc_hw.c b/drivers/media/platform/mediatek/jpeg/mtk_jpeg_enc_hw.c
index 71e85b4bbf127..244018365b6f1 100644
--- a/drivers/media/platform/mediatek/jpeg/mtk_jpeg_enc_hw.c
+++ b/drivers/media/platform/mediatek/jpeg/mtk_jpeg_enc_hw.c
@@ -46,7 +46,6 @@ static const struct mtk_jpeg_enc_qlt mtk_jpeg_enc_quality[] = {
 	{.quality_param = 97, .hardware_value = JPEG_ENC_QUALITY_Q97},
 };
 
-#if defined(CONFIG_OF)
 static const struct of_device_id mtk_jpegenc_drv_ids[] = {
 	{
 		.compatible = "mediatek,mt8195-jpgenc-hw",
@@ -54,7 +53,6 @@ static const struct of_device_id mtk_jpegenc_drv_ids[] = {
 	{},
 };
 MODULE_DEVICE_TABLE(of, mtk_jpegenc_drv_ids);
-#endif
 
 void mtk_jpeg_enc_reset(void __iomem *base)
 {
@@ -377,7 +375,7 @@ static struct platform_driver mtk_jpegenc_hw_driver = {
 	.probe = mtk_jpegenc_hw_probe,
 	.driver = {
 		.name = "mtk-jpegenc-hw",
-		.of_match_table = of_match_ptr(mtk_jpegenc_drv_ids),
+		.of_match_table = mtk_jpegenc_drv_ids,
 	},
 };
 
-- 
2.39.2


  parent reply	other threads:[~2023-06-20 21:03 UTC|newest]

Thread overview: 72+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2023-03-12 13:12 [PATCH 01/28] media: cec: ch7322: drop of_match_ptr for ID table Krzysztof Kozlowski
2023-03-12 13:12 ` [PATCH 02/28] media: cec: meson: " Krzysztof Kozlowski
2023-03-12 20:22   ` Martin Blumenstingl
2023-03-13 11:46   ` Hans Verkuil
2023-03-12 13:12 ` [PATCH 03/28] media: cec: tegra: " Krzysztof Kozlowski
2023-03-13 11:46   ` Hans Verkuil
2023-03-12 13:12 ` [PATCH 04/28] media: rc: gpio-ir-recv: " Krzysztof Kozlowski
2023-03-13 14:45   ` Sean Young
2023-03-12 13:12 ` [PATCH 05/28] media: rc: gpio-ir-tx: " Krzysztof Kozlowski
2023-03-13 14:45   ` Sean Young
2023-03-12 13:12 ` [PATCH 06/28] media: rc: ir-rx51: " Krzysztof Kozlowski
2023-03-13 14:47   ` Sean Young
2023-03-12 13:12 ` [PATCH 07/28] media: platform: allegro-dvt: " Krzysztof Kozlowski
2023-03-13 14:56   ` Michael Tretter
2023-03-12 13:12 ` [PATCH 08/28] media: platform: intel: pxa: " Krzysztof Kozlowski
2023-03-12 13:12 ` [PATCH 09/28] media: platform: samsung: s5p-jpeg: " Krzysztof Kozlowski
2023-03-13 17:08   ` Andrzej Pietrasiewicz
2023-03-12 13:13 ` [PATCH 10/28] media: platform: sti: " Krzysztof Kozlowski
2023-03-12 13:13 ` [PATCH 11/28] media: platform: ti: am437x: " Krzysztof Kozlowski
2023-03-12 13:41   ` Laurent Pinchart
2023-03-13 20:53   ` Lad, Prabhakar
2023-03-12 13:13 ` [PATCH 12/28] media: platform: verisilicon: " Krzysztof Kozlowski
2023-03-12 13:13 ` [PATCH 13/28] media: platform: marvell: " Krzysztof Kozlowski
2023-03-12 13:13 ` [PATCH 14/28] media: platform: mdp3: " Krzysztof Kozlowski
2023-03-12 13:13 ` [PATCH 15/28] media: platform: mdp3: mark OF related data as maybe unused Krzysztof Kozlowski
2023-03-12 13:13 ` [PATCH 16/28] media: platform: sun6i-csi: drop of_match_ptr for ID table Krzysztof Kozlowski
2023-03-14 16:02   ` Paul Kocialkowski
2023-03-14 19:51   ` Jernej Škrabec
2023-03-12 13:13 ` [PATCH 17/28] media: platform: sun6i-mipi-csi2: " Krzysztof Kozlowski
2023-03-14 16:04   ` Paul Kocialkowski
2023-03-14 19:52   ` Jernej Škrabec
2023-03-12 13:13 ` [PATCH 18/28] media: platform: sun8i-a83t-mipi-csi2: " Krzysztof Kozlowski
2023-03-14 16:04   ` Paul Kocialkowski
2023-03-14 19:52   ` Jernej Škrabec
2023-03-12 13:13 ` [PATCH 19/28] media: platform: mdp: mark OF related data as maybe unused Krzysztof Kozlowski
2023-03-12 13:13 ` [PATCH 20/28] media: platform: jpeg: always reference OF data Krzysztof Kozlowski
2023-03-21  8:11   ` Arnd Bergmann
2023-03-21  8:21     ` Krzysztof Kozlowski
2023-03-21  8:24       ` Arnd Bergmann
2023-03-21  8:26         ` Krzysztof Kozlowski
2023-05-25 13:16           ` Sakari Ailus
2023-06-20 21:03   ` Sakari Ailus [this message]
2023-03-12 13:13 ` [PATCH 21/28] media: i2c: isl7998x: drop of_match_ptr for ID table Krzysztof Kozlowski
2023-03-12 13:13 ` [PATCH 22/28] media: i2c: mt9m111: " Krzysztof Kozlowski
2023-03-12 13:43   ` Laurent Pinchart
2023-03-12 13:13 ` [PATCH 23/28] media: i2c: ov2640: " Krzysztof Kozlowski
2023-03-12 13:44   ` Laurent Pinchart
2023-03-12 13:13 ` [PATCH 24/28] media: i2c: ov2680: " Krzysztof Kozlowski
2023-03-12 13:44   ` Laurent Pinchart
2023-03-12 13:13 ` [PATCH 25/28] media: i2c: ov7740: " Krzysztof Kozlowski
2023-03-12 13:44   ` Laurent Pinchart
2023-03-12 13:13 ` [PATCH 26/28] media: i2c: imx290: " Krzysztof Kozlowski
2023-03-12 13:44   ` Laurent Pinchart
2023-03-12 13:13 ` [PATCH 27/28] media: i2c: max9286: " Krzysztof Kozlowski
2023-03-12 13:45   ` Laurent Pinchart
2023-03-12 13:13 ` [PATCH 28/28] media: dvb: mn88443x: " Krzysztof Kozlowski
2023-03-13 11:46 ` [PATCH 01/28] media: cec: ch7322: " Hans Verkuil
2023-03-14 16:31   ` Krzysztof Kozlowski
2023-03-13 22:01 ` Guenter Roeck
2023-03-14  8:34   ` Hans Verkuil
2023-03-14 16:32     ` Krzysztof Kozlowski
2023-03-14 16:45       ` Guenter Roeck
2023-05-12 16:35 ` Krzysztof Kozlowski
2023-05-13  9:57   ` Hans Verkuil
2023-05-25 12:40     ` Sakari Ailus
2023-05-25 13:19       ` Sakari Ailus
2023-06-19 14:13         ` Krzysztof Kozlowski
2023-07-13  6:46           ` Krzysztof Kozlowski
2023-07-13  7:07             ` Hans Verkuil
2023-07-25 10:51             ` Sakari Ailus
2023-07-26  8:20               ` Krzysztof Kozlowski
2023-06-21 18:06 ` Guenter Roeck

Reply instructions:

You may reply publicly to this message via plain-text email
using any one of the following methods:

* Save the following mbox file, import it into your mail client,
  and reply-to-all from there: mbox

  Avoid top-posting and favor interleaved quoting:
  https://en.wikipedia.org/wiki/Posting_style#Interleaved_style

* Reply using the --to, --cc, and --in-reply-to
  switches of git-send-email(1):

  git send-email \
    --in-reply-to=20230620210308.650986-1-sakari.ailus@linux.intel.com \
    --to=sakari.ailus@linux.intel.com \
    --cc=andrew-ct.chen@mediatek.com \
    --cc=andrzejtp2010@gmail.com \
    --cc=angelogioacchino.delregno@collabora.com \
    --cc=bin.liu@mediatek.com \
    --cc=ezequiel@vanguardiasur.com.ar \
    --cc=houlong.wei@mediatek.com \
    --cc=hverkuil-cisco@xs4all.nl \
    --cc=jacek.anaszewski@gmail.com \
    --cc=jacopo+renesas@jmondi.org \
    --cc=jbrunet@baylibre.com \
    --cc=jernej.skrabec@gmail.com \
    --cc=jonathanh@nvidia.com \
    --cc=jrt@google.com \
    --cc=kernel@pengutronix.de \
    --cc=khilman@baylibre.com \
    --cc=kieran.bingham+renesas@ideasonboard.com \
    --cc=krzysztof.kozlowski@linaro.org \
    --cc=laurent.pinchart+renesas@ideasonboard.com \
    --cc=linux-amlogic@lists.infradead.org \
    --cc=linux-arm-kernel@lists.infradead.org \
    --cc=linux-kernel@vger.kernel.org \
    --cc=linux-media@vger.kernel.org \
    --cc=linux-mediatek@lists.infradead.org \
    --cc=linux-rockchip@lists.infradead.org \
    --cc=linux-sunxi@lists.linux.dev \
    --cc=linux-tegra@vger.kernel.org \
    --cc=m.tretter@pengutronix.de \
    --cc=mani@kernel.org \
    --cc=martin.blumenstingl@googlemail.com \
    --cc=matthias.bgg@gmail.com \
    --cc=mchehab@kernel.org \
    --cc=minghsiu.tsai@mediatek.com \
    --cc=neil.armstrong@linaro.org \
    --cc=niklas.soderlund+renesas@ragnatech.se \
    --cc=p.zabel@pengutronix.de \
    --cc=patrice.chotard@foss.st.com \
    --cc=paul.kocialkowski@bootlin.com \
    --cc=prabhakar.csengg@gmail.com \
    --cc=rmfrfs@gmail.com \
    --cc=s.nawrocki@samsung.com \
    --cc=samuel@sholland.org \
    --cc=sean@mess.org \
    --cc=thierry.reding@gmail.com \
    --cc=wens@csie.org \
    --cc=wenyou.yang@microchip.com \
    --cc=yong.deng@magewell.com \
    /path/to/YOUR_REPLY

  https://kernel.org/pub/software/scm/git/docs/git-send-email.html

* If your mail client supports setting the In-Reply-To header
  via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line before the message body.
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for NNTP newsgroup(s).