Re: [RFC PATCH] soc: audio-graph-card2: use correct endpoint when getting link parameters

[Ivaylo Dimitrov]

Hi Morimoto-san,

On 15.01.25 г. 1:49 ч., Kuninori Morimoto wrote:
> Hi Ivaylo
>
> Thank you for clarify your situation.
>
> > > You want to parse "remote" endpoint (= rep) directly, but the function
> > > requests "port" (= rport), and it will use endpoint0 ( != rep).
> > > Is this the main issue you want to fix ?
> >
> > Yes, it is the 'remote' side endpoint, currently it is always remote
> > endpoint0 that is used, because when you get 'port', it is endpoint0 of
> > that port that core uses.
>
> OK, I could understand, and I can agree to your idea.
> Getting "port" from "endpoint" is always stable, but getting "endpoint"
> from "port" without parameter will be issue, indeed.
>
> But I guess your original patch is based on very old kernel ?
> It can't be applied to Mark's for-6.14 branch as-is.
> Please based on latest branch.

Yes, it is based in 6.6, that's why I sent RFC patch, as rebasing will 
not be trivial and I didn't want to spend time on something that could 
possibly be rejected.

> And about git-comment,
>
> 	When link parameters are parsed, it is always endpoint@0 that is used and
> 	parameters set to other endpoints are ignored.
>
> Please indicate that current function requests "port" as parameter,
> thus, it always selects endpoint0, etc. That is easy to understand.

Ok, will do.

Thanks!
Ivo