RE: [RFC PATCH] ASoC: simple-card: Use dai_id from node description

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 



Hi Daniel, Mark

> > > -     args.args_count = (of_graph_get_endpoint_count(node) > 1);
> > > +     args.args_count = (of_graph_get_endpoint_count(node) >= 1);
> >
> > If my understanding was correct, for example you want to use 2nd DAI
> > but your DT has only 1 port (thus, it is using reg property) ?
> 
> Yes.

But hmm... in your case, you need to setup 2ports, and use 2nd port
is assumed approach.
Why you don't setup full port ? Do you have some reason ??


I think almost all DTS are already using "reg" property, thus,
there is no problem if we remove non-reg-support,
but we don't know details.

Removing non-reg-support needs to update many codes. 
But your original patch is enough for 1st approach, and it is easy
to rewind the code if there was some issue.
I can create more detail cleanup code if there was no problem.

But then, I want to know it is necessary.
If there is good enough reasons why you don't setup full-port,
we can/need to remove non-reg-support. But if there is no good reason,
the things we need is not update code but you add full-port setup, I think.





[Index of Archives]     [ALSA User]     [Linux Audio Users]     [Pulse Audio]     [Kernel Archive]     [Asterisk PBX]     [Photo Sharing]     [Linux Sound]     [Video 4 Linux]     [Gimp]     [Yosemite News]

  Powered by Linux