* [PATCH] media: rkisp1: Allow higher input resolution
@ 2024-02-17 18:51 ` Ondřej Jirman
0 siblings, 0 replies; 27+ messages in thread
From: Ondřej Jirman @ 2024-02-17 18:51 UTC (permalink / raw)
To: linux-kernel
Cc: Ondrej Jirman, Dafna Hirschfeld, Laurent Pinchart,
Mauro Carvalho Chehab, Heiko Stuebner,
open list:ROCKCHIP ISP V1 DRIVER,
open list:ROCKCHIP ISP V1 DRIVER,
moderated list:ARM/Rockchip SoC support
From: Ondrej Jirman <megi@xff.cz>
In BSP driver, it is allowed, and it works in practice. Tested on
Pinephone Pro/RK3399 with IMX258 at full res.
Signed-off-by: Ondrej Jirman <megi@xff.cz>
---
drivers/media/platform/rockchip/rkisp1/rkisp1-common.h | 4 ++--
1 file changed, 2 insertions(+), 2 deletions(-)
diff --git a/drivers/media/platform/rockchip/rkisp1/rkisp1-common.h b/drivers/media/platform/rockchip/rkisp1/rkisp1-common.h
index 4b6b28c05b89..74098ddbeeb3 100644
--- a/drivers/media/platform/rockchip/rkisp1/rkisp1-common.h
+++ b/drivers/media/platform/rockchip/rkisp1/rkisp1-common.h
@@ -33,8 +33,8 @@ struct dentry;
#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_MAX_WIDTH 4416
+#define RKISP1_ISP_MAX_HEIGHT 3312
#define RKISP1_ISP_MIN_WIDTH 32
#define RKISP1_ISP_MIN_HEIGHT 32
--
2.43.0
_______________________________________________
Linux-rockchip mailing list
Linux-rockchip@lists.infradead.org
http://lists.infradead.org/mailman/listinfo/linux-rockchip
^ permalink raw reply related [flat|nested] 27+ messages in thread
* [PATCH] media: rkisp1: Allow higher input resolution
@ 2024-02-17 18:51 ` Ondřej Jirman
0 siblings, 0 replies; 27+ messages in thread
From: Ondřej Jirman @ 2024-02-17 18:51 UTC (permalink / raw)
To: linux-kernel
Cc: Ondrej Jirman, Dafna Hirschfeld, Laurent Pinchart,
Mauro Carvalho Chehab, Heiko Stuebner,
open list:ROCKCHIP ISP V1 DRIVER,
open list:ROCKCHIP ISP V1 DRIVER,
moderated list:ARM/Rockchip SoC support
From: Ondrej Jirman <megi@xff.cz>
In BSP driver, it is allowed, and it works in practice. Tested on
Pinephone Pro/RK3399 with IMX258 at full res.
Signed-off-by: Ondrej Jirman <megi@xff.cz>
---
drivers/media/platform/rockchip/rkisp1/rkisp1-common.h | 4 ++--
1 file changed, 2 insertions(+), 2 deletions(-)
diff --git a/drivers/media/platform/rockchip/rkisp1/rkisp1-common.h b/drivers/media/platform/rockchip/rkisp1/rkisp1-common.h
index 4b6b28c05b89..74098ddbeeb3 100644
--- a/drivers/media/platform/rockchip/rkisp1/rkisp1-common.h
+++ b/drivers/media/platform/rockchip/rkisp1/rkisp1-common.h
@@ -33,8 +33,8 @@ struct dentry;
#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_MAX_WIDTH 4416
+#define RKISP1_ISP_MAX_HEIGHT 3312
#define RKISP1_ISP_MIN_WIDTH 32
#define RKISP1_ISP_MIN_HEIGHT 32
--
2.43.0
_______________________________________________
linux-arm-kernel mailing list
linux-arm-kernel@lists.infradead.org
http://lists.infradead.org/mailman/listinfo/linux-arm-kernel
^ permalink raw reply related [flat|nested] 27+ messages in thread
* [PATCH] media: rkisp1: Allow higher input resolution
@ 2024-02-17 18:51 ` Ondřej Jirman
0 siblings, 0 replies; 27+ messages in thread
From: Ondřej Jirman @ 2024-02-17 18:51 UTC (permalink / raw)
To: linux-kernel
Cc: Ondrej Jirman, Dafna Hirschfeld, Laurent Pinchart,
Mauro Carvalho Chehab, Heiko Stuebner,
open list:ROCKCHIP ISP V1 DRIVER,
open list:ROCKCHIP ISP V1 DRIVER,
moderated list:ARM/Rockchip SoC support
From: Ondrej Jirman <megi@xff.cz>
In BSP driver, it is allowed, and it works in practice. Tested on
Pinephone Pro/RK3399 with IMX258 at full res.
Signed-off-by: Ondrej Jirman <megi@xff.cz>
---
drivers/media/platform/rockchip/rkisp1/rkisp1-common.h | 4 ++--
1 file changed, 2 insertions(+), 2 deletions(-)
diff --git a/drivers/media/platform/rockchip/rkisp1/rkisp1-common.h b/drivers/media/platform/rockchip/rkisp1/rkisp1-common.h
index 4b6b28c05b89..74098ddbeeb3 100644
--- a/drivers/media/platform/rockchip/rkisp1/rkisp1-common.h
+++ b/drivers/media/platform/rockchip/rkisp1/rkisp1-common.h
@@ -33,8 +33,8 @@ struct dentry;
#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_MAX_WIDTH 4416
+#define RKISP1_ISP_MAX_HEIGHT 3312
#define RKISP1_ISP_MIN_WIDTH 32
#define RKISP1_ISP_MIN_HEIGHT 32
--
2.43.0
^ permalink raw reply related [flat|nested] 27+ messages in thread
* Re: [PATCH] media: rkisp1: Allow higher input resolution
2024-02-17 18:51 ` Ondřej Jirman
(?)
@ 2024-02-18 20:59 ` Laurent Pinchart
-1 siblings, 0 replies; 27+ messages in thread
From: Laurent Pinchart @ 2024-02-18 20:59 UTC (permalink / raw)
To: Ondřej Jirman
Cc: linux-kernel, Dafna Hirschfeld, Mauro Carvalho Chehab,
Heiko Stuebner, open list:ROCKCHIP ISP V1 DRIVER,
open list:ROCKCHIP ISP V1 DRIVER,
moderated list:ARM/Rockchip SoC support, Paul Elder, Umang Jain
Hi Ondrej,
(CC'ing Paul and Umang)
Thank you for the patch.
On Sat, Feb 17, 2024 at 07:51:58PM +0100, Ondřej Jirman wrote:
> From: Ondrej Jirman <megi@xff.cz>
>
> In BSP driver, it is allowed, and it works in practice. Tested on
> Pinephone Pro/RK3399 with IMX258 at full res.
Paul, Umang, do I recall correctly that you have a similar change ?
Could you review and test this (especially on the i.MX8MP) ?
> Signed-off-by: Ondrej Jirman <megi@xff.cz>
> ---
> drivers/media/platform/rockchip/rkisp1/rkisp1-common.h | 4 ++--
> 1 file changed, 2 insertions(+), 2 deletions(-)
>
> diff --git a/drivers/media/platform/rockchip/rkisp1/rkisp1-common.h b/drivers/media/platform/rockchip/rkisp1/rkisp1-common.h
> index 4b6b28c05b89..74098ddbeeb3 100644
> --- a/drivers/media/platform/rockchip/rkisp1/rkisp1-common.h
> +++ b/drivers/media/platform/rockchip/rkisp1/rkisp1-common.h
> @@ -33,8 +33,8 @@ struct dentry;
> #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_MAX_WIDTH 4416
> +#define RKISP1_ISP_MAX_HEIGHT 3312
> #define RKISP1_ISP_MIN_WIDTH 32
> #define RKISP1_ISP_MIN_HEIGHT 32
>
--
Regards,
Laurent Pinchart
_______________________________________________
Linux-rockchip mailing list
Linux-rockchip@lists.infradead.org
http://lists.infradead.org/mailman/listinfo/linux-rockchip
^ permalink raw reply [flat|nested] 27+ messages in thread
* Re: [PATCH] media: rkisp1: Allow higher input resolution
@ 2024-02-18 20:59 ` Laurent Pinchart
0 siblings, 0 replies; 27+ messages in thread
From: Laurent Pinchart @ 2024-02-18 20:59 UTC (permalink / raw)
To: Ondřej Jirman
Cc: linux-kernel, Dafna Hirschfeld, Mauro Carvalho Chehab,
Heiko Stuebner, open list:ROCKCHIP ISP V1 DRIVER,
open list:ROCKCHIP ISP V1 DRIVER,
moderated list:ARM/Rockchip SoC support, Paul Elder, Umang Jain
Hi Ondrej,
(CC'ing Paul and Umang)
Thank you for the patch.
On Sat, Feb 17, 2024 at 07:51:58PM +0100, Ondřej Jirman wrote:
> From: Ondrej Jirman <megi@xff.cz>
>
> In BSP driver, it is allowed, and it works in practice. Tested on
> Pinephone Pro/RK3399 with IMX258 at full res.
Paul, Umang, do I recall correctly that you have a similar change ?
Could you review and test this (especially on the i.MX8MP) ?
> Signed-off-by: Ondrej Jirman <megi@xff.cz>
> ---
> drivers/media/platform/rockchip/rkisp1/rkisp1-common.h | 4 ++--
> 1 file changed, 2 insertions(+), 2 deletions(-)
>
> diff --git a/drivers/media/platform/rockchip/rkisp1/rkisp1-common.h b/drivers/media/platform/rockchip/rkisp1/rkisp1-common.h
> index 4b6b28c05b89..74098ddbeeb3 100644
> --- a/drivers/media/platform/rockchip/rkisp1/rkisp1-common.h
> +++ b/drivers/media/platform/rockchip/rkisp1/rkisp1-common.h
> @@ -33,8 +33,8 @@ struct dentry;
> #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_MAX_WIDTH 4416
> +#define RKISP1_ISP_MAX_HEIGHT 3312
> #define RKISP1_ISP_MIN_WIDTH 32
> #define RKISP1_ISP_MIN_HEIGHT 32
>
--
Regards,
Laurent Pinchart
_______________________________________________
linux-arm-kernel mailing list
linux-arm-kernel@lists.infradead.org
http://lists.infradead.org/mailman/listinfo/linux-arm-kernel
^ permalink raw reply [flat|nested] 27+ messages in thread
* Re: [PATCH] media: rkisp1: Allow higher input resolution
@ 2024-02-18 20:59 ` Laurent Pinchart
0 siblings, 0 replies; 27+ messages in thread
From: Laurent Pinchart @ 2024-02-18 20:59 UTC (permalink / raw)
To: Ondřej Jirman
Cc: linux-kernel, Dafna Hirschfeld, Mauro Carvalho Chehab,
Heiko Stuebner, open list:ROCKCHIP ISP V1 DRIVER,
open list:ROCKCHIP ISP V1 DRIVER,
moderated list:ARM/Rockchip SoC support, Paul Elder, Umang Jain
Hi Ondrej,
(CC'ing Paul and Umang)
Thank you for the patch.
On Sat, Feb 17, 2024 at 07:51:58PM +0100, Ondřej Jirman wrote:
> From: Ondrej Jirman <megi@xff.cz>
>
> In BSP driver, it is allowed, and it works in practice. Tested on
> Pinephone Pro/RK3399 with IMX258 at full res.
Paul, Umang, do I recall correctly that you have a similar change ?
Could you review and test this (especially on the i.MX8MP) ?
> Signed-off-by: Ondrej Jirman <megi@xff.cz>
> ---
> drivers/media/platform/rockchip/rkisp1/rkisp1-common.h | 4 ++--
> 1 file changed, 2 insertions(+), 2 deletions(-)
>
> diff --git a/drivers/media/platform/rockchip/rkisp1/rkisp1-common.h b/drivers/media/platform/rockchip/rkisp1/rkisp1-common.h
> index 4b6b28c05b89..74098ddbeeb3 100644
> --- a/drivers/media/platform/rockchip/rkisp1/rkisp1-common.h
> +++ b/drivers/media/platform/rockchip/rkisp1/rkisp1-common.h
> @@ -33,8 +33,8 @@ struct dentry;
> #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_MAX_WIDTH 4416
> +#define RKISP1_ISP_MAX_HEIGHT 3312
> #define RKISP1_ISP_MIN_WIDTH 32
> #define RKISP1_ISP_MIN_HEIGHT 32
>
--
Regards,
Laurent Pinchart
^ permalink raw reply [flat|nested] 27+ messages in thread
* Re: [PATCH] media: rkisp1: Allow higher input resolution
2024-02-18 20:59 ` Laurent Pinchart
(?)
@ 2024-02-18 21:01 ` Ondřej Jirman
-1 siblings, 0 replies; 27+ messages in thread
From: Ondřej Jirman @ 2024-02-18 21:01 UTC (permalink / raw)
To: Laurent Pinchart
Cc: linux-kernel, Dafna Hirschfeld, Mauro Carvalho Chehab,
Heiko Stuebner, open list:ROCKCHIP ISP V1 DRIVER,
open list:ROCKCHIP ISP V1 DRIVER,
moderated list:ARM/Rockchip SoC support, Paul Elder, Umang Jain
On Sun, Feb 18, 2024 at 10:59:08PM +0200, Laurent Pinchart wrote:
> Hi Ondrej,
>
> (CC'ing Paul and Umang)
>
> Thank you for the patch.
>
> On Sat, Feb 17, 2024 at 07:51:58PM +0100, Ondřej Jirman wrote:
> > From: Ondrej Jirman <megi@xff.cz>
> >
> > In BSP driver, it is allowed, and it works in practice. Tested on
> > Pinephone Pro/RK3399 with IMX258 at full res.
>
> Paul, Umang, do I recall correctly that you have a similar change ?
> Could you review and test this (especially on the i.MX8MP) ?
It's also a limit from the datasheet, so the change should not be that
controversial:
https://megous.com/dl/tmp/d2b333043ecebaf3.png
(so that it doesn't sound like I just copied the BSP values)
regards,
o.
> > Signed-off-by: Ondrej Jirman <megi@xff.cz>
> > ---
> > drivers/media/platform/rockchip/rkisp1/rkisp1-common.h | 4 ++--
> > 1 file changed, 2 insertions(+), 2 deletions(-)
> >
> > diff --git a/drivers/media/platform/rockchip/rkisp1/rkisp1-common.h b/drivers/media/platform/rockchip/rkisp1/rkisp1-common.h
> > index 4b6b28c05b89..74098ddbeeb3 100644
> > --- a/drivers/media/platform/rockchip/rkisp1/rkisp1-common.h
> > +++ b/drivers/media/platform/rockchip/rkisp1/rkisp1-common.h
> > @@ -33,8 +33,8 @@ struct dentry;
> > #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_MAX_WIDTH 4416
> > +#define RKISP1_ISP_MAX_HEIGHT 3312
> > #define RKISP1_ISP_MIN_WIDTH 32
> > #define RKISP1_ISP_MIN_HEIGHT 32
> >
>
> --
> Regards,
>
> Laurent Pinchart
^ permalink raw reply [flat|nested] 27+ messages in thread
* Re: [PATCH] media: rkisp1: Allow higher input resolution
@ 2024-02-18 21:01 ` Ondřej Jirman
0 siblings, 0 replies; 27+ messages in thread
From: Ondřej Jirman @ 2024-02-18 21:01 UTC (permalink / raw)
To: Laurent Pinchart
Cc: linux-kernel, Dafna Hirschfeld, Mauro Carvalho Chehab,
Heiko Stuebner, open list:ROCKCHIP ISP V1 DRIVER,
open list:ROCKCHIP ISP V1 DRIVER,
moderated list:ARM/Rockchip SoC support, Paul Elder, Umang Jain
On Sun, Feb 18, 2024 at 10:59:08PM +0200, Laurent Pinchart wrote:
> Hi Ondrej,
>
> (CC'ing Paul and Umang)
>
> Thank you for the patch.
>
> On Sat, Feb 17, 2024 at 07:51:58PM +0100, Ondřej Jirman wrote:
> > From: Ondrej Jirman <megi@xff.cz>
> >
> > In BSP driver, it is allowed, and it works in practice. Tested on
> > Pinephone Pro/RK3399 with IMX258 at full res.
>
> Paul, Umang, do I recall correctly that you have a similar change ?
> Could you review and test this (especially on the i.MX8MP) ?
It's also a limit from the datasheet, so the change should not be that
controversial:
https://megous.com/dl/tmp/d2b333043ecebaf3.png
(so that it doesn't sound like I just copied the BSP values)
regards,
o.
> > Signed-off-by: Ondrej Jirman <megi@xff.cz>
> > ---
> > drivers/media/platform/rockchip/rkisp1/rkisp1-common.h | 4 ++--
> > 1 file changed, 2 insertions(+), 2 deletions(-)
> >
> > diff --git a/drivers/media/platform/rockchip/rkisp1/rkisp1-common.h b/drivers/media/platform/rockchip/rkisp1/rkisp1-common.h
> > index 4b6b28c05b89..74098ddbeeb3 100644
> > --- a/drivers/media/platform/rockchip/rkisp1/rkisp1-common.h
> > +++ b/drivers/media/platform/rockchip/rkisp1/rkisp1-common.h
> > @@ -33,8 +33,8 @@ struct dentry;
> > #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_MAX_WIDTH 4416
> > +#define RKISP1_ISP_MAX_HEIGHT 3312
> > #define RKISP1_ISP_MIN_WIDTH 32
> > #define RKISP1_ISP_MIN_HEIGHT 32
> >
>
> --
> Regards,
>
> Laurent Pinchart
_______________________________________________
Linux-rockchip mailing list
Linux-rockchip@lists.infradead.org
http://lists.infradead.org/mailman/listinfo/linux-rockchip
^ permalink raw reply [flat|nested] 27+ messages in thread
* Re: [PATCH] media: rkisp1: Allow higher input resolution
@ 2024-02-18 21:01 ` Ondřej Jirman
0 siblings, 0 replies; 27+ messages in thread
From: Ondřej Jirman @ 2024-02-18 21:01 UTC (permalink / raw)
To: Laurent Pinchart
Cc: linux-kernel, Dafna Hirschfeld, Mauro Carvalho Chehab,
Heiko Stuebner, open list:ROCKCHIP ISP V1 DRIVER,
open list:ROCKCHIP ISP V1 DRIVER,
moderated list:ARM/Rockchip SoC support, Paul Elder, Umang Jain
On Sun, Feb 18, 2024 at 10:59:08PM +0200, Laurent Pinchart wrote:
> Hi Ondrej,
>
> (CC'ing Paul and Umang)
>
> Thank you for the patch.
>
> On Sat, Feb 17, 2024 at 07:51:58PM +0100, Ondřej Jirman wrote:
> > From: Ondrej Jirman <megi@xff.cz>
> >
> > In BSP driver, it is allowed, and it works in practice. Tested on
> > Pinephone Pro/RK3399 with IMX258 at full res.
>
> Paul, Umang, do I recall correctly that you have a similar change ?
> Could you review and test this (especially on the i.MX8MP) ?
It's also a limit from the datasheet, so the change should not be that
controversial:
https://megous.com/dl/tmp/d2b333043ecebaf3.png
(so that it doesn't sound like I just copied the BSP values)
regards,
o.
> > Signed-off-by: Ondrej Jirman <megi@xff.cz>
> > ---
> > drivers/media/platform/rockchip/rkisp1/rkisp1-common.h | 4 ++--
> > 1 file changed, 2 insertions(+), 2 deletions(-)
> >
> > diff --git a/drivers/media/platform/rockchip/rkisp1/rkisp1-common.h b/drivers/media/platform/rockchip/rkisp1/rkisp1-common.h
> > index 4b6b28c05b89..74098ddbeeb3 100644
> > --- a/drivers/media/platform/rockchip/rkisp1/rkisp1-common.h
> > +++ b/drivers/media/platform/rockchip/rkisp1/rkisp1-common.h
> > @@ -33,8 +33,8 @@ struct dentry;
> > #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_MAX_WIDTH 4416
> > +#define RKISP1_ISP_MAX_HEIGHT 3312
> > #define RKISP1_ISP_MIN_WIDTH 32
> > #define RKISP1_ISP_MIN_HEIGHT 32
> >
>
> --
> Regards,
>
> Laurent Pinchart
_______________________________________________
linux-arm-kernel mailing list
linux-arm-kernel@lists.infradead.org
http://lists.infradead.org/mailman/listinfo/linux-arm-kernel
^ permalink raw reply [flat|nested] 27+ messages in thread
* Re: [PATCH] media: rkisp1: Allow higher input resolution
2024-02-18 21:01 ` Ondřej Jirman
(?)
@ 2024-02-19 2:09 ` Adam Ford
-1 siblings, 0 replies; 27+ messages in thread
From: Adam Ford @ 2024-02-19 2:09 UTC (permalink / raw)
To: Ondřej Jirman, Laurent Pinchart, linux-kernel,
Dafna Hirschfeld, Mauro Carvalho Chehab, Heiko Stuebner,
open list:ROCKCHIP ISP V1 DRIVER,
open list:ROCKCHIP ISP V1 DRIVER,
moderated list:ARM/Rockchip SoC support, Paul Elder, Umang Jain
On Sun, Feb 18, 2024 at 3:02 PM Ondřej Jirman <megi@xff.cz> wrote:
>
> On Sun, Feb 18, 2024 at 10:59:08PM +0200, Laurent Pinchart wrote:
> > Hi Ondrej,
> >
> > (CC'ing Paul and Umang)
> >
> > Thank you for the patch.
> >
> > On Sat, Feb 17, 2024 at 07:51:58PM +0100, Ondřej Jirman wrote:
> > > From: Ondrej Jirman <megi@xff.cz>
> > >
> > > In BSP driver, it is allowed, and it works in practice. Tested on
> > > Pinephone Pro/RK3399 with IMX258 at full res.
> >
> > Paul, Umang, do I recall correctly that you have a similar change ?
> > Could you review and test this (especially on the i.MX8MP) ?
>
> It's also a limit from the datasheet, so the change should not be that
> controversial:
>
> https://megous.com/dl/tmp/d2b333043ecebaf3.png
>
> (so that it doesn't sound like I just copied the BSP values)
>
From what I see in the i.MX8M Plus reference manual, it has a max
resolution of 4096x3072, so it might be necessary to move this off
from a #define into a structure that varies by product family.
adam
> regards,
> o.
>
> > > Signed-off-by: Ondrej Jirman <megi@xff.cz>
> > > ---
> > > drivers/media/platform/rockchip/rkisp1/rkisp1-common.h | 4 ++--
> > > 1 file changed, 2 insertions(+), 2 deletions(-)
> > >
> > > diff --git a/drivers/media/platform/rockchip/rkisp1/rkisp1-common.h b/drivers/media/platform/rockchip/rkisp1/rkisp1-common.h
> > > index 4b6b28c05b89..74098ddbeeb3 100644
> > > --- a/drivers/media/platform/rockchip/rkisp1/rkisp1-common.h
> > > +++ b/drivers/media/platform/rockchip/rkisp1/rkisp1-common.h
> > > @@ -33,8 +33,8 @@ struct dentry;
> > > #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_MAX_WIDTH 4416
> > > +#define RKISP1_ISP_MAX_HEIGHT 3312
> > > #define RKISP1_ISP_MIN_WIDTH 32
> > > #define RKISP1_ISP_MIN_HEIGHT 32
> > >
> >
> > --
> > Regards,
> >
> > Laurent Pinchart
>
^ permalink raw reply [flat|nested] 27+ messages in thread
* Re: [PATCH] media: rkisp1: Allow higher input resolution
@ 2024-02-19 2:09 ` Adam Ford
0 siblings, 0 replies; 27+ messages in thread
From: Adam Ford @ 2024-02-19 2:09 UTC (permalink / raw)
To: Ondřej Jirman, Laurent Pinchart, linux-kernel,
Dafna Hirschfeld, Mauro Carvalho Chehab, Heiko Stuebner,
open list:ROCKCHIP ISP V1 DRIVER,
open list:ROCKCHIP ISP V1 DRIVER,
moderated list:ARM/Rockchip SoC support, Paul Elder, Umang Jain
On Sun, Feb 18, 2024 at 3:02 PM Ondřej Jirman <megi@xff.cz> wrote:
>
> On Sun, Feb 18, 2024 at 10:59:08PM +0200, Laurent Pinchart wrote:
> > Hi Ondrej,
> >
> > (CC'ing Paul and Umang)
> >
> > Thank you for the patch.
> >
> > On Sat, Feb 17, 2024 at 07:51:58PM +0100, Ondřej Jirman wrote:
> > > From: Ondrej Jirman <megi@xff.cz>
> > >
> > > In BSP driver, it is allowed, and it works in practice. Tested on
> > > Pinephone Pro/RK3399 with IMX258 at full res.
> >
> > Paul, Umang, do I recall correctly that you have a similar change ?
> > Could you review and test this (especially on the i.MX8MP) ?
>
> It's also a limit from the datasheet, so the change should not be that
> controversial:
>
> https://megous.com/dl/tmp/d2b333043ecebaf3.png
>
> (so that it doesn't sound like I just copied the BSP values)
>
From what I see in the i.MX8M Plus reference manual, it has a max
resolution of 4096x3072, so it might be necessary to move this off
from a #define into a structure that varies by product family.
adam
> regards,
> o.
>
> > > Signed-off-by: Ondrej Jirman <megi@xff.cz>
> > > ---
> > > drivers/media/platform/rockchip/rkisp1/rkisp1-common.h | 4 ++--
> > > 1 file changed, 2 insertions(+), 2 deletions(-)
> > >
> > > diff --git a/drivers/media/platform/rockchip/rkisp1/rkisp1-common.h b/drivers/media/platform/rockchip/rkisp1/rkisp1-common.h
> > > index 4b6b28c05b89..74098ddbeeb3 100644
> > > --- a/drivers/media/platform/rockchip/rkisp1/rkisp1-common.h
> > > +++ b/drivers/media/platform/rockchip/rkisp1/rkisp1-common.h
> > > @@ -33,8 +33,8 @@ struct dentry;
> > > #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_MAX_WIDTH 4416
> > > +#define RKISP1_ISP_MAX_HEIGHT 3312
> > > #define RKISP1_ISP_MIN_WIDTH 32
> > > #define RKISP1_ISP_MIN_HEIGHT 32
> > >
> >
> > --
> > Regards,
> >
> > Laurent Pinchart
>
_______________________________________________
Linux-rockchip mailing list
Linux-rockchip@lists.infradead.org
http://lists.infradead.org/mailman/listinfo/linux-rockchip
^ permalink raw reply [flat|nested] 27+ messages in thread
* Re: [PATCH] media: rkisp1: Allow higher input resolution
@ 2024-02-19 2:09 ` Adam Ford
0 siblings, 0 replies; 27+ messages in thread
From: Adam Ford @ 2024-02-19 2:09 UTC (permalink / raw)
To: Ondřej Jirman, Laurent Pinchart, linux-kernel,
Dafna Hirschfeld, Mauro Carvalho Chehab, Heiko Stuebner,
open list:ROCKCHIP ISP V1 DRIVER,
open list:ROCKCHIP ISP V1 DRIVER,
moderated list:ARM/Rockchip SoC support, Paul Elder, Umang Jain
On Sun, Feb 18, 2024 at 3:02 PM Ondřej Jirman <megi@xff.cz> wrote:
>
> On Sun, Feb 18, 2024 at 10:59:08PM +0200, Laurent Pinchart wrote:
> > Hi Ondrej,
> >
> > (CC'ing Paul and Umang)
> >
> > Thank you for the patch.
> >
> > On Sat, Feb 17, 2024 at 07:51:58PM +0100, Ondřej Jirman wrote:
> > > From: Ondrej Jirman <megi@xff.cz>
> > >
> > > In BSP driver, it is allowed, and it works in practice. Tested on
> > > Pinephone Pro/RK3399 with IMX258 at full res.
> >
> > Paul, Umang, do I recall correctly that you have a similar change ?
> > Could you review and test this (especially on the i.MX8MP) ?
>
> It's also a limit from the datasheet, so the change should not be that
> controversial:
>
> https://megous.com/dl/tmp/d2b333043ecebaf3.png
>
> (so that it doesn't sound like I just copied the BSP values)
>
From what I see in the i.MX8M Plus reference manual, it has a max
resolution of 4096x3072, so it might be necessary to move this off
from a #define into a structure that varies by product family.
adam
> regards,
> o.
>
> > > Signed-off-by: Ondrej Jirman <megi@xff.cz>
> > > ---
> > > drivers/media/platform/rockchip/rkisp1/rkisp1-common.h | 4 ++--
> > > 1 file changed, 2 insertions(+), 2 deletions(-)
> > >
> > > diff --git a/drivers/media/platform/rockchip/rkisp1/rkisp1-common.h b/drivers/media/platform/rockchip/rkisp1/rkisp1-common.h
> > > index 4b6b28c05b89..74098ddbeeb3 100644
> > > --- a/drivers/media/platform/rockchip/rkisp1/rkisp1-common.h
> > > +++ b/drivers/media/platform/rockchip/rkisp1/rkisp1-common.h
> > > @@ -33,8 +33,8 @@ struct dentry;
> > > #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_MAX_WIDTH 4416
> > > +#define RKISP1_ISP_MAX_HEIGHT 3312
> > > #define RKISP1_ISP_MIN_WIDTH 32
> > > #define RKISP1_ISP_MIN_HEIGHT 32
> > >
> >
> > --
> > Regards,
> >
> > Laurent Pinchart
>
_______________________________________________
linux-arm-kernel mailing list
linux-arm-kernel@lists.infradead.org
http://lists.infradead.org/mailman/listinfo/linux-arm-kernel
^ permalink raw reply [flat|nested] 27+ messages in thread
* Re: [PATCH] media: rkisp1: Allow higher input resolution
2024-02-19 2:09 ` Adam Ford
(?)
@ 2024-02-19 9:03 ` Umang Jain
-1 siblings, 0 replies; 27+ messages in thread
From: Umang Jain @ 2024-02-19 9:03 UTC (permalink / raw)
To: Adam Ford, Ondřej Jirman, Laurent Pinchart, linux-kernel,
Dafna Hirschfeld, Mauro Carvalho Chehab, Heiko Stuebner,
open list:ROCKCHIP ISP V1 DRIVER,
open list:ROCKCHIP ISP V1 DRIVER,
moderated list:ARM/Rockchip SoC support, Paul Elder
Hi All,
On 19/02/24 7:39 am, Adam Ford wrote:
> On Sun, Feb 18, 2024 at 3:02 PM Ondřej Jirman <megi@xff.cz> wrote:
>> On Sun, Feb 18, 2024 at 10:59:08PM +0200, Laurent Pinchart wrote:
>>> Hi Ondrej,
>>>
>>> (CC'ing Paul and Umang)
>>>
>>> Thank you for the patch.
>>>
>>> On Sat, Feb 17, 2024 at 07:51:58PM +0100, Ondřej Jirman wrote:
>>>> From: Ondrej Jirman <megi@xff.cz>
>>>>
>>>> In BSP driver, it is allowed, and it works in practice. Tested on
>>>> Pinephone Pro/RK3399 with IMX258 at full res.
>>> Paul, Umang, do I recall correctly that you have a similar change ?
>>> Could you review and test this (especially on the i.MX8MP) ?
>> It's also a limit from the datasheet, so the change should not be that
>> controversial:
>>
>> https://megous.com/dl/tmp/d2b333043ecebaf3.png
>>
>> (so that it doesn't sound like I just copied the BSP values)
>>
> From what I see in the i.MX8M Plus reference manual, it has a max
> resolution of 4096x3072, so it might be necessary to move this off
This is what I (and I assume Paul too) have been working with on i.MX8M
Plus. So it's the known and tested value of max ISP input from out side.
> from a #define into a structure that varies by product family.
Yeah!
>
> adam
>> regards,
>> o.
>>
>>>> Signed-off-by: Ondrej Jirman <megi@xff.cz>
>>>> ---
>>>> drivers/media/platform/rockchip/rkisp1/rkisp1-common.h | 4 ++--
>>>> 1 file changed, 2 insertions(+), 2 deletions(-)
>>>>
>>>> diff --git a/drivers/media/platform/rockchip/rkisp1/rkisp1-common.h b/drivers/media/platform/rockchip/rkisp1/rkisp1-common.h
>>>> index 4b6b28c05b89..74098ddbeeb3 100644
>>>> --- a/drivers/media/platform/rockchip/rkisp1/rkisp1-common.h
>>>> +++ b/drivers/media/platform/rockchip/rkisp1/rkisp1-common.h
>>>> @@ -33,8 +33,8 @@ struct dentry;
>>>> #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_MAX_WIDTH 4416
>>>> +#define RKISP1_ISP_MAX_HEIGHT 3312
>>>> #define RKISP1_ISP_MIN_WIDTH 32
>>>> #define RKISP1_ISP_MIN_HEIGHT 32
>>>>
>>> --
>>> Regards,
>>>
>>> Laurent Pinchart
^ permalink raw reply [flat|nested] 27+ messages in thread
* Re: [PATCH] media: rkisp1: Allow higher input resolution
@ 2024-02-19 9:03 ` Umang Jain
0 siblings, 0 replies; 27+ messages in thread
From: Umang Jain @ 2024-02-19 9:03 UTC (permalink / raw)
To: Adam Ford, Ondřej Jirman, Laurent Pinchart, linux-kernel,
Dafna Hirschfeld, Mauro Carvalho Chehab, Heiko Stuebner,
open list:ROCKCHIP ISP V1 DRIVER,
open list:ROCKCHIP ISP V1 DRIVER,
moderated list:ARM/Rockchip SoC support, Paul Elder
Hi All,
On 19/02/24 7:39 am, Adam Ford wrote:
> On Sun, Feb 18, 2024 at 3:02 PM Ondřej Jirman <megi@xff.cz> wrote:
>> On Sun, Feb 18, 2024 at 10:59:08PM +0200, Laurent Pinchart wrote:
>>> Hi Ondrej,
>>>
>>> (CC'ing Paul and Umang)
>>>
>>> Thank you for the patch.
>>>
>>> On Sat, Feb 17, 2024 at 07:51:58PM +0100, Ondřej Jirman wrote:
>>>> From: Ondrej Jirman <megi@xff.cz>
>>>>
>>>> In BSP driver, it is allowed, and it works in practice. Tested on
>>>> Pinephone Pro/RK3399 with IMX258 at full res.
>>> Paul, Umang, do I recall correctly that you have a similar change ?
>>> Could you review and test this (especially on the i.MX8MP) ?
>> It's also a limit from the datasheet, so the change should not be that
>> controversial:
>>
>> https://megous.com/dl/tmp/d2b333043ecebaf3.png
>>
>> (so that it doesn't sound like I just copied the BSP values)
>>
> From what I see in the i.MX8M Plus reference manual, it has a max
> resolution of 4096x3072, so it might be necessary to move this off
This is what I (and I assume Paul too) have been working with on i.MX8M
Plus. So it's the known and tested value of max ISP input from out side.
> from a #define into a structure that varies by product family.
Yeah!
>
> adam
>> regards,
>> o.
>>
>>>> Signed-off-by: Ondrej Jirman <megi@xff.cz>
>>>> ---
>>>> drivers/media/platform/rockchip/rkisp1/rkisp1-common.h | 4 ++--
>>>> 1 file changed, 2 insertions(+), 2 deletions(-)
>>>>
>>>> diff --git a/drivers/media/platform/rockchip/rkisp1/rkisp1-common.h b/drivers/media/platform/rockchip/rkisp1/rkisp1-common.h
>>>> index 4b6b28c05b89..74098ddbeeb3 100644
>>>> --- a/drivers/media/platform/rockchip/rkisp1/rkisp1-common.h
>>>> +++ b/drivers/media/platform/rockchip/rkisp1/rkisp1-common.h
>>>> @@ -33,8 +33,8 @@ struct dentry;
>>>> #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_MAX_WIDTH 4416
>>>> +#define RKISP1_ISP_MAX_HEIGHT 3312
>>>> #define RKISP1_ISP_MIN_WIDTH 32
>>>> #define RKISP1_ISP_MIN_HEIGHT 32
>>>>
>>> --
>>> Regards,
>>>
>>> Laurent Pinchart
_______________________________________________
Linux-rockchip mailing list
Linux-rockchip@lists.infradead.org
http://lists.infradead.org/mailman/listinfo/linux-rockchip
^ permalink raw reply [flat|nested] 27+ messages in thread
* Re: [PATCH] media: rkisp1: Allow higher input resolution
@ 2024-02-19 9:03 ` Umang Jain
0 siblings, 0 replies; 27+ messages in thread
From: Umang Jain @ 2024-02-19 9:03 UTC (permalink / raw)
To: Adam Ford, Ondřej Jirman, Laurent Pinchart, linux-kernel,
Dafna Hirschfeld, Mauro Carvalho Chehab, Heiko Stuebner,
open list:ROCKCHIP ISP V1 DRIVER,
open list:ROCKCHIP ISP V1 DRIVER,
moderated list:ARM/Rockchip SoC support, Paul Elder
Hi All,
On 19/02/24 7:39 am, Adam Ford wrote:
> On Sun, Feb 18, 2024 at 3:02 PM Ondřej Jirman <megi@xff.cz> wrote:
>> On Sun, Feb 18, 2024 at 10:59:08PM +0200, Laurent Pinchart wrote:
>>> Hi Ondrej,
>>>
>>> (CC'ing Paul and Umang)
>>>
>>> Thank you for the patch.
>>>
>>> On Sat, Feb 17, 2024 at 07:51:58PM +0100, Ondřej Jirman wrote:
>>>> From: Ondrej Jirman <megi@xff.cz>
>>>>
>>>> In BSP driver, it is allowed, and it works in practice. Tested on
>>>> Pinephone Pro/RK3399 with IMX258 at full res.
>>> Paul, Umang, do I recall correctly that you have a similar change ?
>>> Could you review and test this (especially on the i.MX8MP) ?
>> It's also a limit from the datasheet, so the change should not be that
>> controversial:
>>
>> https://megous.com/dl/tmp/d2b333043ecebaf3.png
>>
>> (so that it doesn't sound like I just copied the BSP values)
>>
> From what I see in the i.MX8M Plus reference manual, it has a max
> resolution of 4096x3072, so it might be necessary to move this off
This is what I (and I assume Paul too) have been working with on i.MX8M
Plus. So it's the known and tested value of max ISP input from out side.
> from a #define into a structure that varies by product family.
Yeah!
>
> adam
>> regards,
>> o.
>>
>>>> Signed-off-by: Ondrej Jirman <megi@xff.cz>
>>>> ---
>>>> drivers/media/platform/rockchip/rkisp1/rkisp1-common.h | 4 ++--
>>>> 1 file changed, 2 insertions(+), 2 deletions(-)
>>>>
>>>> diff --git a/drivers/media/platform/rockchip/rkisp1/rkisp1-common.h b/drivers/media/platform/rockchip/rkisp1/rkisp1-common.h
>>>> index 4b6b28c05b89..74098ddbeeb3 100644
>>>> --- a/drivers/media/platform/rockchip/rkisp1/rkisp1-common.h
>>>> +++ b/drivers/media/platform/rockchip/rkisp1/rkisp1-common.h
>>>> @@ -33,8 +33,8 @@ struct dentry;
>>>> #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_MAX_WIDTH 4416
>>>> +#define RKISP1_ISP_MAX_HEIGHT 3312
>>>> #define RKISP1_ISP_MIN_WIDTH 32
>>>> #define RKISP1_ISP_MIN_HEIGHT 32
>>>>
>>> --
>>> Regards,
>>>
>>> Laurent Pinchart
_______________________________________________
linux-arm-kernel mailing list
linux-arm-kernel@lists.infradead.org
http://lists.infradead.org/mailman/listinfo/linux-arm-kernel
^ permalink raw reply [flat|nested] 27+ messages in thread
* Re: [PATCH] media: rkisp1: Allow higher input resolution
2024-02-19 9:03 ` Umang Jain
(?)
@ 2024-02-19 10:11 ` Paul Elder
-1 siblings, 0 replies; 27+ messages in thread
From: Paul Elder @ 2024-02-19 10:11 UTC (permalink / raw)
To: Umang Jain
Cc: Adam Ford, Ondřej Jirman, Laurent Pinchart, linux-kernel,
Dafna Hirschfeld, Mauro Carvalho Chehab, Heiko Stuebner,
open list:ROCKCHIP ISP V1 DRIVER,
open list:ROCKCHIP ISP V1 DRIVER,
moderated list:ARM/Rockchip SoC support
On Mon, Feb 19, 2024 at 02:33:21PM +0530, Umang Jain wrote:
> Hi All,
>
> On 19/02/24 7:39 am, Adam Ford wrote:
> > On Sun, Feb 18, 2024 at 3:02 PM Ondřej Jirman <megi@xff.cz> wrote:
> > > On Sun, Feb 18, 2024 at 10:59:08PM +0200, Laurent Pinchart wrote:
> > > > Hi Ondrej,
> > > >
> > > > (CC'ing Paul and Umang)
> > > >
> > > > Thank you for the patch.
> > > >
> > > > On Sat, Feb 17, 2024 at 07:51:58PM +0100, Ondřej Jirman wrote:
> > > > > From: Ondrej Jirman <megi@xff.cz>
> > > > >
> > > > > In BSP driver, it is allowed, and it works in practice. Tested on
> > > > > Pinephone Pro/RK3399 with IMX258 at full res.
> > > > Paul, Umang, do I recall correctly that you have a similar change ?
> > > > Could you review and test this (especially on the i.MX8MP) ?
> > > It's also a limit from the datasheet, so the change should not be that
> > > controversial:
> > >
> > > https://megous.com/dl/tmp/d2b333043ecebaf3.png
> > >
> > > (so that it doesn't sound like I just copied the BSP values)
> > >
> > From what I see in the i.MX8M Plus reference manual, it has a max
> > resolution of 4096x3072, so it might be necessary to move this off
>
> This is what I (and I assume Paul too) have been working with on i.MX8M
> Plus. So it's the known and tested value of max ISP input from out side.
> > from a #define into a structure that varies by product family.
Yes, this is what needs to be done. Here's what I have in my notes:
- The RK3399 TRM says 4416x3312 max input and output on main path, with
1920x1080 max output on self path.
- The PX30 datasheet [1] says 3264x2448 max input and output on main
path, with 1920x1080 max output on self path.
- The RK3288 documentation [2] (under "Camera Interface and Image
Processor") says 4416x3312 max input and output on main path, with
1920x1080 max output on self path.
- The i.MX8MP reference manual (the open one) [3] (in table 13-1) says
4096x3072 max resolution in single ISP mode
The i.MX8M Plus seems to indeed be limited to 4096x3072, but the TPG is
capable of generating 4416x3312, and the ISP works fine in bypass (and
therefore raw) mode, so technically it has different maximum sizes
depending on the format which makes this more exciting.
In any case, the PX30 (assuming the datasheet is correct) only supports
up to 3264x2448, so the existing #define is incorrect anyway.
I don't have a PX30 nor an RK3288 so I can't test those, and I haven't
set up my OV64A40 yet which (I've heard) can be used to test even bigger
resolutions.
Paul
[1] https://opensource.rock-chips.com/images/8/87/Rockchip_PX30_Datasheet_V1.4-20191227.pdf
[2] https://opensource.rock-chips.com/images/4/49/Rockchip_RK3288_Datasheet_V2.7-20191227.pdf
[3] (requires login) https://www.nxp.com/products/processors-and-microcontrollers/arm-processors/i-mx-applications-processors/i-mx-8-applications-processors/i-mx-8m-plus-arm-cortex-a53-machine-learning-vision-multimedia-and-industrial-iot:IMX8MPLUS
>
> Yeah!
> >
> > adam
> > > regards,
> > > o.
> > >
> > > > > Signed-off-by: Ondrej Jirman <megi@xff.cz>
> > > > > ---
> > > > > drivers/media/platform/rockchip/rkisp1/rkisp1-common.h | 4 ++--
> > > > > 1 file changed, 2 insertions(+), 2 deletions(-)
> > > > >
> > > > > diff --git a/drivers/media/platform/rockchip/rkisp1/rkisp1-common.h b/drivers/media/platform/rockchip/rkisp1/rkisp1-common.h
> > > > > index 4b6b28c05b89..74098ddbeeb3 100644
> > > > > --- a/drivers/media/platform/rockchip/rkisp1/rkisp1-common.h
> > > > > +++ b/drivers/media/platform/rockchip/rkisp1/rkisp1-common.h
> > > > > @@ -33,8 +33,8 @@ struct dentry;
> > > > > #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_MAX_WIDTH 4416
> > > > > +#define RKISP1_ISP_MAX_HEIGHT 3312
> > > > > #define RKISP1_ISP_MIN_WIDTH 32
> > > > > #define RKISP1_ISP_MIN_HEIGHT 32
> > > > >
> > > > --
> > > > Regards,
> > > >
> > > > Laurent Pinchart
>
^ permalink raw reply [flat|nested] 27+ messages in thread
* Re: [PATCH] media: rkisp1: Allow higher input resolution
@ 2024-02-19 10:11 ` Paul Elder
0 siblings, 0 replies; 27+ messages in thread
From: Paul Elder @ 2024-02-19 10:11 UTC (permalink / raw)
To: Umang Jain
Cc: Adam Ford, Ondřej Jirman, Laurent Pinchart, linux-kernel,
Dafna Hirschfeld, Mauro Carvalho Chehab, Heiko Stuebner,
open list:ROCKCHIP ISP V1 DRIVER,
open list:ROCKCHIP ISP V1 DRIVER,
moderated list:ARM/Rockchip SoC support
On Mon, Feb 19, 2024 at 02:33:21PM +0530, Umang Jain wrote:
> Hi All,
>
> On 19/02/24 7:39 am, Adam Ford wrote:
> > On Sun, Feb 18, 2024 at 3:02 PM Ondřej Jirman <megi@xff.cz> wrote:
> > > On Sun, Feb 18, 2024 at 10:59:08PM +0200, Laurent Pinchart wrote:
> > > > Hi Ondrej,
> > > >
> > > > (CC'ing Paul and Umang)
> > > >
> > > > Thank you for the patch.
> > > >
> > > > On Sat, Feb 17, 2024 at 07:51:58PM +0100, Ondřej Jirman wrote:
> > > > > From: Ondrej Jirman <megi@xff.cz>
> > > > >
> > > > > In BSP driver, it is allowed, and it works in practice. Tested on
> > > > > Pinephone Pro/RK3399 with IMX258 at full res.
> > > > Paul, Umang, do I recall correctly that you have a similar change ?
> > > > Could you review and test this (especially on the i.MX8MP) ?
> > > It's also a limit from the datasheet, so the change should not be that
> > > controversial:
> > >
> > > https://megous.com/dl/tmp/d2b333043ecebaf3.png
> > >
> > > (so that it doesn't sound like I just copied the BSP values)
> > >
> > From what I see in the i.MX8M Plus reference manual, it has a max
> > resolution of 4096x3072, so it might be necessary to move this off
>
> This is what I (and I assume Paul too) have been working with on i.MX8M
> Plus. So it's the known and tested value of max ISP input from out side.
> > from a #define into a structure that varies by product family.
Yes, this is what needs to be done. Here's what I have in my notes:
- The RK3399 TRM says 4416x3312 max input and output on main path, with
1920x1080 max output on self path.
- The PX30 datasheet [1] says 3264x2448 max input and output on main
path, with 1920x1080 max output on self path.
- The RK3288 documentation [2] (under "Camera Interface and Image
Processor") says 4416x3312 max input and output on main path, with
1920x1080 max output on self path.
- The i.MX8MP reference manual (the open one) [3] (in table 13-1) says
4096x3072 max resolution in single ISP mode
The i.MX8M Plus seems to indeed be limited to 4096x3072, but the TPG is
capable of generating 4416x3312, and the ISP works fine in bypass (and
therefore raw) mode, so technically it has different maximum sizes
depending on the format which makes this more exciting.
In any case, the PX30 (assuming the datasheet is correct) only supports
up to 3264x2448, so the existing #define is incorrect anyway.
I don't have a PX30 nor an RK3288 so I can't test those, and I haven't
set up my OV64A40 yet which (I've heard) can be used to test even bigger
resolutions.
Paul
[1] https://opensource.rock-chips.com/images/8/87/Rockchip_PX30_Datasheet_V1.4-20191227.pdf
[2] https://opensource.rock-chips.com/images/4/49/Rockchip_RK3288_Datasheet_V2.7-20191227.pdf
[3] (requires login) https://www.nxp.com/products/processors-and-microcontrollers/arm-processors/i-mx-applications-processors/i-mx-8-applications-processors/i-mx-8m-plus-arm-cortex-a53-machine-learning-vision-multimedia-and-industrial-iot:IMX8MPLUS
>
> Yeah!
> >
> > adam
> > > regards,
> > > o.
> > >
> > > > > Signed-off-by: Ondrej Jirman <megi@xff.cz>
> > > > > ---
> > > > > drivers/media/platform/rockchip/rkisp1/rkisp1-common.h | 4 ++--
> > > > > 1 file changed, 2 insertions(+), 2 deletions(-)
> > > > >
> > > > > diff --git a/drivers/media/platform/rockchip/rkisp1/rkisp1-common.h b/drivers/media/platform/rockchip/rkisp1/rkisp1-common.h
> > > > > index 4b6b28c05b89..74098ddbeeb3 100644
> > > > > --- a/drivers/media/platform/rockchip/rkisp1/rkisp1-common.h
> > > > > +++ b/drivers/media/platform/rockchip/rkisp1/rkisp1-common.h
> > > > > @@ -33,8 +33,8 @@ struct dentry;
> > > > > #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_MAX_WIDTH 4416
> > > > > +#define RKISP1_ISP_MAX_HEIGHT 3312
> > > > > #define RKISP1_ISP_MIN_WIDTH 32
> > > > > #define RKISP1_ISP_MIN_HEIGHT 32
> > > > >
> > > > --
> > > > Regards,
> > > >
> > > > Laurent Pinchart
>
_______________________________________________
Linux-rockchip mailing list
Linux-rockchip@lists.infradead.org
http://lists.infradead.org/mailman/listinfo/linux-rockchip
^ permalink raw reply [flat|nested] 27+ messages in thread
* Re: [PATCH] media: rkisp1: Allow higher input resolution
@ 2024-02-19 10:11 ` Paul Elder
0 siblings, 0 replies; 27+ messages in thread
From: Paul Elder @ 2024-02-19 10:11 UTC (permalink / raw)
To: Umang Jain
Cc: Adam Ford, Ondřej Jirman, Laurent Pinchart, linux-kernel,
Dafna Hirschfeld, Mauro Carvalho Chehab, Heiko Stuebner,
open list:ROCKCHIP ISP V1 DRIVER,
open list:ROCKCHIP ISP V1 DRIVER,
moderated list:ARM/Rockchip SoC support
On Mon, Feb 19, 2024 at 02:33:21PM +0530, Umang Jain wrote:
> Hi All,
>
> On 19/02/24 7:39 am, Adam Ford wrote:
> > On Sun, Feb 18, 2024 at 3:02 PM Ondřej Jirman <megi@xff.cz> wrote:
> > > On Sun, Feb 18, 2024 at 10:59:08PM +0200, Laurent Pinchart wrote:
> > > > Hi Ondrej,
> > > >
> > > > (CC'ing Paul and Umang)
> > > >
> > > > Thank you for the patch.
> > > >
> > > > On Sat, Feb 17, 2024 at 07:51:58PM +0100, Ondřej Jirman wrote:
> > > > > From: Ondrej Jirman <megi@xff.cz>
> > > > >
> > > > > In BSP driver, it is allowed, and it works in practice. Tested on
> > > > > Pinephone Pro/RK3399 with IMX258 at full res.
> > > > Paul, Umang, do I recall correctly that you have a similar change ?
> > > > Could you review and test this (especially on the i.MX8MP) ?
> > > It's also a limit from the datasheet, so the change should not be that
> > > controversial:
> > >
> > > https://megous.com/dl/tmp/d2b333043ecebaf3.png
> > >
> > > (so that it doesn't sound like I just copied the BSP values)
> > >
> > From what I see in the i.MX8M Plus reference manual, it has a max
> > resolution of 4096x3072, so it might be necessary to move this off
>
> This is what I (and I assume Paul too) have been working with on i.MX8M
> Plus. So it's the known and tested value of max ISP input from out side.
> > from a #define into a structure that varies by product family.
Yes, this is what needs to be done. Here's what I have in my notes:
- The RK3399 TRM says 4416x3312 max input and output on main path, with
1920x1080 max output on self path.
- The PX30 datasheet [1] says 3264x2448 max input and output on main
path, with 1920x1080 max output on self path.
- The RK3288 documentation [2] (under "Camera Interface and Image
Processor") says 4416x3312 max input and output on main path, with
1920x1080 max output on self path.
- The i.MX8MP reference manual (the open one) [3] (in table 13-1) says
4096x3072 max resolution in single ISP mode
The i.MX8M Plus seems to indeed be limited to 4096x3072, but the TPG is
capable of generating 4416x3312, and the ISP works fine in bypass (and
therefore raw) mode, so technically it has different maximum sizes
depending on the format which makes this more exciting.
In any case, the PX30 (assuming the datasheet is correct) only supports
up to 3264x2448, so the existing #define is incorrect anyway.
I don't have a PX30 nor an RK3288 so I can't test those, and I haven't
set up my OV64A40 yet which (I've heard) can be used to test even bigger
resolutions.
Paul
[1] https://opensource.rock-chips.com/images/8/87/Rockchip_PX30_Datasheet_V1.4-20191227.pdf
[2] https://opensource.rock-chips.com/images/4/49/Rockchip_RK3288_Datasheet_V2.7-20191227.pdf
[3] (requires login) https://www.nxp.com/products/processors-and-microcontrollers/arm-processors/i-mx-applications-processors/i-mx-8-applications-processors/i-mx-8m-plus-arm-cortex-a53-machine-learning-vision-multimedia-and-industrial-iot:IMX8MPLUS
>
> Yeah!
> >
> > adam
> > > regards,
> > > o.
> > >
> > > > > Signed-off-by: Ondrej Jirman <megi@xff.cz>
> > > > > ---
> > > > > drivers/media/platform/rockchip/rkisp1/rkisp1-common.h | 4 ++--
> > > > > 1 file changed, 2 insertions(+), 2 deletions(-)
> > > > >
> > > > > diff --git a/drivers/media/platform/rockchip/rkisp1/rkisp1-common.h b/drivers/media/platform/rockchip/rkisp1/rkisp1-common.h
> > > > > index 4b6b28c05b89..74098ddbeeb3 100644
> > > > > --- a/drivers/media/platform/rockchip/rkisp1/rkisp1-common.h
> > > > > +++ b/drivers/media/platform/rockchip/rkisp1/rkisp1-common.h
> > > > > @@ -33,8 +33,8 @@ struct dentry;
> > > > > #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_MAX_WIDTH 4416
> > > > > +#define RKISP1_ISP_MAX_HEIGHT 3312
> > > > > #define RKISP1_ISP_MIN_WIDTH 32
> > > > > #define RKISP1_ISP_MIN_HEIGHT 32
> > > > >
> > > > --
> > > > Regards,
> > > >
> > > > Laurent Pinchart
>
_______________________________________________
linux-arm-kernel mailing list
linux-arm-kernel@lists.infradead.org
http://lists.infradead.org/mailman/listinfo/linux-arm-kernel
^ permalink raw reply [flat|nested] 27+ messages in thread
* Re: [PATCH] media: rkisp1: Allow higher input resolution
2024-02-19 10:11 ` Paul Elder
(?)
@ 2024-02-19 11:54 ` Laurent Pinchart
-1 siblings, 0 replies; 27+ messages in thread
From: Laurent Pinchart @ 2024-02-19 11:54 UTC (permalink / raw)
To: Paul Elder
Cc: Umang Jain, Adam Ford, Ondřej Jirman, linux-kernel,
Dafna Hirschfeld, Mauro Carvalho Chehab, Heiko Stuebner,
open list:ROCKCHIP ISP V1 DRIVER,
open list:ROCKCHIP ISP V1 DRIVER,
moderated list:ARM/Rockchip SoC support
On Mon, Feb 19, 2024 at 07:11:20PM +0900, Paul Elder wrote:
> On Mon, Feb 19, 2024 at 02:33:21PM +0530, Umang Jain wrote:
> > On 19/02/24 7:39 am, Adam Ford wrote:
> > > On Sun, Feb 18, 2024 at 3:02 PM Ondřej Jirman <megi@xff.cz> wrote:
> > > > On Sun, Feb 18, 2024 at 10:59:08PM +0200, Laurent Pinchart wrote:
> > > > > On Sat, Feb 17, 2024 at 07:51:58PM +0100, Ondřej Jirman wrote:
> > > > > > From: Ondrej Jirman <megi@xff.cz>
> > > > > >
> > > > > > In BSP driver, it is allowed, and it works in practice. Tested on
> > > > > > Pinephone Pro/RK3399 with IMX258 at full res.
> > > > >
> > > > > Paul, Umang, do I recall correctly that you have a similar change ?
> > > > > Could you review and test this (especially on the i.MX8MP) ?
> > > >
> > > > It's also a limit from the datasheet, so the change should not be that
> > > > controversial:
> > > >
> > > > https://megous.com/dl/tmp/d2b333043ecebaf3.png
> > > >
> > > > (so that it doesn't sound like I just copied the BSP values)
> > > >
> > > From what I see in the i.MX8M Plus reference manual, it has a max
> > > resolution of 4096x3072, so it might be necessary to move this off
> >
> > This is what I (and I assume Paul too) have been working with on i.MX8M
> > Plus. So it's the known and tested value of max ISP input from out side.
> >
> > > from a #define into a structure that varies by product family.
>
> Yes, this is what needs to be done. Here's what I have in my notes:
>
> - The RK3399 TRM says 4416x3312 max input and output on main path, with
> 1920x1080 max output on self path.
> - The PX30 datasheet [1] says 3264x2448 max input and output on main
> path, with 1920x1080 max output on self path.
> - The RK3288 documentation [2] (under "Camera Interface and Image
> Processor") says 4416x3312 max input and output on main path, with
> 1920x1080 max output on self path.
> - The i.MX8MP reference manual (the open one) [3] (in table 13-1) says
> 4096x3072 max resolution in single ISP mode
>
> The i.MX8M Plus seems to indeed be limited to 4096x3072, but the TPG is
> capable of generating 4416x3312, and the ISP works fine in bypass (and
> therefore raw) mode, so technically it has different maximum sizes
> depending on the format which makes this more exciting.
>
> In any case, the PX30 (assuming the datasheet is correct) only supports
> up to 3264x2448, so the existing #define is incorrect anyway.
>
> I don't have a PX30 nor an RK3288 so I can't test those, and I haven't
> set up my OV64A40 yet which (I've heard) can be used to test even bigger
> resolutions.
Thanks for the summary Paul.
Ondřej, would you be interested in implementing this ?
> [1] https://opensource.rock-chips.com/images/8/87/Rockchip_PX30_Datasheet_V1.4-20191227.pdf
> [2] https://opensource.rock-chips.com/images/4/49/Rockchip_RK3288_Datasheet_V2.7-20191227.pdf
> [3] (requires login) https://www.nxp.com/products/processors-and-microcontrollers/arm-processors/i-mx-applications-processors/i-mx-8-applications-processors/i-mx-8m-plus-arm-cortex-a53-machine-learning-vision-multimedia-and-industrial-iot:IMX8MPLUS
>
> > > > > > Signed-off-by: Ondrej Jirman <megi@xff.cz>
> > > > > > ---
> > > > > > drivers/media/platform/rockchip/rkisp1/rkisp1-common.h | 4 ++--
> > > > > > 1 file changed, 2 insertions(+), 2 deletions(-)
> > > > > >
> > > > > > diff --git a/drivers/media/platform/rockchip/rkisp1/rkisp1-common.h b/drivers/media/platform/rockchip/rkisp1/rkisp1-common.h
> > > > > > index 4b6b28c05b89..74098ddbeeb3 100644
> > > > > > --- a/drivers/media/platform/rockchip/rkisp1/rkisp1-common.h
> > > > > > +++ b/drivers/media/platform/rockchip/rkisp1/rkisp1-common.h
> > > > > > @@ -33,8 +33,8 @@ struct dentry;
> > > > > > #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_MAX_WIDTH 4416
> > > > > > +#define RKISP1_ISP_MAX_HEIGHT 3312
> > > > > > #define RKISP1_ISP_MIN_WIDTH 32
> > > > > > #define RKISP1_ISP_MIN_HEIGHT 32
> > > > > >
--
Regards,
Laurent Pinchart
^ permalink raw reply [flat|nested] 27+ messages in thread
* Re: [PATCH] media: rkisp1: Allow higher input resolution
@ 2024-02-19 11:54 ` Laurent Pinchart
0 siblings, 0 replies; 27+ messages in thread
From: Laurent Pinchart @ 2024-02-19 11:54 UTC (permalink / raw)
To: Paul Elder
Cc: Umang Jain, Adam Ford, Ondřej Jirman, linux-kernel,
Dafna Hirschfeld, Mauro Carvalho Chehab, Heiko Stuebner,
open list:ROCKCHIP ISP V1 DRIVER,
open list:ROCKCHIP ISP V1 DRIVER,
moderated list:ARM/Rockchip SoC support
On Mon, Feb 19, 2024 at 07:11:20PM +0900, Paul Elder wrote:
> On Mon, Feb 19, 2024 at 02:33:21PM +0530, Umang Jain wrote:
> > On 19/02/24 7:39 am, Adam Ford wrote:
> > > On Sun, Feb 18, 2024 at 3:02 PM Ondřej Jirman <megi@xff.cz> wrote:
> > > > On Sun, Feb 18, 2024 at 10:59:08PM +0200, Laurent Pinchart wrote:
> > > > > On Sat, Feb 17, 2024 at 07:51:58PM +0100, Ondřej Jirman wrote:
> > > > > > From: Ondrej Jirman <megi@xff.cz>
> > > > > >
> > > > > > In BSP driver, it is allowed, and it works in practice. Tested on
> > > > > > Pinephone Pro/RK3399 with IMX258 at full res.
> > > > >
> > > > > Paul, Umang, do I recall correctly that you have a similar change ?
> > > > > Could you review and test this (especially on the i.MX8MP) ?
> > > >
> > > > It's also a limit from the datasheet, so the change should not be that
> > > > controversial:
> > > >
> > > > https://megous.com/dl/tmp/d2b333043ecebaf3.png
> > > >
> > > > (so that it doesn't sound like I just copied the BSP values)
> > > >
> > > From what I see in the i.MX8M Plus reference manual, it has a max
> > > resolution of 4096x3072, so it might be necessary to move this off
> >
> > This is what I (and I assume Paul too) have been working with on i.MX8M
> > Plus. So it's the known and tested value of max ISP input from out side.
> >
> > > from a #define into a structure that varies by product family.
>
> Yes, this is what needs to be done. Here's what I have in my notes:
>
> - The RK3399 TRM says 4416x3312 max input and output on main path, with
> 1920x1080 max output on self path.
> - The PX30 datasheet [1] says 3264x2448 max input and output on main
> path, with 1920x1080 max output on self path.
> - The RK3288 documentation [2] (under "Camera Interface and Image
> Processor") says 4416x3312 max input and output on main path, with
> 1920x1080 max output on self path.
> - The i.MX8MP reference manual (the open one) [3] (in table 13-1) says
> 4096x3072 max resolution in single ISP mode
>
> The i.MX8M Plus seems to indeed be limited to 4096x3072, but the TPG is
> capable of generating 4416x3312, and the ISP works fine in bypass (and
> therefore raw) mode, so technically it has different maximum sizes
> depending on the format which makes this more exciting.
>
> In any case, the PX30 (assuming the datasheet is correct) only supports
> up to 3264x2448, so the existing #define is incorrect anyway.
>
> I don't have a PX30 nor an RK3288 so I can't test those, and I haven't
> set up my OV64A40 yet which (I've heard) can be used to test even bigger
> resolutions.
Thanks for the summary Paul.
Ondřej, would you be interested in implementing this ?
> [1] https://opensource.rock-chips.com/images/8/87/Rockchip_PX30_Datasheet_V1.4-20191227.pdf
> [2] https://opensource.rock-chips.com/images/4/49/Rockchip_RK3288_Datasheet_V2.7-20191227.pdf
> [3] (requires login) https://www.nxp.com/products/processors-and-microcontrollers/arm-processors/i-mx-applications-processors/i-mx-8-applications-processors/i-mx-8m-plus-arm-cortex-a53-machine-learning-vision-multimedia-and-industrial-iot:IMX8MPLUS
>
> > > > > > Signed-off-by: Ondrej Jirman <megi@xff.cz>
> > > > > > ---
> > > > > > drivers/media/platform/rockchip/rkisp1/rkisp1-common.h | 4 ++--
> > > > > > 1 file changed, 2 insertions(+), 2 deletions(-)
> > > > > >
> > > > > > diff --git a/drivers/media/platform/rockchip/rkisp1/rkisp1-common.h b/drivers/media/platform/rockchip/rkisp1/rkisp1-common.h
> > > > > > index 4b6b28c05b89..74098ddbeeb3 100644
> > > > > > --- a/drivers/media/platform/rockchip/rkisp1/rkisp1-common.h
> > > > > > +++ b/drivers/media/platform/rockchip/rkisp1/rkisp1-common.h
> > > > > > @@ -33,8 +33,8 @@ struct dentry;
> > > > > > #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_MAX_WIDTH 4416
> > > > > > +#define RKISP1_ISP_MAX_HEIGHT 3312
> > > > > > #define RKISP1_ISP_MIN_WIDTH 32
> > > > > > #define RKISP1_ISP_MIN_HEIGHT 32
> > > > > >
--
Regards,
Laurent Pinchart
_______________________________________________
Linux-rockchip mailing list
Linux-rockchip@lists.infradead.org
http://lists.infradead.org/mailman/listinfo/linux-rockchip
^ permalink raw reply [flat|nested] 27+ messages in thread
* Re: [PATCH] media: rkisp1: Allow higher input resolution
@ 2024-02-19 11:54 ` Laurent Pinchart
0 siblings, 0 replies; 27+ messages in thread
From: Laurent Pinchart @ 2024-02-19 11:54 UTC (permalink / raw)
To: Paul Elder
Cc: Umang Jain, Adam Ford, Ondřej Jirman, linux-kernel,
Dafna Hirschfeld, Mauro Carvalho Chehab, Heiko Stuebner,
open list:ROCKCHIP ISP V1 DRIVER,
open list:ROCKCHIP ISP V1 DRIVER,
moderated list:ARM/Rockchip SoC support
On Mon, Feb 19, 2024 at 07:11:20PM +0900, Paul Elder wrote:
> On Mon, Feb 19, 2024 at 02:33:21PM +0530, Umang Jain wrote:
> > On 19/02/24 7:39 am, Adam Ford wrote:
> > > On Sun, Feb 18, 2024 at 3:02 PM Ondřej Jirman <megi@xff.cz> wrote:
> > > > On Sun, Feb 18, 2024 at 10:59:08PM +0200, Laurent Pinchart wrote:
> > > > > On Sat, Feb 17, 2024 at 07:51:58PM +0100, Ondřej Jirman wrote:
> > > > > > From: Ondrej Jirman <megi@xff.cz>
> > > > > >
> > > > > > In BSP driver, it is allowed, and it works in practice. Tested on
> > > > > > Pinephone Pro/RK3399 with IMX258 at full res.
> > > > >
> > > > > Paul, Umang, do I recall correctly that you have a similar change ?
> > > > > Could you review and test this (especially on the i.MX8MP) ?
> > > >
> > > > It's also a limit from the datasheet, so the change should not be that
> > > > controversial:
> > > >
> > > > https://megous.com/dl/tmp/d2b333043ecebaf3.png
> > > >
> > > > (so that it doesn't sound like I just copied the BSP values)
> > > >
> > > From what I see in the i.MX8M Plus reference manual, it has a max
> > > resolution of 4096x3072, so it might be necessary to move this off
> >
> > This is what I (and I assume Paul too) have been working with on i.MX8M
> > Plus. So it's the known and tested value of max ISP input from out side.
> >
> > > from a #define into a structure that varies by product family.
>
> Yes, this is what needs to be done. Here's what I have in my notes:
>
> - The RK3399 TRM says 4416x3312 max input and output on main path, with
> 1920x1080 max output on self path.
> - The PX30 datasheet [1] says 3264x2448 max input and output on main
> path, with 1920x1080 max output on self path.
> - The RK3288 documentation [2] (under "Camera Interface and Image
> Processor") says 4416x3312 max input and output on main path, with
> 1920x1080 max output on self path.
> - The i.MX8MP reference manual (the open one) [3] (in table 13-1) says
> 4096x3072 max resolution in single ISP mode
>
> The i.MX8M Plus seems to indeed be limited to 4096x3072, but the TPG is
> capable of generating 4416x3312, and the ISP works fine in bypass (and
> therefore raw) mode, so technically it has different maximum sizes
> depending on the format which makes this more exciting.
>
> In any case, the PX30 (assuming the datasheet is correct) only supports
> up to 3264x2448, so the existing #define is incorrect anyway.
>
> I don't have a PX30 nor an RK3288 so I can't test those, and I haven't
> set up my OV64A40 yet which (I've heard) can be used to test even bigger
> resolutions.
Thanks for the summary Paul.
Ondřej, would you be interested in implementing this ?
> [1] https://opensource.rock-chips.com/images/8/87/Rockchip_PX30_Datasheet_V1.4-20191227.pdf
> [2] https://opensource.rock-chips.com/images/4/49/Rockchip_RK3288_Datasheet_V2.7-20191227.pdf
> [3] (requires login) https://www.nxp.com/products/processors-and-microcontrollers/arm-processors/i-mx-applications-processors/i-mx-8-applications-processors/i-mx-8m-plus-arm-cortex-a53-machine-learning-vision-multimedia-and-industrial-iot:IMX8MPLUS
>
> > > > > > Signed-off-by: Ondrej Jirman <megi@xff.cz>
> > > > > > ---
> > > > > > drivers/media/platform/rockchip/rkisp1/rkisp1-common.h | 4 ++--
> > > > > > 1 file changed, 2 insertions(+), 2 deletions(-)
> > > > > >
> > > > > > diff --git a/drivers/media/platform/rockchip/rkisp1/rkisp1-common.h b/drivers/media/platform/rockchip/rkisp1/rkisp1-common.h
> > > > > > index 4b6b28c05b89..74098ddbeeb3 100644
> > > > > > --- a/drivers/media/platform/rockchip/rkisp1/rkisp1-common.h
> > > > > > +++ b/drivers/media/platform/rockchip/rkisp1/rkisp1-common.h
> > > > > > @@ -33,8 +33,8 @@ struct dentry;
> > > > > > #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_MAX_WIDTH 4416
> > > > > > +#define RKISP1_ISP_MAX_HEIGHT 3312
> > > > > > #define RKISP1_ISP_MIN_WIDTH 32
> > > > > > #define RKISP1_ISP_MIN_HEIGHT 32
> > > > > >
--
Regards,
Laurent Pinchart
_______________________________________________
linux-arm-kernel mailing list
linux-arm-kernel@lists.infradead.org
http://lists.infradead.org/mailman/listinfo/linux-arm-kernel
^ permalink raw reply [flat|nested] 27+ messages in thread
* Re: [PATCH] media: rkisp1: Allow higher input resolution
2024-02-19 10:11 ` Paul Elder
(?)
@ 2024-02-20 23:56 ` Ondřej Jirman
-1 siblings, 0 replies; 27+ messages in thread
From: Ondřej Jirman @ 2024-02-20 23:56 UTC (permalink / raw)
To: Paul Elder
Cc: Umang Jain, Adam Ford, Laurent Pinchart, linux-kernel,
Dafna Hirschfeld, Mauro Carvalho Chehab, Heiko Stuebner,
open list:ROCKCHIP ISP V1 DRIVER,
open list:ROCKCHIP ISP V1 DRIVER,
moderated list:ARM/Rockchip SoC support
On Mon, Feb 19, 2024 at 07:11:20PM +0900, Paul Elder wrote:
> On Mon, Feb 19, 2024 at 02:33:21PM +0530, Umang Jain wrote:
> > Hi All,
> >
> > On 19/02/24 7:39 am, Adam Ford wrote:
> > > On Sun, Feb 18, 2024 at 3:02 PM Ondřej Jirman <megi@xff.cz> wrote:
> > > > On Sun, Feb 18, 2024 at 10:59:08PM +0200, Laurent Pinchart wrote:
> > > > > Hi Ondrej,
> > > > >
> > > > > (CC'ing Paul and Umang)
> > > > >
> > > > > Thank you for the patch.
> > > > >
> > > > > On Sat, Feb 17, 2024 at 07:51:58PM +0100, Ondřej Jirman wrote:
> > > > > > From: Ondrej Jirman <megi@xff.cz>
> > > > > >
> > > > > > In BSP driver, it is allowed, and it works in practice. Tested on
> > > > > > Pinephone Pro/RK3399 with IMX258 at full res.
> > > > > Paul, Umang, do I recall correctly that you have a similar change ?
> > > > > Could you review and test this (especially on the i.MX8MP) ?
> > > > It's also a limit from the datasheet, so the change should not be that
> > > > controversial:
> > > >
> > > > https://megous.com/dl/tmp/d2b333043ecebaf3.png
> > > >
> > > > (so that it doesn't sound like I just copied the BSP values)
> > > >
> > > From what I see in the i.MX8M Plus reference manual, it has a max
> > > resolution of 4096x3072, so it might be necessary to move this off
> >
> > This is what I (and I assume Paul too) have been working with on i.MX8M
> > Plus. So it's the known and tested value of max ISP input from out side.
> > > from a #define into a structure that varies by product family.
>
> Yes, this is what needs to be done. Here's what I have in my notes:
>
> - The RK3399 TRM says 4416x3312 max input and output on main path, with
> 1920x1080 max output on self path.
> - The PX30 datasheet [1] says 3264x2448 max input and output on main
> path, with 1920x1080 max output on self path.
> - The RK3288 documentation [2] (under "Camera Interface and Image
> Processor") says 4416x3312 max input and output on main path, with
> 1920x1080 max output on self path.
> - The i.MX8MP reference manual (the open one) [3] (in table 13-1) says
> 4096x3072 max resolution in single ISP mode
Yeah, RKISP1_RSZ_SP_SRC_MAX_HEIGHT in the driver also seems weird in
this respect:
#define RKISP1_RSZ_SP_SRC_MAX_WIDTH 1920
#define RKISP1_RSZ_SP_SRC_MAX_HEIGHT 1920
regards,
o.
> The i.MX8M Plus seems to indeed be limited to 4096x3072, but the TPG is
> capable of generating 4416x3312, and the ISP works fine in bypass (and
> therefore raw) mode, so technically it has different maximum sizes
> depending on the format which makes this more exciting.
>
> In any case, the PX30 (assuming the datasheet is correct) only supports
> up to 3264x2448, so the existing #define is incorrect anyway.
>
> I don't have a PX30 nor an RK3288 so I can't test those, and I haven't
> set up my OV64A40 yet which (I've heard) can be used to test even bigger
> resolutions.
>
>
> Paul
>
> [1] https://opensource.rock-chips.com/images/8/87/Rockchip_PX30_Datasheet_V1.4-20191227.pdf
> [2] https://opensource.rock-chips.com/images/4/49/Rockchip_RK3288_Datasheet_V2.7-20191227.pdf
> [3] (requires login) https://www.nxp.com/products/processors-and-microcontrollers/arm-processors/i-mx-applications-processors/i-mx-8-applications-processors/i-mx-8m-plus-arm-cortex-a53-machine-learning-vision-multimedia-and-industrial-iot:IMX8MPLUS
>
>
> >
> > Yeah!
> > >
> > > adam
> > > > regards,
> > > > o.
> > > >
> > > > > > Signed-off-by: Ondrej Jirman <megi@xff.cz>
> > > > > > ---
> > > > > > drivers/media/platform/rockchip/rkisp1/rkisp1-common.h | 4 ++--
> > > > > > 1 file changed, 2 insertions(+), 2 deletions(-)
> > > > > >
> > > > > > diff --git a/drivers/media/platform/rockchip/rkisp1/rkisp1-common.h b/drivers/media/platform/rockchip/rkisp1/rkisp1-common.h
> > > > > > index 4b6b28c05b89..74098ddbeeb3 100644
> > > > > > --- a/drivers/media/platform/rockchip/rkisp1/rkisp1-common.h
> > > > > > +++ b/drivers/media/platform/rockchip/rkisp1/rkisp1-common.h
> > > > > > @@ -33,8 +33,8 @@ struct dentry;
> > > > > > #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_MAX_WIDTH 4416
> > > > > > +#define RKISP1_ISP_MAX_HEIGHT 3312
> > > > > > #define RKISP1_ISP_MIN_WIDTH 32
> > > > > > #define RKISP1_ISP_MIN_HEIGHT 32
> > > > > >
> > > > > --
> > > > > Regards,
> > > > >
> > > > > Laurent Pinchart
> >
_______________________________________________
Linux-rockchip mailing list
Linux-rockchip@lists.infradead.org
http://lists.infradead.org/mailman/listinfo/linux-rockchip
^ permalink raw reply [flat|nested] 27+ messages in thread
* Re: [PATCH] media: rkisp1: Allow higher input resolution
@ 2024-02-20 23:56 ` Ondřej Jirman
0 siblings, 0 replies; 27+ messages in thread
From: Ondřej Jirman @ 2024-02-20 23:56 UTC (permalink / raw)
To: Paul Elder
Cc: Umang Jain, Adam Ford, Laurent Pinchart, linux-kernel,
Dafna Hirschfeld, Mauro Carvalho Chehab, Heiko Stuebner,
open list:ROCKCHIP ISP V1 DRIVER,
open list:ROCKCHIP ISP V1 DRIVER,
moderated list:ARM/Rockchip SoC support
On Mon, Feb 19, 2024 at 07:11:20PM +0900, Paul Elder wrote:
> On Mon, Feb 19, 2024 at 02:33:21PM +0530, Umang Jain wrote:
> > Hi All,
> >
> > On 19/02/24 7:39 am, Adam Ford wrote:
> > > On Sun, Feb 18, 2024 at 3:02 PM Ondřej Jirman <megi@xff.cz> wrote:
> > > > On Sun, Feb 18, 2024 at 10:59:08PM +0200, Laurent Pinchart wrote:
> > > > > Hi Ondrej,
> > > > >
> > > > > (CC'ing Paul and Umang)
> > > > >
> > > > > Thank you for the patch.
> > > > >
> > > > > On Sat, Feb 17, 2024 at 07:51:58PM +0100, Ondřej Jirman wrote:
> > > > > > From: Ondrej Jirman <megi@xff.cz>
> > > > > >
> > > > > > In BSP driver, it is allowed, and it works in practice. Tested on
> > > > > > Pinephone Pro/RK3399 with IMX258 at full res.
> > > > > Paul, Umang, do I recall correctly that you have a similar change ?
> > > > > Could you review and test this (especially on the i.MX8MP) ?
> > > > It's also a limit from the datasheet, so the change should not be that
> > > > controversial:
> > > >
> > > > https://megous.com/dl/tmp/d2b333043ecebaf3.png
> > > >
> > > > (so that it doesn't sound like I just copied the BSP values)
> > > >
> > > From what I see in the i.MX8M Plus reference manual, it has a max
> > > resolution of 4096x3072, so it might be necessary to move this off
> >
> > This is what I (and I assume Paul too) have been working with on i.MX8M
> > Plus. So it's the known and tested value of max ISP input from out side.
> > > from a #define into a structure that varies by product family.
>
> Yes, this is what needs to be done. Here's what I have in my notes:
>
> - The RK3399 TRM says 4416x3312 max input and output on main path, with
> 1920x1080 max output on self path.
> - The PX30 datasheet [1] says 3264x2448 max input and output on main
> path, with 1920x1080 max output on self path.
> - The RK3288 documentation [2] (under "Camera Interface and Image
> Processor") says 4416x3312 max input and output on main path, with
> 1920x1080 max output on self path.
> - The i.MX8MP reference manual (the open one) [3] (in table 13-1) says
> 4096x3072 max resolution in single ISP mode
Yeah, RKISP1_RSZ_SP_SRC_MAX_HEIGHT in the driver also seems weird in
this respect:
#define RKISP1_RSZ_SP_SRC_MAX_WIDTH 1920
#define RKISP1_RSZ_SP_SRC_MAX_HEIGHT 1920
regards,
o.
> The i.MX8M Plus seems to indeed be limited to 4096x3072, but the TPG is
> capable of generating 4416x3312, and the ISP works fine in bypass (and
> therefore raw) mode, so technically it has different maximum sizes
> depending on the format which makes this more exciting.
>
> In any case, the PX30 (assuming the datasheet is correct) only supports
> up to 3264x2448, so the existing #define is incorrect anyway.
>
> I don't have a PX30 nor an RK3288 so I can't test those, and I haven't
> set up my OV64A40 yet which (I've heard) can be used to test even bigger
> resolutions.
>
>
> Paul
>
> [1] https://opensource.rock-chips.com/images/8/87/Rockchip_PX30_Datasheet_V1.4-20191227.pdf
> [2] https://opensource.rock-chips.com/images/4/49/Rockchip_RK3288_Datasheet_V2.7-20191227.pdf
> [3] (requires login) https://www.nxp.com/products/processors-and-microcontrollers/arm-processors/i-mx-applications-processors/i-mx-8-applications-processors/i-mx-8m-plus-arm-cortex-a53-machine-learning-vision-multimedia-and-industrial-iot:IMX8MPLUS
>
>
> >
> > Yeah!
> > >
> > > adam
> > > > regards,
> > > > o.
> > > >
> > > > > > Signed-off-by: Ondrej Jirman <megi@xff.cz>
> > > > > > ---
> > > > > > drivers/media/platform/rockchip/rkisp1/rkisp1-common.h | 4 ++--
> > > > > > 1 file changed, 2 insertions(+), 2 deletions(-)
> > > > > >
> > > > > > diff --git a/drivers/media/platform/rockchip/rkisp1/rkisp1-common.h b/drivers/media/platform/rockchip/rkisp1/rkisp1-common.h
> > > > > > index 4b6b28c05b89..74098ddbeeb3 100644
> > > > > > --- a/drivers/media/platform/rockchip/rkisp1/rkisp1-common.h
> > > > > > +++ b/drivers/media/platform/rockchip/rkisp1/rkisp1-common.h
> > > > > > @@ -33,8 +33,8 @@ struct dentry;
> > > > > > #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_MAX_WIDTH 4416
> > > > > > +#define RKISP1_ISP_MAX_HEIGHT 3312
> > > > > > #define RKISP1_ISP_MIN_WIDTH 32
> > > > > > #define RKISP1_ISP_MIN_HEIGHT 32
> > > > > >
> > > > > --
> > > > > Regards,
> > > > >
> > > > > Laurent Pinchart
> >
_______________________________________________
linux-arm-kernel mailing list
linux-arm-kernel@lists.infradead.org
http://lists.infradead.org/mailman/listinfo/linux-arm-kernel
^ permalink raw reply [flat|nested] 27+ messages in thread
* Re: [PATCH] media: rkisp1: Allow higher input resolution
@ 2024-02-20 23:56 ` Ondřej Jirman
0 siblings, 0 replies; 27+ messages in thread
From: Ondřej Jirman @ 2024-02-20 23:56 UTC (permalink / raw)
To: Paul Elder
Cc: Umang Jain, Adam Ford, Laurent Pinchart, linux-kernel,
Dafna Hirschfeld, Mauro Carvalho Chehab, Heiko Stuebner,
open list:ROCKCHIP ISP V1 DRIVER,
open list:ROCKCHIP ISP V1 DRIVER,
moderated list:ARM/Rockchip SoC support
On Mon, Feb 19, 2024 at 07:11:20PM +0900, Paul Elder wrote:
> On Mon, Feb 19, 2024 at 02:33:21PM +0530, Umang Jain wrote:
> > Hi All,
> >
> > On 19/02/24 7:39 am, Adam Ford wrote:
> > > On Sun, Feb 18, 2024 at 3:02 PM Ondřej Jirman <megi@xff.cz> wrote:
> > > > On Sun, Feb 18, 2024 at 10:59:08PM +0200, Laurent Pinchart wrote:
> > > > > Hi Ondrej,
> > > > >
> > > > > (CC'ing Paul and Umang)
> > > > >
> > > > > Thank you for the patch.
> > > > >
> > > > > On Sat, Feb 17, 2024 at 07:51:58PM +0100, Ondřej Jirman wrote:
> > > > > > From: Ondrej Jirman <megi@xff.cz>
> > > > > >
> > > > > > In BSP driver, it is allowed, and it works in practice. Tested on
> > > > > > Pinephone Pro/RK3399 with IMX258 at full res.
> > > > > Paul, Umang, do I recall correctly that you have a similar change ?
> > > > > Could you review and test this (especially on the i.MX8MP) ?
> > > > It's also a limit from the datasheet, so the change should not be that
> > > > controversial:
> > > >
> > > > https://megous.com/dl/tmp/d2b333043ecebaf3.png
> > > >
> > > > (so that it doesn't sound like I just copied the BSP values)
> > > >
> > > From what I see in the i.MX8M Plus reference manual, it has a max
> > > resolution of 4096x3072, so it might be necessary to move this off
> >
> > This is what I (and I assume Paul too) have been working with on i.MX8M
> > Plus. So it's the known and tested value of max ISP input from out side.
> > > from a #define into a structure that varies by product family.
>
> Yes, this is what needs to be done. Here's what I have in my notes:
>
> - The RK3399 TRM says 4416x3312 max input and output on main path, with
> 1920x1080 max output on self path.
> - The PX30 datasheet [1] says 3264x2448 max input and output on main
> path, with 1920x1080 max output on self path.
> - The RK3288 documentation [2] (under "Camera Interface and Image
> Processor") says 4416x3312 max input and output on main path, with
> 1920x1080 max output on self path.
> - The i.MX8MP reference manual (the open one) [3] (in table 13-1) says
> 4096x3072 max resolution in single ISP mode
Yeah, RKISP1_RSZ_SP_SRC_MAX_HEIGHT in the driver also seems weird in
this respect:
#define RKISP1_RSZ_SP_SRC_MAX_WIDTH 1920
#define RKISP1_RSZ_SP_SRC_MAX_HEIGHT 1920
regards,
o.
> The i.MX8M Plus seems to indeed be limited to 4096x3072, but the TPG is
> capable of generating 4416x3312, and the ISP works fine in bypass (and
> therefore raw) mode, so technically it has different maximum sizes
> depending on the format which makes this more exciting.
>
> In any case, the PX30 (assuming the datasheet is correct) only supports
> up to 3264x2448, so the existing #define is incorrect anyway.
>
> I don't have a PX30 nor an RK3288 so I can't test those, and I haven't
> set up my OV64A40 yet which (I've heard) can be used to test even bigger
> resolutions.
>
>
> Paul
>
> [1] https://opensource.rock-chips.com/images/8/87/Rockchip_PX30_Datasheet_V1.4-20191227.pdf
> [2] https://opensource.rock-chips.com/images/4/49/Rockchip_RK3288_Datasheet_V2.7-20191227.pdf
> [3] (requires login) https://www.nxp.com/products/processors-and-microcontrollers/arm-processors/i-mx-applications-processors/i-mx-8-applications-processors/i-mx-8m-plus-arm-cortex-a53-machine-learning-vision-multimedia-and-industrial-iot:IMX8MPLUS
>
>
> >
> > Yeah!
> > >
> > > adam
> > > > regards,
> > > > o.
> > > >
> > > > > > Signed-off-by: Ondrej Jirman <megi@xff.cz>
> > > > > > ---
> > > > > > drivers/media/platform/rockchip/rkisp1/rkisp1-common.h | 4 ++--
> > > > > > 1 file changed, 2 insertions(+), 2 deletions(-)
> > > > > >
> > > > > > diff --git a/drivers/media/platform/rockchip/rkisp1/rkisp1-common.h b/drivers/media/platform/rockchip/rkisp1/rkisp1-common.h
> > > > > > index 4b6b28c05b89..74098ddbeeb3 100644
> > > > > > --- a/drivers/media/platform/rockchip/rkisp1/rkisp1-common.h
> > > > > > +++ b/drivers/media/platform/rockchip/rkisp1/rkisp1-common.h
> > > > > > @@ -33,8 +33,8 @@ struct dentry;
> > > > > > #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_MAX_WIDTH 4416
> > > > > > +#define RKISP1_ISP_MAX_HEIGHT 3312
> > > > > > #define RKISP1_ISP_MIN_WIDTH 32
> > > > > > #define RKISP1_ISP_MIN_HEIGHT 32
> > > > > >
> > > > > --
> > > > > Regards,
> > > > >
> > > > > Laurent Pinchart
> >
^ permalink raw reply [flat|nested] 27+ messages in thread
* Re: [PATCH] media: rkisp1: Allow higher input resolution
2024-02-19 11:54 ` Laurent Pinchart
(?)
@ 2024-02-20 23:59 ` Ondřej Jirman
-1 siblings, 0 replies; 27+ messages in thread
From: Ondřej Jirman @ 2024-02-20 23:59 UTC (permalink / raw)
To: Laurent Pinchart
Cc: Paul Elder, Umang Jain, Adam Ford, linux-kernel,
Dafna Hirschfeld, Mauro Carvalho Chehab, Heiko Stuebner,
open list:ROCKCHIP ISP V1 DRIVER,
open list:ROCKCHIP ISP V1 DRIVER,
moderated list:ARM/Rockchip SoC support
Hello Laurent,
On Mon, Feb 19, 2024 at 01:54:30PM +0200, Laurent Pinchart wrote:
> On Mon, Feb 19, 2024 at 07:11:20PM +0900, Paul Elder wrote:
> > On Mon, Feb 19, 2024 at 02:33:21PM +0530, Umang Jain wrote:
> > > On 19/02/24 7:39 am, Adam Ford wrote:
> > > > On Sun, Feb 18, 2024 at 3:02 PM Ondřej Jirman <megi@xff.cz> wrote:
> > > > > On Sun, Feb 18, 2024 at 10:59:08PM +0200, Laurent Pinchart wrote:
> > > > > > On Sat, Feb 17, 2024 at 07:51:58PM +0100, Ondřej Jirman wrote:
> > > > > > > From: Ondrej Jirman <megi@xff.cz>
> > > > > > >
> > > > > > > In BSP driver, it is allowed, and it works in practice. Tested on
> > > > > > > Pinephone Pro/RK3399 with IMX258 at full res.
> > > > > >
> > > > > > Paul, Umang, do I recall correctly that you have a similar change ?
> > > > > > Could you review and test this (especially on the i.MX8MP) ?
> > > > >
> > > > > It's also a limit from the datasheet, so the change should not be that
> > > > > controversial:
> > > > >
> > > > > https://megous.com/dl/tmp/d2b333043ecebaf3.png
> > > > >
> > > > > (so that it doesn't sound like I just copied the BSP values)
> > > > >
> > > > From what I see in the i.MX8M Plus reference manual, it has a max
> > > > resolution of 4096x3072, so it might be necessary to move this off
> > >
> > > This is what I (and I assume Paul too) have been working with on i.MX8M
> > > Plus. So it's the known and tested value of max ISP input from out side.
> > >
> > > > from a #define into a structure that varies by product family.
> >
> > Yes, this is what needs to be done. Here's what I have in my notes:
> >
> > - The RK3399 TRM says 4416x3312 max input and output on main path, with
> > 1920x1080 max output on self path.
> > - The PX30 datasheet [1] says 3264x2448 max input and output on main
> > path, with 1920x1080 max output on self path.
> > - The RK3288 documentation [2] (under "Camera Interface and Image
> > Processor") says 4416x3312 max input and output on main path, with
> > 1920x1080 max output on self path.
> > - The i.MX8MP reference manual (the open one) [3] (in table 13-1) says
> > 4096x3072 max resolution in single ISP mode
> >
> > The i.MX8M Plus seems to indeed be limited to 4096x3072, but the TPG is
> > capable of generating 4416x3312, and the ISP works fine in bypass (and
> > therefore raw) mode, so technically it has different maximum sizes
> > depending on the format which makes this more exciting.
> >
> > In any case, the PX30 (assuming the datasheet is correct) only supports
> > up to 3264x2448, so the existing #define is incorrect anyway.
> >
> > I don't have a PX30 nor an RK3288 so I can't test those, and I haven't
> > set up my OV64A40 yet which (I've heard) can be used to test even bigger
> > resolutions.
>
> Thanks for the summary Paul.
>
> Ondřej, would you be interested in implementing this ?
Something like this?
https://lore.kernel.org/lkml/20240220235720.3010608-1-megi@xff.cz/T/#u
Only the first two SoCs are supported by rkisp1 driver upstream, yet. So I
only have to deal with input and mainpath maximum ATM. So the patch is rather
simple.
kind regards,
o.
> > [1] https://opensource.rock-chips.com/images/8/87/Rockchip_PX30_Datasheet_V1.4-20191227.pdf
> > [2] https://opensource.rock-chips.com/images/4/49/Rockchip_RK3288_Datasheet_V2.7-20191227.pdf
> > [3] (requires login) https://www.nxp.com/products/processors-and-microcontrollers/arm-processors/i-mx-applications-processors/i-mx-8-applications-processors/i-mx-8m-plus-arm-cortex-a53-machine-learning-vision-multimedia-and-industrial-iot:IMX8MPLUS
> >
> > > > > > > Signed-off-by: Ondrej Jirman <megi@xff.cz>
> > > > > > > ---
> > > > > > > drivers/media/platform/rockchip/rkisp1/rkisp1-common.h | 4 ++--
> > > > > > > 1 file changed, 2 insertions(+), 2 deletions(-)
> > > > > > >
> > > > > > > diff --git a/drivers/media/platform/rockchip/rkisp1/rkisp1-common.h b/drivers/media/platform/rockchip/rkisp1/rkisp1-common.h
> > > > > > > index 4b6b28c05b89..74098ddbeeb3 100644
> > > > > > > --- a/drivers/media/platform/rockchip/rkisp1/rkisp1-common.h
> > > > > > > +++ b/drivers/media/platform/rockchip/rkisp1/rkisp1-common.h
> > > > > > > @@ -33,8 +33,8 @@ struct dentry;
> > > > > > > #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_MAX_WIDTH 4416
> > > > > > > +#define RKISP1_ISP_MAX_HEIGHT 3312
> > > > > > > #define RKISP1_ISP_MIN_WIDTH 32
> > > > > > > #define RKISP1_ISP_MIN_HEIGHT 32
> > > > > > >
>
> --
> Regards,
>
> Laurent Pinchart
_______________________________________________
Linux-rockchip mailing list
Linux-rockchip@lists.infradead.org
http://lists.infradead.org/mailman/listinfo/linux-rockchip
^ permalink raw reply [flat|nested] 27+ messages in thread
* Re: [PATCH] media: rkisp1: Allow higher input resolution
@ 2024-02-20 23:59 ` Ondřej Jirman
0 siblings, 0 replies; 27+ messages in thread
From: Ondřej Jirman @ 2024-02-20 23:59 UTC (permalink / raw)
To: Laurent Pinchart
Cc: Paul Elder, Umang Jain, Adam Ford, linux-kernel,
Dafna Hirschfeld, Mauro Carvalho Chehab, Heiko Stuebner,
open list:ROCKCHIP ISP V1 DRIVER,
open list:ROCKCHIP ISP V1 DRIVER,
moderated list:ARM/Rockchip SoC support
Hello Laurent,
On Mon, Feb 19, 2024 at 01:54:30PM +0200, Laurent Pinchart wrote:
> On Mon, Feb 19, 2024 at 07:11:20PM +0900, Paul Elder wrote:
> > On Mon, Feb 19, 2024 at 02:33:21PM +0530, Umang Jain wrote:
> > > On 19/02/24 7:39 am, Adam Ford wrote:
> > > > On Sun, Feb 18, 2024 at 3:02 PM Ondřej Jirman <megi@xff.cz> wrote:
> > > > > On Sun, Feb 18, 2024 at 10:59:08PM +0200, Laurent Pinchart wrote:
> > > > > > On Sat, Feb 17, 2024 at 07:51:58PM +0100, Ondřej Jirman wrote:
> > > > > > > From: Ondrej Jirman <megi@xff.cz>
> > > > > > >
> > > > > > > In BSP driver, it is allowed, and it works in practice. Tested on
> > > > > > > Pinephone Pro/RK3399 with IMX258 at full res.
> > > > > >
> > > > > > Paul, Umang, do I recall correctly that you have a similar change ?
> > > > > > Could you review and test this (especially on the i.MX8MP) ?
> > > > >
> > > > > It's also a limit from the datasheet, so the change should not be that
> > > > > controversial:
> > > > >
> > > > > https://megous.com/dl/tmp/d2b333043ecebaf3.png
> > > > >
> > > > > (so that it doesn't sound like I just copied the BSP values)
> > > > >
> > > > From what I see in the i.MX8M Plus reference manual, it has a max
> > > > resolution of 4096x3072, so it might be necessary to move this off
> > >
> > > This is what I (and I assume Paul too) have been working with on i.MX8M
> > > Plus. So it's the known and tested value of max ISP input from out side.
> > >
> > > > from a #define into a structure that varies by product family.
> >
> > Yes, this is what needs to be done. Here's what I have in my notes:
> >
> > - The RK3399 TRM says 4416x3312 max input and output on main path, with
> > 1920x1080 max output on self path.
> > - The PX30 datasheet [1] says 3264x2448 max input and output on main
> > path, with 1920x1080 max output on self path.
> > - The RK3288 documentation [2] (under "Camera Interface and Image
> > Processor") says 4416x3312 max input and output on main path, with
> > 1920x1080 max output on self path.
> > - The i.MX8MP reference manual (the open one) [3] (in table 13-1) says
> > 4096x3072 max resolution in single ISP mode
> >
> > The i.MX8M Plus seems to indeed be limited to 4096x3072, but the TPG is
> > capable of generating 4416x3312, and the ISP works fine in bypass (and
> > therefore raw) mode, so technically it has different maximum sizes
> > depending on the format which makes this more exciting.
> >
> > In any case, the PX30 (assuming the datasheet is correct) only supports
> > up to 3264x2448, so the existing #define is incorrect anyway.
> >
> > I don't have a PX30 nor an RK3288 so I can't test those, and I haven't
> > set up my OV64A40 yet which (I've heard) can be used to test even bigger
> > resolutions.
>
> Thanks for the summary Paul.
>
> Ondřej, would you be interested in implementing this ?
Something like this?
https://lore.kernel.org/lkml/20240220235720.3010608-1-megi@xff.cz/T/#u
Only the first two SoCs are supported by rkisp1 driver upstream, yet. So I
only have to deal with input and mainpath maximum ATM. So the patch is rather
simple.
kind regards,
o.
> > [1] https://opensource.rock-chips.com/images/8/87/Rockchip_PX30_Datasheet_V1.4-20191227.pdf
> > [2] https://opensource.rock-chips.com/images/4/49/Rockchip_RK3288_Datasheet_V2.7-20191227.pdf
> > [3] (requires login) https://www.nxp.com/products/processors-and-microcontrollers/arm-processors/i-mx-applications-processors/i-mx-8-applications-processors/i-mx-8m-plus-arm-cortex-a53-machine-learning-vision-multimedia-and-industrial-iot:IMX8MPLUS
> >
> > > > > > > Signed-off-by: Ondrej Jirman <megi@xff.cz>
> > > > > > > ---
> > > > > > > drivers/media/platform/rockchip/rkisp1/rkisp1-common.h | 4 ++--
> > > > > > > 1 file changed, 2 insertions(+), 2 deletions(-)
> > > > > > >
> > > > > > > diff --git a/drivers/media/platform/rockchip/rkisp1/rkisp1-common.h b/drivers/media/platform/rockchip/rkisp1/rkisp1-common.h
> > > > > > > index 4b6b28c05b89..74098ddbeeb3 100644
> > > > > > > --- a/drivers/media/platform/rockchip/rkisp1/rkisp1-common.h
> > > > > > > +++ b/drivers/media/platform/rockchip/rkisp1/rkisp1-common.h
> > > > > > > @@ -33,8 +33,8 @@ struct dentry;
> > > > > > > #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_MAX_WIDTH 4416
> > > > > > > +#define RKISP1_ISP_MAX_HEIGHT 3312
> > > > > > > #define RKISP1_ISP_MIN_WIDTH 32
> > > > > > > #define RKISP1_ISP_MIN_HEIGHT 32
> > > > > > >
>
> --
> Regards,
>
> Laurent Pinchart
^ permalink raw reply [flat|nested] 27+ messages in thread
* Re: [PATCH] media: rkisp1: Allow higher input resolution
@ 2024-02-20 23:59 ` Ondřej Jirman
0 siblings, 0 replies; 27+ messages in thread
From: Ondřej Jirman @ 2024-02-20 23:59 UTC (permalink / raw)
To: Laurent Pinchart
Cc: Paul Elder, Umang Jain, Adam Ford, linux-kernel,
Dafna Hirschfeld, Mauro Carvalho Chehab, Heiko Stuebner,
open list:ROCKCHIP ISP V1 DRIVER,
open list:ROCKCHIP ISP V1 DRIVER,
moderated list:ARM/Rockchip SoC support
Hello Laurent,
On Mon, Feb 19, 2024 at 01:54:30PM +0200, Laurent Pinchart wrote:
> On Mon, Feb 19, 2024 at 07:11:20PM +0900, Paul Elder wrote:
> > On Mon, Feb 19, 2024 at 02:33:21PM +0530, Umang Jain wrote:
> > > On 19/02/24 7:39 am, Adam Ford wrote:
> > > > On Sun, Feb 18, 2024 at 3:02 PM Ondřej Jirman <megi@xff.cz> wrote:
> > > > > On Sun, Feb 18, 2024 at 10:59:08PM +0200, Laurent Pinchart wrote:
> > > > > > On Sat, Feb 17, 2024 at 07:51:58PM +0100, Ondřej Jirman wrote:
> > > > > > > From: Ondrej Jirman <megi@xff.cz>
> > > > > > >
> > > > > > > In BSP driver, it is allowed, and it works in practice. Tested on
> > > > > > > Pinephone Pro/RK3399 with IMX258 at full res.
> > > > > >
> > > > > > Paul, Umang, do I recall correctly that you have a similar change ?
> > > > > > Could you review and test this (especially on the i.MX8MP) ?
> > > > >
> > > > > It's also a limit from the datasheet, so the change should not be that
> > > > > controversial:
> > > > >
> > > > > https://megous.com/dl/tmp/d2b333043ecebaf3.png
> > > > >
> > > > > (so that it doesn't sound like I just copied the BSP values)
> > > > >
> > > > From what I see in the i.MX8M Plus reference manual, it has a max
> > > > resolution of 4096x3072, so it might be necessary to move this off
> > >
> > > This is what I (and I assume Paul too) have been working with on i.MX8M
> > > Plus. So it's the known and tested value of max ISP input from out side.
> > >
> > > > from a #define into a structure that varies by product family.
> >
> > Yes, this is what needs to be done. Here's what I have in my notes:
> >
> > - The RK3399 TRM says 4416x3312 max input and output on main path, with
> > 1920x1080 max output on self path.
> > - The PX30 datasheet [1] says 3264x2448 max input and output on main
> > path, with 1920x1080 max output on self path.
> > - The RK3288 documentation [2] (under "Camera Interface and Image
> > Processor") says 4416x3312 max input and output on main path, with
> > 1920x1080 max output on self path.
> > - The i.MX8MP reference manual (the open one) [3] (in table 13-1) says
> > 4096x3072 max resolution in single ISP mode
> >
> > The i.MX8M Plus seems to indeed be limited to 4096x3072, but the TPG is
> > capable of generating 4416x3312, and the ISP works fine in bypass (and
> > therefore raw) mode, so technically it has different maximum sizes
> > depending on the format which makes this more exciting.
> >
> > In any case, the PX30 (assuming the datasheet is correct) only supports
> > up to 3264x2448, so the existing #define is incorrect anyway.
> >
> > I don't have a PX30 nor an RK3288 so I can't test those, and I haven't
> > set up my OV64A40 yet which (I've heard) can be used to test even bigger
> > resolutions.
>
> Thanks for the summary Paul.
>
> Ondřej, would you be interested in implementing this ?
Something like this?
https://lore.kernel.org/lkml/20240220235720.3010608-1-megi@xff.cz/T/#u
Only the first two SoCs are supported by rkisp1 driver upstream, yet. So I
only have to deal with input and mainpath maximum ATM. So the patch is rather
simple.
kind regards,
o.
> > [1] https://opensource.rock-chips.com/images/8/87/Rockchip_PX30_Datasheet_V1.4-20191227.pdf
> > [2] https://opensource.rock-chips.com/images/4/49/Rockchip_RK3288_Datasheet_V2.7-20191227.pdf
> > [3] (requires login) https://www.nxp.com/products/processors-and-microcontrollers/arm-processors/i-mx-applications-processors/i-mx-8-applications-processors/i-mx-8m-plus-arm-cortex-a53-machine-learning-vision-multimedia-and-industrial-iot:IMX8MPLUS
> >
> > > > > > > Signed-off-by: Ondrej Jirman <megi@xff.cz>
> > > > > > > ---
> > > > > > > drivers/media/platform/rockchip/rkisp1/rkisp1-common.h | 4 ++--
> > > > > > > 1 file changed, 2 insertions(+), 2 deletions(-)
> > > > > > >
> > > > > > > diff --git a/drivers/media/platform/rockchip/rkisp1/rkisp1-common.h b/drivers/media/platform/rockchip/rkisp1/rkisp1-common.h
> > > > > > > index 4b6b28c05b89..74098ddbeeb3 100644
> > > > > > > --- a/drivers/media/platform/rockchip/rkisp1/rkisp1-common.h
> > > > > > > +++ b/drivers/media/platform/rockchip/rkisp1/rkisp1-common.h
> > > > > > > @@ -33,8 +33,8 @@ struct dentry;
> > > > > > > #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_MAX_WIDTH 4416
> > > > > > > +#define RKISP1_ISP_MAX_HEIGHT 3312
> > > > > > > #define RKISP1_ISP_MIN_WIDTH 32
> > > > > > > #define RKISP1_ISP_MIN_HEIGHT 32
> > > > > > >
>
> --
> Regards,
>
> Laurent Pinchart
_______________________________________________
linux-arm-kernel mailing list
linux-arm-kernel@lists.infradead.org
http://lists.infradead.org/mailman/listinfo/linux-arm-kernel
^ permalink raw reply [flat|nested] 27+ messages in thread
end of thread, other threads:[~2024-02-21 0:03 UTC | newest]
Thread overview: 27+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2024-02-17 18:51 [PATCH] media: rkisp1: Allow higher input resolution Ondřej Jirman
2024-02-17 18:51 ` Ondřej Jirman
2024-02-17 18:51 ` Ondřej Jirman
2024-02-18 20:59 ` Laurent Pinchart
2024-02-18 20:59 ` Laurent Pinchart
2024-02-18 20:59 ` Laurent Pinchart
2024-02-18 21:01 ` Ondřej Jirman
2024-02-18 21:01 ` Ondřej Jirman
2024-02-18 21:01 ` Ondřej Jirman
2024-02-19 2:09 ` Adam Ford
2024-02-19 2:09 ` Adam Ford
2024-02-19 2:09 ` Adam Ford
2024-02-19 9:03 ` Umang Jain
2024-02-19 9:03 ` Umang Jain
2024-02-19 9:03 ` Umang Jain
2024-02-19 10:11 ` Paul Elder
2024-02-19 10:11 ` Paul Elder
2024-02-19 10:11 ` Paul Elder
2024-02-19 11:54 ` Laurent Pinchart
2024-02-19 11:54 ` Laurent Pinchart
2024-02-19 11:54 ` Laurent Pinchart
2024-02-20 23:59 ` Ondřej Jirman
2024-02-20 23:59 ` Ondřej Jirman
2024-02-20 23:59 ` Ondřej Jirman
2024-02-20 23:56 ` Ondřej Jirman
2024-02-20 23:56 ` Ondřej Jirman
2024-02-20 23:56 ` Ondřej Jirman
This is an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.