Received-SPF: pass (google.com: domain of linux-kernel-owner@vger.kernel.org designates 23.128.96.18 as permitted sender) client-ip=23.128.96.18;
IronPort-SDR: 5JEUEd8+EqksHve3TxbomDBI93iIsVfIpPnuTg8G9pmbzP/mnKY0RdMOslcTEcuAh0ggvX5LGg
 zbupLVxtSvRw==
IronPort-SDR: +nrXKwSZzV//j+aXOP97YPj85q1rBwY5FhpqG9bSEEfrOrwvPViTqHDfT576ZcXGUSJHYI2UP6
 d5FZJs7DAijg==
Subject: Re: [PATCH 0/3] add channel constraint for BDW machine drivers
From:   Cezary Rojewski <cezary.rojewski@intel.com>
To:     Brent Lu <brent.lu@intel.com>, alsa-devel@alsa-project.org
Cc:     Pierre-Louis Bossart <pierre-louis.bossart@linux.intel.com>,
        Liam Girdwood <liam.r.girdwood@linux.intel.com>,
        Jie Yang <yang.jie@linux.intel.com>,
        Mark Brown <broonie@kernel.org>,
        Jaroslav Kysela <perex@perex.cz>,
        Takashi Iwai <tiwai@suse.com>, Ben Zhang <benzh@chromium.org>,
        Mac Chiang <mac.chiang@intel.com>,
        Guennadi Liakhovetski <guennadi.liakhovetski@linux.intel.com>,
        Kuninori Morimoto <kuninori.morimoto.gx@renesas.com>,
        linux-kernel@vger.kernel.org
References: <1587976638-29806-1-git-send-email-brent.lu@intel.com>
 <1375d0b1-fafa-95b5-9a06-eefb1897ca42@intel.com>
Message-ID: <1bcd3310-34c3-7d90-cb18-f474d9e30c25@intel.com>
Date:   Mon, 27 Apr 2020 13:15:19 +0200
User-Agent: Mozilla/5.0 (Windows NT 10.0; WOW64; rv:68.0) Gecko/20100101
 Thunderbird/68.6.0
MIME-Version: 1.0
In-Reply-To: <1375d0b1-fafa-95b5-9a06-eefb1897ca42@intel.com>
Content-Type: text/plain; charset=utf-8; format=flowed
Content-Language: en-US
Content-Transfer-Encoding: 8bit
Sender: linux-kernel-owner@vger.kernel.org
Precedence: bulk

On 2020-04-27 13:01, Cezary Rojewski wrote:
> On 2020-04-27 10:37, Brent Lu wrote:
>> The machine driver bdw-rt5650 (for Google buddy) supports 2 or 4-channel
>> recording while other two drivers support only 2-channel recording. HW
>> constraints are implemented to reflect the hardware limitation on BDW
>> platform.
>>
> 
> Message body relates to bdw-rt5650 while the series touches every of BDW 
> boards.

Ignore my first sentence :-) Second still applies though

> 
> Apart from review given for each and every patch (although most issues 
> are shared so there is not as much to address) my question is:
> - are these hw limitations or software (machine board) limitations?
> 
> Czarek
> 
>> Brent Lu (3):
>>    ASoC: bdw-rt5677: channel constraint support
>>    ASoC: bdw-rt5650: channel constraint support
>>    ASoC: broadwell: channel constraint support
>>
>>   sound/soc/intel/boards/bdw-rt5650.c | 34 
>> ++++++++++++++++++++++++++++++++++
>>   sound/soc/intel/boards/bdw-rt5677.c | 33 
>> +++++++++++++++++++++++++++++++++
>>   sound/soc/intel/boards/broadwell.c  | 33 
>> +++++++++++++++++++++++++++++++++
>>   3 files changed, 100 insertions(+)
>>