From mboxrd@z Thu Jan 1 00:00:00 1970 Received: from mail-pf1-f182.google.com (mail-pf1-f182.google.com [209.85.210.182]) (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 AD44F31F9BC for ; Wed, 1 Apr 2026 18:17:08 +0000 (UTC) Authentication-Results: smtp.subspace.kernel.org; arc=none smtp.client-ip=209.85.210.182 ARC-Seal:i=1; a=rsa-sha256; d=subspace.kernel.org; s=arc-20240116; t=1775067430; cv=none; b=eiKYR4w0ZRpoLbRzMicaTbmtED2tfrLpNLn8NN/DATey+XhBRemoF3V5HiwBhHidA6TQjGa0eHo+Bzvoe9+tilI6SQJPrxM35vMAwI+BzQfnia2OBQrGUFyX9AJA1jScbMj4g9wfVaKMaxvvXofibdhV5Qa3Y+vIMa6lB5ZL7pE= ARC-Message-Signature:i=1; a=rsa-sha256; d=subspace.kernel.org; s=arc-20240116; t=1775067430; c=relaxed/simple; bh=N/EZlwfTVAXMZYR5+Cbo+7F19oN0R7nVr7sjA/fVH28=; h=From:To:Cc:Subject:Date:Message-Id:In-Reply-To:References: MIME-Version; b=NXMwaaRovapXj0cCFHC6f/hDcHzy3nSf3U7C7LQudh1JNE6Qjq+siT9GfjKCp5yleFoGjgs80m5YI4EVLX7OQS/G85rD5eJukPcIqwqn+aho5OiHM5l0ExH1dMfk5MHjfsdCCuG+Kp7lUJ7NMa1mvXi9xR/0i0or9Ny+IYISwrg= 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=c02290qF; arc=none smtp.client-ip=209.85.210.182 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="c02290qF" Received: by mail-pf1-f182.google.com with SMTP id d2e1a72fcca58-82cf636dac8so24828b3a.3 for ; Wed, 01 Apr 2026 11:17:08 -0700 (PDT) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=gmail.com; s=20251104; t=1775067428; x=1775672228; darn=vger.kernel.org; h=content-transfer-encoding:mime-version:references:in-reply-to :message-id:date:subject:cc:to:from:from:to:cc:subject:date :message-id:reply-to; bh=BAnrp5sLZFK9z49ZYni0sXiNyaOFM3jLPCzXU4u6bHg=; b=c02290qFo91/X4aXoah+xoOTxcIxGx+cDuMxt3bIziQJsG7I14rFSa3Qsw+8jXKx68 k3sKmVhIgyRDxBTWKWTQ+IAt7aIlbwtAn5qWMSZIQgQMdu+yfeTbWvcIue6GKcYN9tzJ E48XnCb8z/xMHnhe078UNsP0rUfvVsJZYTyBp5geLAgFN5eGKmk3Ei2mA+fekSNempUf TS83xp43Jo0UfOozn18hDGNFlUspC9gqTVXB85ZVdQoz++AJ8N3j5Wsu3sSNmgVmix47 RZl3X5dSLcyRC1WIJRI4jfZCoH+X9vRlbuMm2f0RW6ejvQSOKWGIxXmWyQKe2icVGizd 0kfQ== X-Google-DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=1e100.net; s=20251104; t=1775067428; x=1775672228; h=content-transfer-encoding:mime-version:references:in-reply-to :message-id:date:subject:cc:to:from:x-gm-gg:x-gm-message-state:from :to:cc:subject:date:message-id:reply-to; bh=BAnrp5sLZFK9z49ZYni0sXiNyaOFM3jLPCzXU4u6bHg=; b=NisUFr1sK45Kl7Gw3TP1wd3oGzbX5AAhYuyzFRccmu8nbaEErrXwTuM4Gdur8sgR53 IOnMi1AtUPY3Fs9Lm/C24hWO1HsmFd6wPCqSPAj6HSv0D7l5Xqqku+jynNwYj46y5EPR YH1NgomcuFTuqbPXamqEOXIhZVs0eKl6RwMoyf1LmzaYNgTUG2fawJhTidW/JE4vJUVh ib1b3cYEQZfDKtEISoBY2T1jv3EhckpzfHmA16gN2/XAfgpZr4xpWmGEqa/iQNtpd9S0 VUDdWSIMI/cQPmZydm9L7jUxEAqHGcgR1rZ22WF1quP8uBSdtfzuT/rkhKUzWq5kUkSF Dj1A== X-Forwarded-Encrypted: i=1; AJvYcCUNc+SjNKi3TrJyh6MLsR+Eb6F7h9MG/PgWgYKyuvUPtDaq+uRIYRj8vhhEPKZpWypELqG+SC0XOSufT4M=@vger.kernel.org X-Gm-Message-State: AOJu0Ywao6t47lC7nlcMRNaTR4D1O1Yw8PAFq8Cc+4m7VV1pD7muVXHW J4vTzBoLwQD5CNmAp2KoMBlptxwwEKvUWLed8qdnMeFbymbYrs0z95pw X-Gm-Gg: ATEYQzwy60vZtJdKvMz3bLHGKjYOC70TN5DYi6B6R7VKFQsNGV7X/qlie6yCclF43K/ dtT26tDqDyzBDJUyJjDYt1x7aCkfKDzJftD1omVV8ZVp1ddHpg5qkEgZSsHF+5t1G2zLXcGLyZQ jwoF7wgVPcvDtbrGF0ca6hyVpUGLHO751h0yPnOaJ6vgVILKl6LUyWyB3Dmem8oPS2rgb1XdFFR AiEbHkjR6hfzZXCWxL9+ky6IHNuz8VwjzisbCrWm6qtwL7jTo6J1MBqgyjVjJyIUi848MbVCEcP 4dVR5TfIzhFhzjmN5w/F17iQ58ZEfoJ36adRrCffEO11dKszQXv1hAhFuof0JVDaEitJhl7+nYh rXscxzNFjoO8Lo2T31VBYugVMQ3rJ3nxaWT/QL0YdRZoXLSwtxzO+4lemS+FR3OAg11rBRe3M4V rVtHMdMlz7jHmlUhClUks30MtgOi//uk7SGx7nXbikrDDi6JhQ X-Received: by 2002:a05:6a00:2395:b0:82c:a048:30da with SMTP id d2e1a72fcca58-82ce88ea317mr4687167b3a.8.1775067428037; Wed, 01 Apr 2026 11:17:08 -0700 (PDT) Received: from hu-ckantibh-hyd.qualcomm.com ([202.46.23.25]) by smtp.gmail.com with ESMTPSA id d2e1a72fcca58-82cf9ca14b3sm666249b3a.54.2026.04.01.11.17.05 (version=TLS1_3 cipher=TLS_AES_256_GCM_SHA384 bits=256/256); Wed, 01 Apr 2026 11:17:07 -0700 (PDT) From: Sanjay Chitroda X-Google-Original-From: Sanjay Chitroda To: hansg@kernel.org, sakari.ailus@linux.intel.com, mchehab@kernel.org Cc: hverkuil+cisco@kernel.org, linux-media@vger.kernel.org, linux-kernel@vger.kernel.org Subject: [PATCH 2/3] media: i2c: gc0310: use cached client and device pointers Date: Wed, 1 Apr 2026 23:46:56 +0530 Message-Id: <20260401181657.654055-3-sanjayembedded@gmail.com> X-Mailer: git-send-email 2.34.1 In-Reply-To: <20260401181657.654055-1-sanjayembedded@gmail.com> References: <20260401181657.654055-1-sanjayembedded@gmail.com> Precedence: bulk X-Mailing-List: linux-kernel@vger.kernel.org List-Id: List-Subscribe: List-Unsubscribe: MIME-Version: 1.0 Content-Transfer-Encoding: 8bit From: Sanjay Chitroda The driver was repeatedly retrieving the i2c_client using v4l2_get_subdevdata() only to access the underlying struct device. Replace this with the cached sensor->client and sensor->dev pointers, which are already available in the sensor structure. This simplifies the code, avoids redundant subdev lookups, and makes the driver more consistent with common V4L2 sensor driver patterns. No functional change intended. Signed-off-by: Sanjay Chitroda --- drivers/media/i2c/gc0310.c | 72 ++++++++++++++++++++------------------ 1 file changed, 37 insertions(+), 35 deletions(-) diff --git a/drivers/media/i2c/gc0310.c b/drivers/media/i2c/gc0310.c index 72a82ad4118a..e538479fee2e 100644 --- a/drivers/media/i2c/gc0310.c +++ b/drivers/media/i2c/gc0310.c @@ -84,6 +84,9 @@ #define to_gc0310_sensor(x) container_of(x, struct gc0310_device, sd) struct gc0310_device { + struct device *dev; + struct i2c_client *client; + struct v4l2_subdev sd; struct media_pad pad; @@ -409,28 +412,27 @@ static int gc0310_power_on(struct device *dev) static int gc0310_detect(struct gc0310_device *sensor) { - struct i2c_client *client = v4l2_get_subdevdata(&sensor->sd); u64 val; int ret; - if (!i2c_check_functionality(client->adapter, I2C_FUNC_I2C)) + if (!i2c_check_functionality(sensor->client->adapter, I2C_FUNC_I2C)) return -ENODEV; ret = cci_read(sensor->regmap, GC0310_SC_CMMN_CHIP_ID_REG, &val, NULL); if (ret < 0) { - dev_err(&client->dev, "read sensor_id failed: %d\n", ret); + dev_err(sensor->dev, "read sensor_id failed: %d\n", ret); return -ENODEV; } - dev_dbg(&client->dev, "sensor ID = 0x%llx\n", val); + dev_dbg(sensor->dev, "sensor ID = 0x%llx\n", val); if (val != GC0310_ID) { - dev_err(&client->dev, "sensor ID error, read id = 0x%llx, target id = 0x%x\n", + dev_err(sensor->dev, "sensor ID error, read id = 0x%llx, target id = 0x%x\n", val, GC0310_ID); return -ENODEV; } - dev_dbg(&client->dev, "detect gc0310 success\n"); + dev_dbg(sensor->dev, "detect gc0310 success\n"); return 0; } @@ -440,10 +442,9 @@ static int gc0310_enable_streams(struct v4l2_subdev *sd, u32 pad, u64 streams_mask) { struct gc0310_device *sensor = to_gc0310_sensor(sd); - struct i2c_client *client = v4l2_get_subdevdata(sd); int ret; - ret = pm_runtime_resume_and_get(&client->dev); + ret = pm_runtime_resume_and_get(sensor->dev); if (ret) return ret; @@ -474,7 +475,7 @@ static int gc0310_enable_streams(struct v4l2_subdev *sd, error_power_down: if (ret) - pm_runtime_put(&client->dev); + pm_runtime_put(sensor->dev); return ret; } @@ -484,7 +485,6 @@ static int gc0310_disable_streams(struct v4l2_subdev *sd, u32 pad, u64 streams_mask) { struct gc0310_device *sensor = to_gc0310_sensor(sd); - struct i2c_client *client = v4l2_get_subdevdata(sd); int ret = 0; cci_write(sensor->regmap, GC0310_RESET_RELATED_REG, @@ -494,7 +494,7 @@ static int gc0310_disable_streams(struct v4l2_subdev *sd, cci_write(sensor->regmap, GC0310_RESET_RELATED_REG, GC0310_REGISTER_PAGE_0, &ret); - pm_runtime_put(&client->dev); + pm_runtime_put(sensor->dev); return ret; } @@ -559,7 +559,6 @@ static const struct v4l2_subdev_internal_ops gc0310_internal_ops = { static int gc0310_init_controls(struct gc0310_device *sensor) { - struct i2c_client *client = v4l2_get_subdevdata(&sensor->sd); struct v4l2_ctrl_handler *hdl = &sensor->ctrls.handler; struct v4l2_fwnode_device_properties props; int exp_max, ret; @@ -597,7 +596,7 @@ static int gc0310_init_controls(struct gc0310_device *sensor) GC0310_H_BLANK_DEFAULT, 1, GC0310_H_BLANK_DEFAULT); - ret = v4l2_fwnode_device_parse(&client->dev, &props); + ret = v4l2_fwnode_device_parse(sensor->dev, &props); if (ret) return ret; @@ -621,10 +620,10 @@ static void gc0310_remove(struct i2c_client *client) v4l2_subdev_cleanup(sd); media_entity_cleanup(&sensor->sd.entity); v4l2_ctrl_handler_free(&sensor->ctrls.handler); - pm_runtime_disable(&client->dev); - if (!pm_runtime_status_suspended(&client->dev)) { - gc0310_power_off(&client->dev); - pm_runtime_set_suspended(&client->dev); + pm_runtime_disable(sensor->dev); + if (!pm_runtime_status_suspended(sensor->dev)) { + gc0310_power_off(sensor->dev); + pm_runtime_set_suspended(sensor->dev); } } @@ -695,15 +694,18 @@ static int gc0310_probe(struct i2c_client *client) if (!sensor) return -ENOMEM; - sensor->reset = devm_gpiod_get(&client->dev, "reset", GPIOD_OUT_HIGH); + sensor->client = client; + sensor->dev = &client->dev; + + sensor->reset = devm_gpiod_get(sensor->dev, "reset", GPIOD_OUT_HIGH); if (IS_ERR(sensor->reset)) { - return dev_err_probe(&client->dev, PTR_ERR(sensor->reset), + return dev_err_probe(sensor->dev, PTR_ERR(sensor->reset), "getting reset GPIO\n"); } - sensor->powerdown = devm_gpiod_get(&client->dev, "powerdown", GPIOD_OUT_HIGH); + sensor->powerdown = devm_gpiod_get(sensor->dev, "powerdown", GPIOD_OUT_HIGH); if (IS_ERR(sensor->powerdown)) { - return dev_err_probe(&client->dev, PTR_ERR(sensor->powerdown), + return dev_err_probe(sensor->dev, PTR_ERR(sensor->powerdown), "getting powerdown GPIO\n"); } @@ -713,11 +715,11 @@ static int gc0310_probe(struct i2c_client *client) if (IS_ERR(sensor->regmap)) return PTR_ERR(sensor->regmap); - gc0310_power_on(&client->dev); + gc0310_power_on(sensor->dev); - pm_runtime_set_active(&client->dev); - pm_runtime_get_noresume(&client->dev); - pm_runtime_enable(&client->dev); + pm_runtime_set_active(sensor->dev); + pm_runtime_get_noresume(sensor->dev); + pm_runtime_enable(sensor->dev); ret = gc0310_detect(sensor); if (ret) @@ -734,34 +736,34 @@ static int gc0310_probe(struct i2c_client *client) ret = media_entity_pads_init(&sensor->sd.entity, 1, &sensor->pad); if (ret) { - dev_err_probe(&client->dev, ret, "failed to init entity pads\n"); + dev_err_probe(sensor->dev, ret, "failed to init entity pads\n"); goto error_handler_free; } sensor->sd.state_lock = sensor->ctrls.handler.lock; ret = v4l2_subdev_init_finalize(&sensor->sd); if (ret) { - dev_err_probe(&client->dev, ret, "subdev init error\n"); + dev_err_probe(sensor->dev, ret, "subdev init error\n"); goto error_media_entity; } ret = v4l2_async_register_subdev_sensor(&sensor->sd); if (ret) { - dev_err_probe(&client->dev, ret, + dev_err_probe(sensor->dev, ret, "failed to register gc0310 sub-device\n"); goto error_subdev_cleanup; } - pm_runtime_set_autosuspend_delay(&client->dev, 1000); - pm_runtime_use_autosuspend(&client->dev); - pm_runtime_put_autosuspend(&client->dev); + pm_runtime_set_autosuspend_delay(sensor->dev, 1000); + pm_runtime_use_autosuspend(sensor->dev); + pm_runtime_put_autosuspend(sensor->dev); return 0; error_subdev_cleanup: v4l2_subdev_cleanup(&sensor->sd); - pm_runtime_disable(&client->dev); - pm_runtime_set_suspended(&client->dev); + pm_runtime_disable(sensor->dev); + pm_runtime_set_suspended(sensor->dev); error_media_entity: media_entity_cleanup(&sensor->sd.entity); @@ -770,8 +772,8 @@ static int gc0310_probe(struct i2c_client *client) v4l2_ctrl_handler_free(&sensor->ctrls.handler); error_power_off: - pm_runtime_put_noidle(&client->dev); - gc0310_power_off(&client->dev); + pm_runtime_put_noidle(sensor->dev); + gc0310_power_off(sensor->dev); return ret; } -- 2.34.1