2020-07-30 08:20:49

by Brent Lu

[permalink] [raw]
Subject: [PATCH v2 0/2] Add period size constraint for Atom Chromebook

Two different constraints are implemented: one is in platform's CPU
DAI to enforce period sizes which are already used in Android BSP. The
other is in Atom Chromebook's machine driver to use 240 as period size.

Changes since v1:
-Add comma at the end of media_period_size array declaration.

Brent Lu (1):
ASoC: intel: atom: Add period size constraint

Yu-Hsuan Hsu (1):
ASoC: Intel: Add period size constraint on strago board

sound/soc/intel/atom/sst-mfld-platform-pcm.c | 15 +++++++++++++++
sound/soc/intel/boards/cht_bsw_max98090_ti.c | 14 +++++++++++++-
sound/soc/intel/boards/cht_bsw_rt5645.c | 14 +++++++++++++-
3 files changed, 41 insertions(+), 2 deletions(-)

--
2.7.4


2020-07-30 08:42:25

by Andy Shevchenko

[permalink] [raw]
Subject: Re: [PATCH v2 0/2] Add period size constraint for Atom Chromebook

On Thu, Jul 30, 2020 at 04:13:33PM +0800, Brent Lu wrote:
> Two different constraints are implemented: one is in platform's CPU
> DAI to enforce period sizes which are already used in Android BSP. The
> other is in Atom Chromebook's machine driver to use 240 as period size.
>
> Changes since v1:
> -Add comma at the end of media_period_size array declaration.

You missed my tags, any reason why?

--
With Best Regards,
Andy Shevchenko


2020-07-30 15:04:15

by Takashi Iwai

[permalink] [raw]
Subject: Re: [PATCH v2 0/2] Add period size constraint for Atom Chromebook

On Thu, 30 Jul 2020 10:13:33 +0200,
Brent Lu wrote:
>
> Two different constraints are implemented: one is in platform's CPU
> DAI to enforce period sizes which are already used in Android BSP. The
> other is in Atom Chromebook's machine driver to use 240 as period size.
>
> Changes since v1:
> -Add comma at the end of media_period_size array declaration.

Is it a hardware restriction? Unless it's a must for some hardware
issues, enforcing such a small period size is nothing but a drawback
for a system without CRAS.


thanks,

Takashi

>
> Brent Lu (1):
> ASoC: intel: atom: Add period size constraint
>
> Yu-Hsuan Hsu (1):
> ASoC: Intel: Add period size constraint on strago board
>
> sound/soc/intel/atom/sst-mfld-platform-pcm.c | 15 +++++++++++++++
> sound/soc/intel/boards/cht_bsw_max98090_ti.c | 14 +++++++++++++-
> sound/soc/intel/boards/cht_bsw_rt5645.c | 14 +++++++++++++-
> 3 files changed, 41 insertions(+), 2 deletions(-)
>
> --
> 2.7.4
>

2020-07-30 15:26:27

by Brent Lu

[permalink] [raw]
Subject: RE: [PATCH v2 0/2] Add period size constraint for Atom Chromebook

> >
> > Two different constraints are implemented: one is in platform's CPU
> > DAI to enforce period sizes which are already used in Android BSP. The
> > other is in Atom Chromebook's machine driver to use 240 as period size.
> >
> > Changes since v1:
> > -Add comma at the end of media_period_size array declaration.
>
> Is it a hardware restriction? Unless it's a must for some hardware issues,
> enforcing such a small period size is nothing but a drawback for a system
> without CRAS.

Hi Takashi,

This patch is cherry-picked from Chrome's branch which is originally implemented
by google. They found this value works best with CRAS and these two machine
drivers are for Chromebooks. Other BSW machine drivers are untouched.


Regards,
Brent

>
>
> thanks,
>
> Takashi
>
> >
> > Brent Lu (1):
> > ASoC: intel: atom: Add period size constraint
> >
> > Yu-Hsuan Hsu (1):
> > ASoC: Intel: Add period size constraint on strago board
> >
> > sound/soc/intel/atom/sst-mfld-platform-pcm.c | 15 +++++++++++++++
> > sound/soc/intel/boards/cht_bsw_max98090_ti.c | 14 +++++++++++++-
> > sound/soc/intel/boards/cht_bsw_rt5645.c | 14 +++++++++++++-
> > 3 files changed, 41 insertions(+), 2 deletions(-)
> >
> > --
> > 2.7.4
> >