Re: [PATCH 07/17] media: camss: csid-170: support more than one lite vfe

From: Robert Foss
Date: Thu May 20 2021 - 08:39:14 EST


On Tue, 11 May 2021 at 20:08, Jonathan Marek <jonathan@xxxxxxxx> wrote:
>
> Change the IS_LITE condition so that it returns true for the second lite
> vfe found on titan 480 hardware (8250), which will have id == 3.
>
> Signed-off-by: Jonathan Marek <jonathan@xxxxxxxx>
> ---
> drivers/media/platform/qcom/camss/camss-csid-170.c | 2 +-
> 1 file changed, 1 insertion(+), 1 deletion(-)
>
> diff --git a/drivers/media/platform/qcom/camss/camss-csid-170.c b/drivers/media/platform/qcom/camss/camss-csid-170.c
> index 3958bacd7b97..af134ded241d 100644
> --- a/drivers/media/platform/qcom/camss/camss-csid-170.c
> +++ b/drivers/media/platform/qcom/camss/camss-csid-170.c
> @@ -21,7 +21,7 @@
> * interface support. As a result of that it has an
> * alternate register layout.
> */
> -#define IS_LITE (csid->id == 2 ? 1 : 0)
> +#define IS_LITE (csid->id >= 2 ? 1 : 0)
>
> #define CSID_HW_VERSION 0x0
> #define HW_VERSION_STEPPING 0

Reviewed-by: Robert Foss <robert.foss@xxxxxxxxxx>