Re: [alsa-devel] [RESEND 0/4] Add a better separation between i.MX8 families - Pierre-Louis Bossart

From: Pierre-Louis Bossart <pierre-louis.bossart@linux.intel.com>
To: Daniel Baluta <daniel.baluta@oss.nxp.com>,
	broonie@kernel.org, robh+dt@kernel.org
Cc: devicetree@vger.kernel.org, alsa-devel@alsa-project.org,
	festevam@gmail.com, linux-kernel@vger.kernel.org,
	linux-imx@nxp.com, Daniel Baluta <daniel.baluta@nxp.com>
Subject: Re: [alsa-devel] [RESEND 0/4] Add a better separation between i.MX8 families
Date: Mon, 10 Feb 2020 08:23:24 -0600	[thread overview]
Message-ID: <73c72736-ca09-1501-4978-e0041b605aed@linux.intel.com> (raw)
In-Reply-To: <20200210095817.13226-1-daniel.baluta@oss.nxp.com>


On 2/10/20 3:58 AM, Daniel Baluta wrote:
> From: Daniel Baluta <daniel.baluta@nxp.com>
> 
> So far the implementation was designed to support  a generic platform
> named i.MX8. Anyhow, now working with specific i.MX8 instances we need
> to account for the differences.
> 
> i.MX8 naming can be confusing at the first glance, so we need to have
> a clean separation between different platforms.
> 
> Here is the split of i.MX8 per platforms. Notice that i.MX8 names
> the entire family, but also a sub-family.
> 
> imx8
> ├── imx8
> │   └── imx8qm (*)
> ├── imx8m
> │   ├── imx8mm
> │   ├── imx8mn
> │   ├── imx8mp (*)
> │   └── imx8mq
> └── imx8x
>      └── imx8qxp (*)
> 
> Platforms marked with (*) contain a DSP. In the future there might be
> more platforms.
> 
> This patchseries does the following:
> 	* renames imx8 to imx8x (because the only supported platform now
>          is imx8qxp).
>          * adds support for imx8 (which is imx8qm)
> 
> Paul Olaru (4):
>    ASoC: SOF: Rename i.MX8 platform to i.MX8X
>    ASoC: SOF: imx8: Add ops for i.MX8QM
>    ASoC: SOF: Add i.MX8QM device descriptor
>    dt-bindings: dsp: fsl: Add fsl,imx8qm-dsp entry

Reviewed-by: Pierre-Louis Bossart <pierre-louis.bossart@linux.intel.com>

> 
>   .../devicetree/bindings/dsp/fsl,dsp.yaml      |  1 +
>   sound/soc/sof/imx/imx8.c                      | 57 ++++++++++++++++++-
>   sound/soc/sof/sof-of-dev.c                    | 10 ++++
>   3 files changed, 65 insertions(+), 3 deletions(-)
>