* [PATCH] media: i2c: Kconfig: Select COMMON_CLK for ACPI-based systems
@ 2025-07-08 16:16 Mehdi Djait
2025-07-08 16:28 ` Arnd Bergmann
0 siblings, 1 reply; 3+ messages in thread
From: Mehdi Djait @ 2025-07-08 16:16 UTC (permalink / raw)
To: sakari.ailus, laurent.pinchart
Cc: jacopo.mondi, hverkuil, kieran.bingham, mchehab, hdegoede, arnd,
linux-media, linux-kernel, Mehdi Djait
ACPI-based systems are required to create and register a fixed rate clock
using the new v4l2 helper devm_v4l2_sensor_clk_get() that uses the
common clk framework.
Ensure that COMMON_CLK is selected.
Signed-off-by: Mehdi Djait <mehdi.djait@linux.intel.com>
---
drivers/media/i2c/Kconfig | 1 +
1 file changed, 1 insertion(+)
diff --git a/drivers/media/i2c/Kconfig b/drivers/media/i2c/Kconfig
index e68202954a8f..f974a0e0dcec 100644
--- a/drivers/media/i2c/Kconfig
+++ b/drivers/media/i2c/Kconfig
@@ -31,6 +31,7 @@ menuconfig VIDEO_CAMERA_SENSOR
select MEDIA_CONTROLLER
select V4L2_FWNODE
select VIDEO_V4L2_SUBDEV_API
+ select COMMON_CLK if ACPI && !HAVE_LEGACY_CLK
default y
if VIDEO_CAMERA_SENSOR
--
2.49.0
^ permalink raw reply related [flat|nested] 3+ messages in thread
* Re: [PATCH] media: i2c: Kconfig: Select COMMON_CLK for ACPI-based systems
2025-07-08 16:16 [PATCH] media: i2c: Kconfig: Select COMMON_CLK for ACPI-based systems Mehdi Djait
@ 2025-07-08 16:28 ` Arnd Bergmann
2025-07-09 7:48 ` Mehdi Djait
0 siblings, 1 reply; 3+ messages in thread
From: Arnd Bergmann @ 2025-07-08 16:28 UTC (permalink / raw)
To: Mehdi Djait, Sakari Ailus, laurent.pinchart
Cc: Jacopo Mondi, Hans Verkuil, Kieran Bingham, Mauro Carvalho Chehab,
Hans de Goede, linux-media, linux-kernel
On Tue, Jul 8, 2025, at 18:16, Mehdi Djait wrote:
> ACPI-based systems are required to create and register a fixed rate clock
> using the new v4l2 helper devm_v4l2_sensor_clk_get() that uses the
> common clk framework.
>
> Ensure that COMMON_CLK is selected.
>
> Signed-off-by: Mehdi Djait <mehdi.djait@linux.intel.com>
You should never 'select' another subsystem or a drvier from
an unrelated subsystem.
What you want here instead is 'depends on COMMON_CLK'.
Arnd
^ permalink raw reply [flat|nested] 3+ messages in thread
* Re: [PATCH] media: i2c: Kconfig: Select COMMON_CLK for ACPI-based systems
2025-07-08 16:28 ` Arnd Bergmann
@ 2025-07-09 7:48 ` Mehdi Djait
0 siblings, 0 replies; 3+ messages in thread
From: Mehdi Djait @ 2025-07-09 7:48 UTC (permalink / raw)
To: Arnd Bergmann
Cc: Sakari Ailus, laurent.pinchart, Jacopo Mondi, Hans Verkuil,
Kieran Bingham, Mauro Carvalho Chehab, Hans de Goede, linux-media,
linux-kernel
Hello Arnd,
Thank you for the review.
On Tue, Jul 08, 2025 at 06:28:12PM +0200, Arnd Bergmann wrote:
> On Tue, Jul 8, 2025, at 18:16, Mehdi Djait wrote:
> > ACPI-based systems are required to create and register a fixed rate clock
> > using the new v4l2 helper devm_v4l2_sensor_clk_get() that uses the
> > common clk framework.
> >
> > Ensure that COMMON_CLK is selected.
> >
> > Signed-off-by: Mehdi Djait <mehdi.djait@linux.intel.com>
>
> You should never 'select' another subsystem or a drvier from
> an unrelated subsystem.
>
> What you want here instead is 'depends on COMMON_CLK'.
Ack.
I will change this in the v2.
--
Kind Regards
Mehdi Djait
^ permalink raw reply [flat|nested] 3+ messages in thread
end of thread, other threads:[~2025-07-09 7:48 UTC | newest]
Thread overview: 3+ messages (download: mbox.gz follow: Atom feed
-- links below jump to the message on this page --
2025-07-08 16:16 [PATCH] media: i2c: Kconfig: Select COMMON_CLK for ACPI-based systems Mehdi Djait
2025-07-08 16:28 ` Arnd Bergmann
2025-07-09 7:48 ` Mehdi Djait
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).