All of lore.kernel.org
 help / color / mirror / Atom feed
* [PATCH 0/5] media: rkisp1: Fix LSC initial configuration on i.MX8MP
@ 2022-08-17  2:18 ` Laurent Pinchart
  0 siblings, 0 replies; 46+ messages in thread
From: Laurent Pinchart @ 2022-08-17  2:18 UTC (permalink / raw)
  To: linux-media
  Cc: Dafna Hirschfeld, Heiko Stuebner, linux-rockchip, Paul Elder,
	Florian Sylvestre

Hello,

This patch series fixes the Lens Shading Correction initial
configuration on the i.MX8MP.

The i.MX8MP integrates an ISP8000Nano v18.02, which unlike other
versions currently supported by the driver, gates access to the LSC RAM
with the ISP_CTRL.ISP_ENABLE bit. The initial LSC configuration being
performed before the ISP gets enabled, the writes to the RAM are
ignored, leading to incorrect results.

The series starts with four small drive-by cleanups of the LSC code, and
patch 5/5 then fixes the issue. I'm not totally thrilled by the code
architecture, but I'm not sure why, and I have a feeling doing better
would require a large refactoring of the ISP parameters handling. If
anyone sees an option for a better implementation, please say so.

The series is based on top of "[PATCH 0/7] media: rkisp1: Fix and
improve color space support" ([1]). Reviews for that base series would
thus be appreciated too.

[1] https://lore.kernel.org/linux-media/20220815065235.23797-1-laurent.pinchart@ideasonboard.com

Laurent Pinchart (5):
  media: rkisp1: Clean up LSC configuration code
  media: rkisp1: Store LSC register values in u32 variables
  media: rkisp1: Simplify LSC x/y size and grad register macros
  media: rkisp1: Use RKISP1_CIF_ISP_LSC_GRAD_SIZE() for gradient
    registers
  media: rkisp1: Configure LSC after enabling the ISP

 .../platform/rockchip/rkisp1/rkisp1-common.h  |  29 +-
 .../platform/rockchip/rkisp1/rkisp1-isp.c     |   9 +-
 .../platform/rockchip/rkisp1/rkisp1-params.c  | 378 ++++++++++--------
 .../platform/rockchip/rkisp1/rkisp1-regs.h    |  20 +-
 4 files changed, 239 insertions(+), 197 deletions(-)

-- 
Regards,

Laurent Pinchart


^ permalink raw reply	[flat|nested] 46+ messages in thread

end of thread, other threads:[~2022-09-04 13:13 UTC | newest]

Thread overview: 46+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2022-08-17  2:18 [PATCH 0/5] media: rkisp1: Fix LSC initial configuration on i.MX8MP Laurent Pinchart
2022-08-17  2:18 ` Laurent Pinchart
2022-08-17  2:18 ` [PATCH 1/5] media: rkisp1: Clean up LSC configuration code Laurent Pinchart
2022-08-17  2:18   ` Laurent Pinchart
2022-08-18  2:32   ` Dafna Hirschfeld
2022-08-18  2:32     ` Dafna Hirschfeld
2022-08-17  2:18 ` [PATCH 2/5] media: rkisp1: Store LSC register values in u32 variables Laurent Pinchart
2022-08-17  2:18   ` Laurent Pinchart
2022-08-18  2:37   ` Dafna Hirschfeld
2022-08-18  2:37     ` Dafna Hirschfeld
2022-08-17  2:18 ` [PATCH 3/5] media: rkisp1: Simplify LSC x/y size and grad register macros Laurent Pinchart
2022-08-17  2:18   ` Laurent Pinchart
2022-08-18  2:39   ` Dafna Hirschfeld
2022-08-18  2:39     ` Dafna Hirschfeld
2022-08-17  2:18 ` [PATCH 4/5] media: rkisp1: Use RKISP1_CIF_ISP_LSC_GRAD_SIZE() for gradient registers Laurent Pinchart
2022-08-17  2:18   ` Laurent Pinchart
2022-08-18  2:50   ` Dafna Hirschfeld
2022-08-18  2:50     ` Dafna Hirschfeld
2022-08-18  6:17     ` Laurent Pinchart
2022-08-18  6:17       ` Laurent Pinchart
2022-08-23 17:21   ` [PATCH v1.1 4/5] media: rkisp1: Use correct macro " Laurent Pinchart
2022-08-23 17:21     ` Laurent Pinchart
2022-08-26 19:06     ` Dafna Hirschfeld
2022-08-26 19:06       ` Dafna Hirschfeld
2022-08-17  2:18 ` [PATCH 5/5] media: rkisp1: Configure LSC after enabling the ISP Laurent Pinchart
2022-08-17  2:18   ` Laurent Pinchart
2022-08-18  3:45   ` Dafna Hirschfeld
2022-08-18  3:45     ` Dafna Hirschfeld
2022-08-18  6:25     ` Laurent Pinchart
2022-08-18  6:25       ` Laurent Pinchart
2022-08-22 17:08       ` Dafna Hirschfeld
2022-08-22 17:08         ` Dafna Hirschfeld
2022-08-23 17:21   ` [PATCH v1.1 " Laurent Pinchart
2022-08-23 17:21     ` Laurent Pinchart
2022-09-03  3:09     ` Dafna Hirschfeld
2022-09-03  3:09       ` Dafna Hirschfeld
2022-09-03 20:39       ` Laurent Pinchart
2022-09-03 20:39         ` Laurent Pinchart
2022-09-03 21:01     ` [PATCH v1.2 " Laurent Pinchart
2022-09-03 21:01       ` Laurent Pinchart
2022-09-04 13:12       ` Dafna Hirschfeld
2022-09-04 13:12         ` Dafna Hirschfeld
2022-09-02  9:07 ` [PATCH 0/5] media: rkisp1: Fix LSC initial configuration on i.MX8MP Dafna Hirschfeld
2022-09-02  9:07   ` Dafna Hirschfeld
2022-09-02 10:07   ` Laurent Pinchart
2022-09-02 10:07     ` Laurent Pinchart

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.