Re: [PATCH v7 6/9] media: chips-media: wave6: Improve debugging capabilities

On Thu, Sep 10, 2026 at 04:38:27AM +0000, Nas Chung wrote:

Hi, Frank.

quoted

-----Original Message-----
From: Frank Li <redacted>
Sent: Thursday, September 10, 2026 5:41 AM
To: Nas Chung <nas.chung@chipsnmedia.com>
Cc: mchehab@kernel.org; hverkuil@xs4all.nl; robh@kernel.org;
krzk+dt@kernel.org; conor+dt@kernel.org; shawnguo@kernel.org;
s.hauer@pengutronix.de; linux-media@vger.kernel.org;
devicetree@vger.kernel.org; linux-kernel@vger.kernel.org; linux-imx@nxp.com;
linux-arm-kernel@lists.infradead.org; jackson.lee
[off-list ref]; lafley.kim [off-list ref];
marek.vasut@mailbox.org; Ming Qian [off-list ref]
Subject: Re: [PATCH v7 6/9] media: chips-media: wave6: Improve debugging
capabilities

On Fri, Sep 04, 2026 at 03:46:32PM +0900, Nas Chung wrote:

media: chips-media: wave6: Add trace and debugfs

I'll address this in v8.

quoted

quoted

Add debugfs entries and trace events to provide detailed
debugging information.

Nit: need extra space line here.

OK.

quoted

quoted

These enhancements help diagnose issues and improve debugging

Remove "These enhancements".

I'll remove it.

...

quoted

quoted

+
+#ifndef __WAVE6_VPU_DBG_H__
+#define __WAVE6_VPU_DBG_H__
+
+int wave6_vpu_create_dbgfs_file(struct vpu_instance *inst);
+void wave6_vpu_remove_dbgfs_file(struct vpu_instance *inst);

You add this patch 6. but call this funcation at patch 4.

You need make sure each patch can build without break bisect.

You are right that patch 4 calls what patch 6 declares.

But the build does not break: wave6/Kconfig and wave6/Makefile are added in
patch 8, so nothing under wave6/ is compiled in patches 3-7. I built every
commit individually.

Still not good yet, you can add call wave6_vpu_create_dbgfs_file() at
patch 6, instead of patch4.

You need make sure your code is built and passed at every patch. You
can add library/makefile firstly, then enable it gradulately. But need make
sure pass build test, some kernel CI tools build each patch and easy to
locate the problem, if build failure at last patch, which cause some
difficult to locate the real patch, which cause problem.

Frank

Thanks.
Nas.

quoted

Frank

quoted

+
+#endif /* __WAVE6_VPU_DBG_H__ */
--
2.31.1