12/03/2024 07:17, fengchengwen:
> Sorry for same reply,
> 
> @Vipin,
> the title should reflect which submodule, so pls add prefix: test/dma-perf:

No it should be app/dma-perf.
If not sure about a prefix, you may check the git history for the directory.

> Suggest: test/dma-perf: refine size parameter details in doc
> Just a suggestion, you could refine it.
> 
> On 2024/3/12 14:07, fengchengwen wrote:
> > Acked-by: Chengwen Feng <fengcheng...@huawei.com>

Please add your ack below the sign-off for clearer reply chain.

> > On 2024/3/12 12:22, Vipin Varghese wrote:
> >> For configuration parameters `mem_size` and `buf_size` are represented
> >> as megabytes and bytes respectively in application. Update the
> >> documentation and to represent the same.
> >>
> >> V2 Changes:
> >>  - add mem_size and buf_size in config.ini, suggested by Chengwen.
> >>
> >> Signed-off-by: Vipin Varghese <vipin.vargh...@amd.com>

Applied, thanks.


Reply via email to