Received-SPF: pass (google.com: domain of linux-kernel-owner@vger.kernel.org designates 2620:137:e000::1:20 as permitted sender) client-ip=2620:137:e000::1:20;
Message-ID: <c3386d45-b643-b4aa-c868-5c113cd2955f@intel.com>
Date:   Wed, 17 Aug 2022 15:19:50 +0200
User-Agent: Mozilla/5.0 (Windows NT 10.0; Win64; x64; rv:91.0) Gecko/20100101
 Firefox/91.0 Thunderbird/91.12.0
Subject: Re: [PATCH 2/2] ASoC: Intel: Skylake: try to get NHLT blob with PCM
 params as fallback
Content-Language: en-US
To:     Icenowy Zheng <uwu@icenowy.me>
CC:     <alsa-devel@alsa-project.org>, <linux-kernel@vger.kernel.org>,
        Pierre-Louis Bossart <pierre-louis.bossart@linux.intel.com>,
        Liam Girdwood <liam.r.girdwood@linux.intel.com>,
        Peter Ujfalusi <peter.ujfalusi@linux.intel.com>,
        Bard Liao <yung-chuan.liao@linux.intel.com>,
        Ranjani Sridharan <ranjani.sridharan@linux.intel.com>,
        Kai Vehmanen <kai.vehmanen@linux.intel.com>,
        Mark Brown <broonie@kernel.org>,
        "Jaroslav Kysela" <perex@perex.cz>, Takashi Iwai <tiwai@suse.com>
References: <20220725111002.143765-1-uwu@icenowy.me>
 <20220725111002.143765-2-uwu@icenowy.me>
 <4208aece-0aea-ba86-9a00-9e217546e1dd@intel.com>
 <3346b62c3cd7e07766457c9140849a31d6d8775c.camel@icenowy.me>
 <85648922-27df-051d-8ae8-d0e4e810198a@intel.com>
 <1891bc72834cb6e25d479c3968c04bcef94c3ad5.camel@icenowy.me>
