From: Charles Keepax <ckeepax@opensource.cirrus.com>
To: Kuninori Morimoto <kuninori.morimoto.gx@renesas.com>
Cc: vinod.koul@intel.com, alsa-devel@alsa-project.org,
broonie@kernel.org, lgirdwood@gmail.com,
dan.carpenter@oracle.com
Subject: Re: [PATCH 1/2] ASoC: compress: Correct handling of compressed ops
Date: Fri, 26 Jan 2018 10:03:51 +0000 [thread overview]
Message-ID: <20180126100351.bygmlfndmwiqfsvp@localhost.localdomain> (raw)
In-Reply-To: <87mv114fz4.wl%kuninori.morimoto.gx@renesas.com>
On Fri, Jan 26, 2018 at 12:33:53AM +0000, Kuninori Morimoto wrote:
> In existing drivers, the component which have compr_ops *was* originally
> "platform" component only.
> The main purpose why we want to replace platform (and codec) to component is that
> CPU/Codec/Platform categorize is no longer best match for modern device.
>
> Thus, if we can replace all CPU/Codec/Platform into component, every device
> can have every feature, with no categorize !
> In other words, we don't know how many device have it.
>
Yeah I think there is quite a lot more work to get to that point
though, we need to understand how we will present this to
user-space.
> But anyway, we don't have such situation, and we don't know how to adjust to it
> at this point.
> Thus Yes !! I can 100% agree your opinion.
> Thank you for your help
>
> One note is that "rtd->platform" itself will be removed if all platforms were replaced.
> Then, soc_compr_get_component() / soc_compr_get_ops() might be merged or adjusted.
>
Yes that was my thinking as well, once you remove that you should
only need some minor refactoring to make the updates.
Thanks,
Charles
next prev parent reply other threads:[~2018-01-26 10:03 UTC|newest]
Thread overview: 16+ messages / expand[flat|nested] mbox.gz Atom feed top
2018-01-24 13:55 [PATCH 1/2] ASoC: compress: Correct handling of compressed ops Charles Keepax
2018-01-24 13:55 ` [PATCH 2/2] ASoC: compress: Fixup some minor style issues Charles Keepax
2018-01-24 14:33 ` Ladislav Michl
2018-01-24 14:50 ` Charles Keepax
2018-01-24 14:29 ` [PATCH 1/2] ASoC: compress: Correct handling of compressed ops Vinod Koul
2018-01-24 14:49 ` Charles Keepax
2018-01-24 15:33 ` Mark Brown
2018-01-24 16:00 ` Charles Keepax
2018-01-24 16:05 ` Mark Brown
2018-01-24 17:14 ` Charles Keepax
2018-01-25 11:03 ` Charles Keepax
2018-01-25 1:18 ` Kuninori Morimoto
2018-01-25 9:47 ` Charles Keepax
2018-01-26 0:33 ` Kuninori Morimoto
2018-01-26 10:03 ` Charles Keepax [this message]
2018-01-26 11:48 ` Mark Brown
Reply instructions:
You may reply publicly to this message via plain-text email
using any one of the following methods:
* Save the following mbox file, import it into your mail client,
and reply-to-all from there: mbox
Avoid top-posting and favor interleaved quoting:
https://en.wikipedia.org/wiki/Posting_style#Interleaved_style
* Reply using the --to, --cc, and --in-reply-to
switches of git-send-email(1):
git send-email \
--in-reply-to=20180126100351.bygmlfndmwiqfsvp@localhost.localdomain \
--to=ckeepax@opensource.cirrus.com \
--cc=alsa-devel@alsa-project.org \
--cc=broonie@kernel.org \
--cc=dan.carpenter@oracle.com \
--cc=kuninori.morimoto.gx@renesas.com \
--cc=lgirdwood@gmail.com \
--cc=vinod.koul@intel.com \
/path/to/YOUR_REPLY
https://kernel.org/pub/software/scm/git/docs/git-send-email.html
* If your mail client supports setting the In-Reply-To header
via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line
before the message body.
This is an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.