Hi Sebastian, Thank you for the patch. On Thu, Mar 25, 2021 at 08:27:01PM +0100, Sebastian Fricke wrote: > The incentive for this patch was to overcome the inability of the > ISP device to work with resolutions greater than 4032x3024. > Increase the limit to 4416x3312 to allow higher input resolutions. > Use the old resolution to crop the input resolution down to the maximum > size for the ISP to process frames. > > Signed-off-by: Sebastian Fricke <sebastian.fricke@xxxxxxxxxx> > --- > drivers/media/platform/rockchip/rkisp1/rkisp1-common.h | 10 ++++++---- > 1 file changed, 6 insertions(+), 4 deletions(-) > > diff --git a/drivers/media/platform/rockchip/rkisp1/rkisp1-common.h b/drivers/media/platform/rockchip/rkisp1/rkisp1-common.h > index 038c303a8aed..553effda4d5e 100644 > --- a/drivers/media/platform/rockchip/rkisp1/rkisp1-common.h > +++ b/drivers/media/platform/rockchip/rkisp1/rkisp1-common.h > @@ -30,10 +30,12 @@ > #define RKISP1_ISP_SD_SINK BIT(1) > > /* min and max values for the widths and heights of the entities */ > -#define RKISP1_ISP_MAX_WIDTH 4032 > -#define RKISP1_ISP_MAX_HEIGHT 3024 > -#define RKISP1_ISP_MIN_WIDTH 32 > -#define RKISP1_ISP_MIN_HEIGHT 32 > +#define RKISP1_ISP_MAX_WIDTH 4416 > +#define RKISP1_ISP_MAX_HEIGHT 3312 > +#define RKISP1_ISP_MAX_WIDTH_PROCESSING 4032 > +#define RKISP1_ISP_MAX_HEIGHT_PROCESSING 3024 Maybe s/_PROCESSING/_CROP/, or a name more descriptive ? I would also squash this patch with 2/2, as otherwise you'll introduce a bisection breakage (applying 1/2 only will leave the driver in an incorrect state). > +#define RKISP1_ISP_MIN_WIDTH 32 > +#define RKISP1_ISP_MIN_HEIGHT 32 > > #define RKISP1_RSZ_MP_SRC_MAX_WIDTH 4416 > #define RKISP1_RSZ_MP_SRC_MAX_HEIGHT 3312 -- Regards, Laurent Pinchart