From:   Cezary Rojewski <cezary.rojewski@intel.com>
In-Reply-To: <1891bc72834cb6e25d479c3968c04bcef94c3ad5.camel@icenowy.me>
Content-Type: text/plain; charset="UTF-8"; format=flowed
Content-Transfer-Encoding: 8bit
MIME-Version: 1.0
X-MS-Exchange-AntiSpam-MessageData-ChunkCount: 1
X-MS-Exchange-AntiSpam-MessageData-0: =?utf-8?B?ay9aZCtOUFk0UHJSR3FEeHdOcjY0QStxQzFubzRCZTVEcllvV2NHM2R6UmlQ?=
 =?utf-8?B?bnMzZjF1dlp0eDdnR0k1UDJBd08vc2NadVNRM3NZTU52M0Fld2IzVzRBcS9v?=
 =?utf-8?B?Q01HM0hqdldUOUJ3SDRROXB5MzhSTkRjMndNYzN5VFJ1Tm5GeDNydnBJczQy?=
 =?utf-8?B?T3NVSTFuUEZaZGFxWXBTN21yd3dtYnZJMHdmSFlIaG0rVm02ekpIQk54WWRx?=
 =?utf-8?B?cERvNDl5VnphZWlVV1RZUFpoSkdJVEFXUjQyNk5rcDgvUjR3aE16WEhRRlpW?=
 =?utf-8?B?c0FZclBrSmJadUdRd1NmcTVSbW1tZmlpVkgrZHVnNENyUG8vV25LdmVQN1Jt?=
 =?utf-8?B?cUxEUER0dThKMis4Q0VobWI0VmlaRDYyYXM4cjQzK0JVMDN2cmhtTGtybUdE?=
 =?utf-8?B?WDdmYWpmNUk5V0ZkcXVkVGZHSFNBczNmZ0FiYVFTeUsyMHo4T2p2ZlNUcEhT?=
 =?utf-8?B?aWhmdE1nMmNSVUY3ZXdSYTRCMy8zVVl5aWJaeXZDakxGUk43bHMyK1c1bVVu?=
 =?utf-8?B?TlhnTTZpa0NJbUQwRjNQSmlxSUF5YXdlSXJFWFR0T1paVW16Q0FSRk5EZHFw?=
 =?utf-8?B?U1ArWUIvMzFWS1lBdFRvcFhEbUdadjgxTC8vMEpIWmtTSElOZGcxU3lTbGk3?=
 =?utf-8?B?Y3ltNnJFYTZLL25md3BmMGFLaXZDWUZPaTRqOFo4VXpmVUJKSHdEMDdkZnBI?=
 =?utf-8?B?c2xzVG9tZUxoWWpBWjV6VTBCYVJnMnQxelVWTkY4Z3dqU3E5L2dFT3hlVllO?=
 =?utf-8?B?ME9VTUtRRDNzb1BicHAvdHhmS0xmSkV2NVhhdDVqYUxhMFd6SHJJTnZ3dTE0?=
 =?utf-8?B?RTNwRVY3eHNOMWJXN2U3bHNKOUtzWXk5bzJXcWtyeXYxOUZUUG0rcTlPQ1c4?=
 =?utf-8?B?UHh1VkJIWkMxVzhhcVNZNU4xNXBQUWdNaVZjaFZUcEhCd29ONjg4OGVVTmlU?=
 =?utf-8?B?Y0E3elZRSFhqTHlCUFlyZUFOM2h1SVEwaGZDbE1aMmllQVlVUXlwSmxsa3BS?=
 =?utf-8?B?eVpGd2Rrc3lyWFJsWm9Id1RaR3BwektRc3NJaXVTR1NLdHYydDd0dmcxUWdD?=
 =?utf-8?B?dXBZaTE2aW9KdURMeTZXUVJLL0pCMzR4YkZKaUxmUnBtZC8wa0s1VlF6dUdG?=
 =?utf-8?B?Q0VVQSsrdVJMYzQ5QldZNFdGcEJaU1ZUVTJ4T2dZREgxVnc5NjNyaG5CQWZ3?=
 =?utf-8?B?UFlRWHBPNVJIbTBiSHkzTUxYNGpvZGswZ2tCemlPTDZvY3JzeWhRVUsycysw?=
 =?utf-8?B?NVdUWmxXUGVLMXc0RS8wOC9sWXdBdk1sTnBGMW5ObmtJZU5uOS9leDZSUVZF?=
 =?utf-8?B?NlMrVlk1TFdZc0Vmd1VIRHRlQUZyQ0tRaDFVa3paMUtrbHdETXhmaUxKTkJH?=
 =?utf-8?B?by91NDdQRVQ3SCtpVS8vbDNSR0REaWV6RWxseWdsK3ZKMDlLWXNnbGI5L2Mx?=
 =?utf-8?B?V2R3TmNHZGNpbVRzb2R5Z09MU1h5ZlY0U016T2I0RHE0UzNlOW5hbWhDZDdG?=
 =?utf-8?B?dUQvblRCbVNQY1BKNTYzbHN1MmpGeUhDYy9ZZVhKb2VjODFRaUpSbFVwRkdj?=
 =?utf-8?B?WkdlalJpUlhsTlhpUXBLWHZON0lJWU5TSzhQK2gwc1FNQTBoamRyb0YyTENW?=
 =?utf-8?B?ekZ4bXRUT0NURVJ2cVJhWHJoMEs4Q0pzNnE4OFJGV1hFQXZXRFZsdVhJMDlq?=
 =?utf-8?B?Y2lDSHo5SU5JNEYzblpPbmRCUlRHMThpZXNsbHFhWHFxNTBqVlBON1RtN0Uy?=
 =?utf-8?B?eHVvVDlvcEIxZjNWVncydmREakZ6YS82eUFLWFdmSnhacU1ZekYvSmJBckVw?=
 =?utf-8?B?dk12RFZjZEUySG93eW9DSXhpbWFaeU5ta3Roc3I4R1A3U1JabW92ZjBZS1hT?=
 =?utf-8?B?aG9iK2M3dzlENmlXT2s4VlBjZ3l0ZVBmSURXK0xoWkErY1ZGWmJZc2hHYmo3?=
 =?utf-8?B?TlpKNUtDZC93SFNNRmtqWFZJNHFwRlFGYjhMU2w1VW5NOWgvT3k5aTZYQWhV?=
 =?utf-8?B?b0c1WUthZ0NManRrWWd5OEQ1eDlORnQ1YWNNNlByRlJ1TDFqVjNLei9WME80?=
 =?utf-8?B?MmNRRGM0SE53WWs3ck5OT203Um9rSDhnc25zR1RXcEgrK2UydEh3bWN5Z1k4?=
 =?utf-8?B?L2JscjlmRnVZcU9ONE9TMGhwM0FYaVhPT1c4QWI0bS9hRlNNQXJpT2E5bEFF?=
 =?utf-8?B?Nnc9PQ==?=
