On Thu Jun 19 23:47:09 2025 +0300, Laurent Pinchart wrote:
> The subdev core now limits the number of active routes to
> V4L2_FRAME_DESC_ENTRY_MAX. Drop the duplicated check in the ds90ub913
> driver.
> 
> Signed-off-by: Laurent Pinchart <[email protected]>
> Reviewed-by: Jacopo Mondi <[email protected]>
> Reviewed-by: Tomi Valkeinen <[email protected]>
> Reviewed-by: Lad Prabhakar <[email protected]>
> Signed-off-by: Sakari Ailus <[email protected]>
> Signed-off-by: Hans Verkuil <[email protected]>

Patch committed.

Thanks,
Hans Verkuil

 drivers/media/i2c/ds90ub913.c | 8 --------
 1 file changed, 8 deletions(-)

---

diff --git a/drivers/media/i2c/ds90ub913.c b/drivers/media/i2c/ds90ub913.c
index c34ffeae5f50..bc74499b0a96 100644
--- a/drivers/media/i2c/ds90ub913.c
+++ b/drivers/media/i2c/ds90ub913.c
@@ -337,14 +337,6 @@ static int _ub913_set_routing(struct v4l2_subdev *sd,
        unsigned int i;
        int ret;
 
-       /*
-        * Note: we can only support up to V4L2_FRAME_DESC_ENTRY_MAX, until
-        * frame desc is made dynamically allocated.
-        */
-
-       if (routing->num_routes > V4L2_FRAME_DESC_ENTRY_MAX)
-               return -EINVAL;
-
        ret = v4l2_subdev_routing_validate(sd, routing,
                                           V4L2_SUBDEV_ROUTING_ONLY_1_TO_1);
        if (ret)

Reply via email to