media: renesas: vsp1: Simplify partition calculation
authorLaurent Pinchart <laurent.pinchart+renesas@ideasonboard.com>
Thu, 16 Nov 2023 01:00:41 +0000 (03:00 +0200)
committerLaurent Pinchart <laurent.pinchart+renesas@ideasonboard.com>
Wed, 19 Jun 2024 22:36:47 +0000 (01:36 +0300)
When calculation a partition in vsp1_pipeline_calculate_partition(),
there is no need to handle the case where the whole image is covered by
a single partition locally. In that case, the index and div_size
parameters are 0 and format->width respectively, which makes the general
code behave exactly as the special case. Drop the special case.

Signed-off-by: Laurent Pinchart <laurent.pinchart+renesas@ideasonboard.com>
Reviewed-by: Jacopo Mondi <jacopo.mondi+renesas@ideasonboard.com>
drivers/media/platform/renesas/vsp1/vsp1_pipe.c

index fc66dcfdb8389b8163addfa8527a9cb92e7ff7ec..ac52ed8f65babcef1d888441a9805a632ccabaf1 100644 (file)
@@ -495,16 +495,6 @@ void vsp1_pipeline_calculate_partition(struct vsp1_pipeline *pipe,
        format = v4l2_subdev_state_get_format(pipe->output->entity.state,
                                              RWPF_PAD_SINK);
 
-       /* A single partition simply processes the output size in full. */
-       if (pipe->partitions <= 1) {
-               window.left = 0;
-               window.width = format->width;
-
-               vsp1_pipeline_propagate_partition(pipe, partition, index,
-                                                 &window);
-               return;
-       }
-
        /* Initialise the partition with sane starting conditions. */
        window.left = index * div_size;
        window.width = div_size;