> +     for (i = 0; i < fd.num_entries; i++) {
> +             struct v4l2_mbus_frame_desc_entry *entry = &fd.entry[i];
> +             int source_pad;
> +
> +             source_pad = rcsi2_vc_to_pad(entry->bus.csi2.channel);
> +             if (source_pad < 0) {
> +                     dev_err(priv->dev, "Virtual Channel out of range: %u\n",
> +                             entry->bus.csi2.channel);
> +                     return -ENOSPC;

Why -ENOSPC here?

AFAIU negative source_pad here means driver internal error (frame desc
returned from rcsi2_get_remote_frame_desc() is invalid).  Then I think
error return should be -EIO.

Nikita

Reply via email to