Hello, This series fixes multiple issues and performs further cleanups for the rkisp1 driver. Patches 01/21 to 07/21 fix issues in the error paths at probe time that lead to kernel crashes in case of errors. The .remove() path is also improved as a result. Patches 08/21 and 09/21 then clean up register definitions, patch 10/21 swaps the arguments to rkisp1_write(), and patch 11/21 simplifies access to the resizer registers. Patches 12/21 to 18/21 move on to improving debugfs support, with code first moved to a separate file (12/21) that can be conditionally compiled (13/21), support for collecting statistics on the input interface (14/21, very useful to check if the ISP actually receives data) and for dumping core and ISP registers (15/21). Patch 16/21 continues that rework by moving the resizer register dumps support to debugfs too, and patch 17/21 consolidates the resizer dumps code with the core and ISP. Patch 18/21 completes the work by making the register name length dynamic. Finally, patch 19/21 simplifies the V4L2 querycap implementation on video nodes, and patches 20/21 and 21/21 fix alignment and indentation. More fixes and improvements should follow, so I'd like to merge sooner than latter if possible, to avoid having to rebase large series. Laurent Pinchart (20): media: rkisp1: capture: Initialize entity before video device media: rkisp1: capture: Fix and simplify (un)registration media: rkisp1: isp: Fix and simplify (un)registration media: rkisp1: resizer: Fix and simplify (un)registration media: rkisp1: params: Fix and simplify (un)registration media: rkisp1: stats: Simplify (un)registration media: rkisp1: Simplify rkisp1_entities_register() error path media: rkisp1: regs: Don't use BIT() macro for multi-bit register fields media: rkisp1: regs: Rename CCL, ICCL and IRCL registers with VI_ prefix media: rkisp1: Swap value and address arguments to rkisp1_write() media: rkisp1: resizer: Simplify register access media: rkisp1: Move debugfs code to a separate file media: rkisp1: Compile debugfs support conditionally media: rkisp1: debug: Collect input status by sampling ISP_FLAGS_SHD media: rkisp1: debug: Add debugfs files to dump core and ISP registers media: rkisp1: debug: Move resizer register dump to debugfs media: rkisp1: debug: Consolidate reg dumps for shadow registers media: rkisp1: debug: Compute max register length name dynamically media: rkisp1: Align macro definitions media: rkisp1: Drop parentheses and fix indentation in rkisp1_probe() Paul Elder (1): media: rkisp1: capture: Bypass the main device for handling querycap .../media/platform/rockchip/rkisp1/Makefile | 17 +- .../platform/rockchip/rkisp1/rkisp1-capture.c | 159 ++-- .../platform/rockchip/rkisp1/rkisp1-common.h | 48 +- .../platform/rockchip/rkisp1/rkisp1-debug.c | 225 ++++++ .../platform/rockchip/rkisp1/rkisp1-dev.c | 79 +- .../platform/rockchip/rkisp1/rkisp1-isp.c | 156 ++-- .../platform/rockchip/rkisp1/rkisp1-params.c | 705 +++++++++--------- .../platform/rockchip/rkisp1/rkisp1-regs.h | 190 +++-- .../platform/rockchip/rkisp1/rkisp1-resizer.c | 204 ++--- .../platform/rockchip/rkisp1/rkisp1-stats.c | 13 +- 10 files changed, 952 insertions(+), 844 deletions(-) create mode 100644 drivers/media/platform/rockchip/rkisp1/rkisp1-debug.c -- Regards, Laurent Pinchart