On Tue Feb 25 01:15:43 2025 +0000, Dr. David Alan Gilbert wrote:
> omap3isp_print_status() was added in 2011 by
> commit 448de7e7850b ("[media] omap3isp: OMAP3 ISP core")
> but has remained unused.
> 
> Remove it (and it's associated #defines).
> 
> Signed-off-by: Dr. David Alan Gilbert <li...@treblig.org>
> Signed-off-by: Hans Verkuil <hverk...@xs4all.nl>

Patch committed.

Thanks,
Hans Verkuil

 drivers/media/platform/ti/omap3isp/isp.c | 37 --------------------------------
 drivers/media/platform/ti/omap3isp/isp.h |  2 --
 2 files changed, 39 deletions(-)

---

diff --git a/drivers/media/platform/ti/omap3isp/isp.c 
b/drivers/media/platform/ti/omap3isp/isp.c
index 7d549803fb92..f51cf6119e97 100644
--- a/drivers/media/platform/ti/omap3isp/isp.c
+++ b/drivers/media/platform/ti/omap3isp/isp.c
@@ -1475,43 +1475,6 @@ void omap3isp_put(struct isp_device *isp)
  * Platform device driver
  */
 
-/*
- * omap3isp_print_status - Prints the values of the ISP Control Module 
registers
- * @isp: OMAP3 ISP device
- */
-#define ISP_PRINT_REGISTER(isp, name)\
-       dev_dbg(isp->dev, "###ISP " #name "=0x%08x\n", \
-               isp_reg_readl(isp, OMAP3_ISP_IOMEM_MAIN, ISP_##name))
-#define SBL_PRINT_REGISTER(isp, name)\
-       dev_dbg(isp->dev, "###SBL " #name "=0x%08x\n", \
-               isp_reg_readl(isp, OMAP3_ISP_IOMEM_SBL, ISPSBL_##name))
-
-void omap3isp_print_status(struct isp_device *isp)
-{
-       dev_dbg(isp->dev, "-------------ISP Register dump--------------\n");
-
-       ISP_PRINT_REGISTER(isp, SYSCONFIG);
-       ISP_PRINT_REGISTER(isp, SYSSTATUS);
-       ISP_PRINT_REGISTER(isp, IRQ0ENABLE);
-       ISP_PRINT_REGISTER(isp, IRQ0STATUS);
-       ISP_PRINT_REGISTER(isp, TCTRL_GRESET_LENGTH);
-       ISP_PRINT_REGISTER(isp, TCTRL_PSTRB_REPLAY);
-       ISP_PRINT_REGISTER(isp, CTRL);
-       ISP_PRINT_REGISTER(isp, TCTRL_CTRL);
-       ISP_PRINT_REGISTER(isp, TCTRL_FRAME);
-       ISP_PRINT_REGISTER(isp, TCTRL_PSTRB_DELAY);
-       ISP_PRINT_REGISTER(isp, TCTRL_STRB_DELAY);
-       ISP_PRINT_REGISTER(isp, TCTRL_SHUT_DELAY);
-       ISP_PRINT_REGISTER(isp, TCTRL_PSTRB_LENGTH);
-       ISP_PRINT_REGISTER(isp, TCTRL_STRB_LENGTH);
-       ISP_PRINT_REGISTER(isp, TCTRL_SHUT_LENGTH);
-
-       SBL_PRINT_REGISTER(isp, PCR);
-       SBL_PRINT_REGISTER(isp, SDR_REQ_EXP);
-
-       dev_dbg(isp->dev, "--------------------------------------------\n");
-}
-
 #ifdef CONFIG_PM
 
 /*
diff --git a/drivers/media/platform/ti/omap3isp/isp.h 
b/drivers/media/platform/ti/omap3isp/isp.h
index b4793631ad97..60acf3401ac9 100644
--- a/drivers/media/platform/ti/omap3isp/isp.h
+++ b/drivers/media/platform/ti/omap3isp/isp.h
@@ -260,8 +260,6 @@ void omap3isp_configure_bridge(struct isp_device *isp,
 struct isp_device *omap3isp_get(struct isp_device *isp);
 void omap3isp_put(struct isp_device *isp);
 
-void omap3isp_print_status(struct isp_device *isp);
-
 void omap3isp_sbl_enable(struct isp_device *isp, enum isp_sbl_resource res);
 void omap3isp_sbl_disable(struct isp_device *isp, enum isp_sbl_resource res);
 

Reply via email to