X-MS-Exchange-CrossTenant-Network-Message-Id: fdc2b463-6ef0-4602-46c2-08da80532e82
X-MS-Exchange-CrossTenant-AuthSource: DS0PR11MB6375.namprd11.prod.outlook.com
X-MS-Exchange-CrossTenant-AuthAs: Internal
X-MS-Exchange-CrossTenant-OriginalArrivalTime: 17 Aug 2022 13:19:57.8370
 (UTC)
X-MS-Exchange-CrossTenant-FromEntityHeader: Hosted
X-MS-Exchange-CrossTenant-Id: 46c98d88-e344-4ed4-8496-4ed7712e255d
X-MS-Exchange-CrossTenant-MailboxType: HOSTED
X-MS-Exchange-CrossTenant-UserPrincipalName: WfRWmjD/5Rd8SdgMkR/Ju9ZWKU1m4FHYOTS7keFSGIjjn21Cu+CESM6BXdZZIdIDtvDHKpHt8KGaRC11U+uqgSHLrHjf7thxIZL6x+/uHns=
X-MS-Exchange-Transport-CrossTenantHeadersStamped: CO6PR11MB5668
X-OriginatorOrg: intel.com
X-Spam-Status: No, score=-4.4 required=5.0 tests=BAYES_00,DKIMWL_WL_HIGH,
        DKIM_SIGNED,DKIM_VALID,DKIM_VALID_AU,DKIM_VALID_EF,NICE_REPLY_A,
        RCVD_IN_DNSWL_MED,SPF_HELO_PASS,SPF_NONE,T_SCC_BODY_TEXT_LINE,
        URIBL_BLOCKED autolearn=ham autolearn_force=no version=3.4.6
X-Spam-Checker-Version: SpamAssassin 3.4.6 (2021-04-09) on
        lindbergh.monkeyblade.net
Precedence: bulk
List-ID: <linux-kernel.vger.kernel.org>
X-Mailing-List: linux-kernel@vger.kernel.org

On 2022-08-17 2:48 PM, Icenowy Zheng wrote:
> 在 2022-08-16星期二的 21:08 +0200，Cezary Rojewski写道：

...

> 
> Yes, it's Chell.
> 
> BTW do you need other ACPI tables?

At this point, no. Thanks for confirming the platform's name.

>>
>> ..and at this point I probably know more than enough. We have tested
>> basically all of the KBL and AML configurations when fixing
>> regressions
>> during recent skylake-driver up-revs. But Chell (and Lars for that
>> matter) families were not among them as these are based on SKL. I'll
>> follow up on this with our partners and come back here. I'm almost
>> certain topology files for the two families mentioned were not
>> updated
>> along the way.
> 
> Could this be an issue of Coreboot, which generates the NHLT table?

NHLT was left alone across all the updates. Updating it is one way of 
fixing problems but I don't believe it's necessary in your case. 
Topology update is more desirable approach.

> BTW I think Google pinned the official OS of this hardware to a much
> lower kernel version (but I don't want to use the official OS because
> of limited storage of Chell and lack of VM ability of the OS on Chell).

That clarifies things out. Guess the kernel version used there is v4.4 
(plus a ton of un-upstreamed patches). Again, will propagate the 
information up the chain. Perhaps one of the solutions for end-users 
would be providing working UCM files to alsa-topology-conf repo so users 
are not powerless in situations such as this one.


Regards,
Czarek