[libcamera-devel] [PATCH v2 4/6] pipeline: raspberrypi: Remove "unpacked" format penalty in mode selection

[David Plowman]

Hi Naush

Thanks for the patch.

On Fri, 22 Oct 2021 at 15:40, Naushir Patuck <naush at raspberrypi.com> wrote:
>
> With the recent change to switch to programming the sensor device directly,
> the notion of packed vs unpacked modes are not relevent, since that is a
> Unicam format construct. Remove any scoring based on packed/unpacked modes.
>
> Signed-off-by: Naushir Patuck <naush at raspberrypi.com>
> ---
>  src/libcamera/pipeline/raspberrypi/raspberrypi.cpp | 4 ----
>  1 file changed, 4 deletions(-)
>
> diff --git a/src/libcamera/pipeline/raspberrypi/raspberrypi.cpp b/src/libcamera/pipeline/raspberrypi/raspberrypi.cpp
> index 45725527d66e..2b70b877e70a 100644
> --- a/src/libcamera/pipeline/raspberrypi/raspberrypi.cpp
> +++ b/src/libcamera/pipeline/raspberrypi/raspberrypi.cpp
> @@ -109,7 +109,6 @@ V4L2SubdeviceFormat findBestMode(const SensorFormats &formatsMap, const Size &re
>  #define PENALTY_8BIT           2000.0
>  #define PENALTY_10BIT          1000.0
>  #define PENALTY_12BIT             0.0
> -#define PENALTY_UNPACKED        500.0
>
>         /* Calculate the closest/best mode from the user requested size. */
>         for (const auto &iter : formatsMap) {
> @@ -127,9 +126,6 @@ V4L2SubdeviceFormat findBestMode(const SensorFormats &formatsMap, const Size &re
>                         score += PENALTY_AR * scoreFormat(reqAr, modeAr);
>
>                         /* Add any penalties... this is not an exact science! */
> -                       if (!info.packed)
> -                               score += PENALTY_UNPACKED;
> -

Agreed, good to see this go.

Reviewed-by: David Plowman <david.plowman at raspberrypi.com>

David

>                         if (info.bitsPerPixel == 12)
>                                 score += PENALTY_12BIT;
>                         else if (info.bitsPerPixel == 10)
> --
> 2.25.1
>