media: ti-vpe: cal: Inline cal_camerarx_max_lanes() in its only caller
authorLaurent Pinchart <laurent.pinchart@ideasonboard.com>
Mon, 6 Jul 2020 18:36:37 +0000 (20:36 +0200)
committerMauro Carvalho Chehab <mchehab+huawei@kernel.org>
Sun, 19 Jul 2020 08:37:32 +0000 (10:37 +0200)
The cal_camerarx_max_lanes() function is a one-liner that has a single
caller. It doesn't improve readability. Inline it in its caller.

Signed-off-by: Laurent Pinchart <laurent.pinchart@ideasonboard.com>
Reviewed-by: Benoit Parrot <bparrot@ti.com>
Signed-off-by: Hans Verkuil <hverkuil-cisco@xs4all.nl>
Signed-off-by: Mauro Carvalho Chehab <mchehab+huawei@kernel.org>
drivers/media/platform/ti-vpe/cal.c

index 31878a3..4be1fa7 100644 (file)
@@ -481,19 +481,13 @@ static void cal_quickdump_regs(struct cal_dev *cal)
  * ------------------------------------------------------------------
  */
 
-static u32 cal_camerarx_max_lanes(struct cal_camerarx *phy)
-{
-       return phy->cal->data->camerarx[phy->instance].num_lanes;
-}
-
 static void cal_camerarx_enable(struct cal_camerarx *phy)
 {
-       u32 max_lanes;
+       u32 num_lanes = phy->cal->data->camerarx[phy->instance].num_lanes;
 
        regmap_field_write(phy->fields[F_CAMMODE], 0);
        /* Always enable all lanes at the phy control level */
-       max_lanes = (1 << cal_camerarx_max_lanes(phy)) - 1;
-       regmap_field_write(phy->fields[F_LANEENABLE], max_lanes);
+       regmap_field_write(phy->fields[F_LANEENABLE], (1 << num_lanes) - 1);
        /* F_CSI_MODE is not present on every architecture */
        if (phy->fields[F_CSI_MODE])
                regmap_field_write(phy->fields[F_CSI_MODE], 1);