Re: [PATCH V3 11/11] accel/amdxdna: Add firmware debug buffer support

[Lizhi Hou]

On 10/9/24 09:22, Lizhi Hou wrote:
> On 10/4/24 11:33, Jeffrey Hugo wrote:
> > On 9/11/2024 12:06 PM, Lizhi Hou wrote:
> > > User application may allocate a debug buffer and attach it to an NPU
> > > context through the driver. Then the NPU firmware prints its debug
> > > information to this buffer for debugging.
> >
> > I feel like I must be missing something. It looks like this patch 
> > accepts a buffer from the user, and stores it. However I don't see 
> > how the NPU firmware ever learns that this special buffer exists to 
> > then use it.
>
> The debug function is incomplete while I was creating the patch. Thus, 
> I put a line in TODO "Improve debug bo support".
>
> We made progress on this feature. Should I add the code to V4 patch? 
> Or I can add it within future patches.

Discussed this more internally. And we would postpone debug_bo feature. 
Thus, I will  drop this patch for now.


Thanks,

Lizhi

> Thanks,
>
> Lizhi