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

[Nas Chung]

Hi, Frank.

>-----Original Message-----
>From: Frank Li <Frank.li@xxxxxxxxxxx>
>Sent: Thursday, September 10, 2026 5:41 AM
>To: Nas Chung <nas.chung@xxxxxxxxxxxxxxx>
>Cc: mchehab@xxxxxxxxxx; hverkuil@xxxxxxxxx; robh@xxxxxxxxxx;
>krzk+dt@xxxxxxxxxx; conor+dt@xxxxxxxxxx; shawnguo@xxxxxxxxxx;
>s.hauer@xxxxxxxxxxxxxx; linux-media@xxxxxxxxxxxxxxx;
>devicetree@xxxxxxxxxxxxxxx; linux-kernel@xxxxxxxxxxxxxxx; linux-imx@xxxxxxx;
>linux-arm-kernel@xxxxxxxxxxxxxxxxxxx; jackson.lee
><jackson.lee@xxxxxxxxxxxxxxx>; lafley.kim <lafley.kim@xxxxxxxxxxxxxxx>;
>marek.vasut@xxxxxxxxxxx; Ming Qian <ming.qian@xxxxxxxxxxx>
>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.

>
>> Add debugfs entries and trace events to provide detailed
>> debugging information.
>
>Nit: need extra space line here.

OK.

>
>> These enhancements help diagnose issues and improve debugging
>
>Remove "These enhancements".

I'll remove it.

...

>
>> +
>> +#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.

Thanks.
Nas.

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