From mboxrd@z Thu Jan 1 00:00:00 1970 Received: from mail-pl1-f180.google.com (mail-pl1-f180.google.com [209.85.214.180]) (using TLSv1.2 with cipher ECDHE-RSA-AES128-GCM-SHA256 (128/128 bits)) (No client certificate requested) by smtp.subspace.kernel.org (Postfix) with ESMTPS id 72A92378D84 for ; Fri, 10 Jul 2026 05:25:29 +0000 (UTC) Authentication-Results: smtp.subspace.kernel.org; arc=none smtp.client-ip=209.85.214.180 ARC-Seal:i=1; a=rsa-sha256; d=subspace.kernel.org; s=arc-20240116; t=1783661130; cv=none; b=h4r3hVbDagnJYj/wrF0zb8CK8g1HHDAHnaUCM7G+5ND+WRNdsBn/J02hzSDkHU24yP9BRXFZzaMQNLx3LS08yY6qLP3gctm4/dpPIrWu3STvuLt7gCFt5MgRsdDPKf4809abfVFtOq/2pxU5AjZSmD202ElK/b5XDmxtvfetaZM= ARC-Message-Signature:i=1; a=rsa-sha256; d=subspace.kernel.org; s=arc-20240116; t=1783661130; c=relaxed/simple; bh=6BFg6sPI3okgrHxzhlSMRgyX9vuNYoJ8TgSTZlqHBhE=; h=From:To:Cc:Subject:Date:Message-Id:MIME-Version; b=alpURl0sIF4ib7KVMfnnc8NDZ5qyWDQbV2vY9ULlbEYzWrt4RKOJ4NJNVjTa1IaIs4tdwHDKlKSe4kXRUPFhrvUwKyQxqaS0LKiFka0Ufi6L2OYZLCzxqoeT5RRAkAppTf3sqN0qUa3gPDKl3SAvBXsWcFdNxFcYDd49NsN+IAs= ARC-Authentication-Results:i=1; smtp.subspace.kernel.org; dmarc=pass (p=none dis=none) header.from=gmail.com; spf=pass smtp.mailfrom=gmail.com; dkim=pass (2048-bit key) header.d=gmail.com header.i=@gmail.com header.b=pjln7nLG; arc=none smtp.client-ip=209.85.214.180 Authentication-Results: smtp.subspace.kernel.org; dmarc=pass (p=none dis=none) header.from=gmail.com Authentication-Results: smtp.subspace.kernel.org; spf=pass smtp.mailfrom=gmail.com Authentication-Results: smtp.subspace.kernel.org; dkim=pass (2048-bit key) header.d=gmail.com header.i=@gmail.com header.b="pjln7nLG" Received: by mail-pl1-f180.google.com with SMTP id d9443c01a7336-2ce87c7e3bbso3495925ad.1 for ; Thu, 09 Jul 2026 22:25:29 -0700 (PDT) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=gmail.com; s=20251104; t=1783661129; x=1784265929; darn=vger.kernel.org; h=content-transfer-encoding:mime-version:message-id:date:subject:cc :to:from:from:to:cc:subject:date:message-id:reply-to:content-type; bh=7+yyDbnS7MzY461JKpKTsvuYdfC2If2+XrHZ9AJhM98=; b=pjln7nLG9f8Qp3cyZaZwOVs5XXjh5PjuUIBZkuek+HaT8ImPJsdKiEuwk9eYlTAjw0 0wxJYN2ObM7HI8mU/j9iAeakpiX3V4TmYgOQViY+F4Yt6jzQy/qpUWzIjyxhWwv91hjv B9A133c7uqpZiw/dNeYi0C5sjOcH+H6EDXPSF5wXN48QPQ0BUsKjvPAF+C0vZ+k6OuG/ J2I23gdHv5y+Y6hXrh/y80eNF+zYpA//0bzxzD6BrYHcJRIuXXV7aTTzZYx3pbPjxLrh JhU913Ls8NoUcfcuqe8fH40815tEcZd0oX980B7wlLU82e/ZK1hwn4xKVuWtudAMwWoW t5AA== X-Google-DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=1e100.net; s=20251104; t=1783661129; x=1784265929; h=content-transfer-encoding:mime-version:message-id:date:subject:cc :to:from:x-gm-gg:x-gm-message-state:from:to:cc:subject:date :message-id:reply-to:content-type; bh=7+yyDbnS7MzY461JKpKTsvuYdfC2If2+XrHZ9AJhM98=; b=XgqobueAVGvyFEsCTgwswavBEUYhwwpmU6mpxMptts6ydzmmoG8klqbKXPSlmJPPXk 8j+hc12U/Za+qillyHb0iI9Pc0QbOuLSvS+4c0B/a9MkxA6RQH+ceU4nvx+5906UHXPd hb/dp0BJ7eFw5liEfr2h+5UsgSlD1lxf/himRl5wtZTkf7RY9v0ScmZF+XBWoL1Pxhmk HI85NWR6/r08e2eoXhWhu8ybldA1MoQO9C1sl+0OY7AdNZ1Bno5WIN2Wazx7LoVYR7ML 0/ReV8e2XjmYcnOtPjQ6iYQQ3QgbpvmG7h3PaHandfqWX5OKxeAz1iUxWUgBgQoOaxNo EV5w== X-Forwarded-Encrypted: i=1; AHgh+RqM8OvgrShfHkudNTJifKsKjKRDHCV5brw+CPV4HJvYBt45774eR7abjKs60U06gBsvXybxjobt4xi6zLE=@vger.kernel.org X-Gm-Message-State: AOJu0Yy7R5JP0TlUQQEDXtbehSkUxkl4KHUk5CgIB0pfOtEk5qiJUPvf d837KAxf/Si1SNvyHsr0v84LbE0Jc2oSpz1zNhJvhtiB9o/mVSlzqqnd X-Gm-Gg: AfdE7ckPjL09NHZjSSqeMmNrpz+5gm2zGuRq1sANignglOL/K7/YO7Y8frL49kpb6OG aCOdn0tbXP2HpSD3B+HGXtFgZQiaKK26G3gcLAF5+2CPwl1haLrY+G2qJgjJKaF4a4/e/4nB12u gUKdG146BH2EDGoo9vHe+Pd9ETSR9GY8I8xhhYhf0xMxnv/U2vVF7bHVytmaEMXVMQkBD2+1ktN gm3s8bk4Pp+Y63Rp8GvnKBfqxlbvw/5HggNwAFCjfH/z5JWxgnBFE53RvDY++WE3XQtZgKzG0z8 BiRhC385v6gm5Gv3uhvqvRtucoa1hNa7CuMPn25HKXN7BYoBu43MZJ4reL4KKf70AGq0ysNgdR0 WSm2r1Eh7XAfXQVOc2h7ecCOY8qRbWFXC/zusWjEnkiItBgYI7NqMG9n7XcG7hbWJ2x5Lc0tYhP tap+Ihyiobm/HSy5MENlUXo0620F5X6HzoSFuKIR/b X-Received: by 2002:a05:6a21:329e:b0:3bf:d1f9:b1df with SMTP id adf61e73a8af0-3c0bcc241abmr12412257637.54.1783661128588; Thu, 09 Jul 2026 22:25:28 -0700 (PDT) Received: from hu-ckantibh-hyd.qualcomm.com ([202.46.23.25]) by smtp.gmail.com with ESMTPSA id a92af1059eb24-13b87b92b27sm947768c88.6.2026.07.09.22.25.26 (version=TLS1_3 cipher=TLS_AES_256_GCM_SHA384 bits=256/256); Thu, 09 Jul 2026 22:25:28 -0700 (PDT) From: Sanjay Chitroda To: hansg@kernel.org, sakari.ailus@linux.intel.com, mchehab@kernel.org Cc: linux-media@vger.kernel.org, linux-kernel@vger.kernel.org Subject: [PATCH v2] media: i2c: gc0310: Use devm_v4l2_sensor_clk_get() Date: Fri, 10 Jul 2026 10:55:23 +0530 Message-Id: <20260710052523.1580208-1-sanjayembeddedse@gmail.com> X-Mailer: git-send-email 2.34.1 Precedence: bulk X-Mailing-List: linux-kernel@vger.kernel.org List-Id: List-Subscribe: List-Unsubscribe: MIME-Version: 1.0 Content-Transfer-Encoding: 8bit Several camera sensor drivers access the "clock-frequency" property directly to retrieve the external clock rate or handle the external clock manually in the driver. While this is valid on a subset of ACPI platforms, implementing this logic directly in drivers is deprecated and can lead to inconsistent behaviour across drivers. This driver supports ACPI platforms only. It currently retrieves the external clock rate from the "clock-frequency" property and fails probing if the rate does not match the expected value, which is the correct policy for ACPI platforms. Switch to using the devm_v4l2_sensor_clk_get() helper to standardise clock handling. This preserves the existing behaviour on ACPI platforms that specify a clock-frequency property without providing a clock. On platforms that provide a clock, the helper will program the clock to the rate specified by clock-frequency, which is also consistent with the driver's expectations. Reviewed-by: Hans de Goede Signed-off-by: Sanjay Chitroda --- Changes in v2: - With input from Hans de drop first 2 changes and rebase 3rd change along with reviewed tag on top of mainline tree and v7.2-rc2 commit tag. - Link to v1: https://lore.kernel.org/20260401181657.654055-1-sanjayembedded@gmail.com --- drivers/media/i2c/gc0310.c | 30 ++++++++++++++---------------- 1 file changed, 14 insertions(+), 16 deletions(-) diff --git a/drivers/media/i2c/gc0310.c b/drivers/media/i2c/gc0310.c index 7af4d66f42a0..754e82ad50ae 100644 --- a/drivers/media/i2c/gc0310.c +++ b/drivers/media/i2c/gc0310.c @@ -6,6 +6,7 @@ * Copyright (c) 2023-2025 Hans de Goede */ +#include #include #include #include @@ -84,6 +85,8 @@ #define to_gc0310_sensor(x) container_of(x, struct gc0310_device, sd) struct gc0310_device { + struct clk *clk; + struct v4l2_subdev sd; struct media_pad pad; @@ -635,7 +638,6 @@ static int gc0310_check_hwcfg(struct device *dev) }; struct fwnode_handle *ep_fwnode; unsigned long link_freq_bitmap; - u32 mclk; int ret; /* @@ -647,21 +649,6 @@ static int gc0310_check_hwcfg(struct device *dev) return dev_err_probe(dev, -EPROBE_DEFER, "waiting for fwnode graph endpoint\n"); - ret = fwnode_property_read_u32(dev_fwnode(dev), "clock-frequency", - &mclk); - if (ret) { - fwnode_handle_put(ep_fwnode); - return dev_err_probe(dev, ret, - "reading clock-frequency property\n"); - } - - if (mclk != GC0310_MCLK_FREQ) { - fwnode_handle_put(ep_fwnode); - return dev_err_probe(dev, -EINVAL, - "external clock %u is not supported\n", - mclk); - } - ret = v4l2_fwnode_endpoint_alloc_parse(ep_fwnode, &bus_cfg); fwnode_handle_put(ep_fwnode); if (ret) @@ -685,6 +672,7 @@ static int gc0310_check_hwcfg(struct device *dev) static int gc0310_probe(struct i2c_client *client) { struct gc0310_device *sensor; + unsigned long freq; int ret; ret = gc0310_check_hwcfg(&client->dev); @@ -695,6 +683,16 @@ static int gc0310_probe(struct i2c_client *client) if (!sensor) return -ENOMEM; + sensor->clk = devm_v4l2_sensor_clk_get(&client->dev, NULL); + if (IS_ERR(sensor->clk)) + return dev_err_probe(&client->dev, PTR_ERR(sensor->clk), + "failed to get clock\n"); + + freq = clk_get_rate(sensor->clk); + if (freq != GC0310_MCLK_FREQ) + return dev_err_probe(&client->dev, -EINVAL, + "external clock %lu is not supported\n", freq); + sensor->reset = devm_gpiod_get(&client->dev, "reset", GPIOD_OUT_HIGH); if (IS_ERR(sensor->reset)) { return dev_err_probe(&client->dev, PTR_ERR(sensor->reset), -- --