Re: [PATCH v4 1/1] media: v4l2-common: Add helper function media_bus_fmt_to_csi2_(bpp|dt)()
From: Sakari Ailus
Date: Wed May 13 2026 - 17:17:04 EST
Hi Frank,
On Wed, May 13, 2026 at 04:59:47PM -0400, Frank Li wrote:
> +u32 media_bus_fmt_to_csi2_dt(int bus_fmt)
> +{
> + const struct media_bus_fmt_info *info = media_bus_fmt_info_get(bus_fmt);
> +
> + return info ? info->dt : MIPI_CSI2_DT_INVALIDATE;
> +}
> +EXPORT_SYMBOL_GPL(media_bus_fmt_to_csi2_dt);
> +
> +u32 media_bus_fmt_to_csi2_bpp(int bus_fmt)
> +{
> + const struct media_bus_fmt_info *info = media_bus_fmt_info_get(bus_fmt);
> +
> + return info ? info->bpp : 0;
> +}
> +EXPORT_SYMBOL_GPL(media_bus_fmt_to_csi2_bpp);
Regarding the return values -- could we use int for that so we could use
standard error codes, e.g. -EINVAL?
--
Regards,
Sakari Ailus