linux-kernel - Re: [PATCH v7 2/2] media: i2c: add os05b10 image sensor driver

lists.openwall.net		lists / announce owl-users owl-dev john-users john-dev passwdqc-users yescrypt popa3d-users / oss-security kernel-hardening musl sabotage tlsify passwords / crypt-dev xvendor / Bugtraq Full-Disclosure linux-kernel linux-netdev linux-ext4 linux-hardening linux-cve-announce PHC
Open Source and information security mailing list archives

Hash Suite: Windows password security audit tool. GUI, reports in PDF.

[<prev] [next>] [<thread-prev] [day] [month] [year] [list]

Message-ID: <c905f11a-671f-428c-83a9-00485f507c0a@wanadoo.fr>
Date: Wed, 31 Dec 2025 14:48:08 +0100
From: Christophe JAILLET <christophe.jaillet@...adoo.fr>
To: Himanshu Bhavani <himanshu.bhavani@...iconsignals.io>, robh@...nel.org,
 krzk+dt@...nel.org, sakari.ailus@...ux.intel.com
Cc: Elgin Perumbilly <elgin.perumbilly@...iconsignals.io>,
 Vladimir Zapolskiy <vladimir.zapolskiy@...aro.org>,
 Mehdi Djait <mehdi.djait@...ux.intel.com>,
 Mauro Carvalho Chehab <mchehab@...nel.org>,
 Conor Dooley <conor+dt@...nel.org>, Hans Verkuil <hverkuil@...nel.org>,
 Hans de Goede <hansg@...nel.org>, André Apitzsch
 <git@...tzsch.eu>, Laurent Pinchart <laurent.pinchart@...asonboard.com>,
 Dongcheng Yan <dongcheng.yan@...el.com>,
 Benjamin Mugnier <benjamin.mugnier@...s.st.com>,
 Sylvain Petinot <sylvain.petinot@...s.st.com>,
 Bryan O'Donoghue <bryan.odonoghue@...aro.org>,
 Heimir Thor Sverrisson <heimir.sverrisson@...il.com>,
 Jingjing Xiong <jingjing.xiong@...el.com>, linux-media@...r.kernel.org,
 devicetree@...r.kernel.org, linux-kernel@...r.kernel.org
Subject: Re: [PATCH v7 2/2] media: i2c: add os05b10 image sensor driver

Le 31/12/2025 à 08:06, Himanshu Bhavani a écrit :
> Add a v4l2 subdevice driver for the Omnivision OS05B10 sensor.
> 
> The Omnivision OS05B10 image sensor with an active
> array size of 2592 x 1944.
> 
> The following features are supported:
> - Manual exposure an gain control support
> - vblank/hblank control support
> - Supported resolution: 2592 x 1944 @ 60fps (SBGGR10)

Hi,

2 nitpicks, should there be a v8.

> +static int os05b10_parse_endpoint(struct os05b10 *os05b10)
> +{
> +	struct v4l2_fwnode_endpoint bus_cfg = {
> +		.bus_type = V4L2_MBUS_CSI2_DPHY
> +	};
> +	unsigned long link_freq_bitmap;
> +	struct fwnode_handle *ep;
> +	int ret;
> +
> +	ep = fwnode_graph_get_endpoint_by_id(dev_fwnode(os05b10->dev), 0, 0, 0);
> +	if (!ep) {
> +		dev_err(os05b10->dev, "Failed to get next endpoint\n");
> +		return -EINVAL;
> +	}
> +
> +	ret = v4l2_fwnode_endpoint_alloc_parse(ep, &bus_cfg);
> +	fwnode_handle_put(ep);
> +	if (ret)
> +		return ret;
> +
> +	if (bus_cfg.bus.mipi_csi2.num_data_lanes != 4) {
> +		ret = dev_err_probe(os05b10->dev, -EINVAL,
> +				    "only 4 data lanes are supported\n");
> +		goto error_out;
> +	}
> +
> +	ret = v4l2_link_freq_to_bitmap(os05b10->dev, bus_cfg.link_frequencies,
> +				       bus_cfg.nr_of_link_frequencies,
> +				       link_frequencies,
> +				       ARRAY_SIZE(link_frequencies),
> +				       &link_freq_bitmap);
> +

Nitpick: blanck line can be removed.

> +	if (ret)
> +		dev_err(os05b10->dev, "only 600MHz frequency is available\n");
> +
> +error_out:
> +	v4l2_fwnode_endpoint_free(&bus_cfg);
> +
> +	return ret;
> +}

...

> +static const struct of_device_id os05b10_id[] = {
> +	{ .compatible = "ovti,os05b10" },
> +	{ /* sentinel */ },

Nitpick: No need for trailing , after a terminator.

CJ

> +};
> +
> +MODULE_DEVICE_TABLE(of, os05b10_id);
> +
> +static struct i2c_driver os05b10_driver = {
> +	.driver = {
> +		.name = "os05b10",
> +		.pm = pm_ptr(&os05b10_pm_ops),
> +		.of_match_table = os05b10_id,
> +	},
> +	.probe = os05b10_probe,
> +	.remove = os05b10_remove,
> +};
> +
> +module_i2c_driver(os05b10_driver);
> +
> +MODULE_DESCRIPTION("OS05B10 Camera Sensor Driver");
> +MODULE_AUTHOR("Himanshu Bhavani <himanshu.bhavani@...iconsignals.io>");
> +MODULE_AUTHOR("Elgin Perumbilly <elgin.perumbilly@...iconsignals.io>");
> +MODULE_LICENSE("GPL");
> --
> 2.34.1
>