Subject: Re: [PATCH v3 1/3] ALSA: hda/tegra: Fix Tegra194 HDA reset failure
To:     Greg KH <gregkh@linuxfoundation.org>
Cc:     Dmitry Osipenko <digetx@gmail.com>, tiwai@suse.com,
        broonie@kernel.org, lgirdwood@gmail.com, robh+dt@kernel.org,
        thierry.reding@gmail.com, perex@perex.cz, jonathanh@nvidia.com,
        mkumard@nvidia.com, alsa-devel@alsa-project.org,
        devicetree@vger.kernel.org, linux-tegra@vger.kernel.org,
        linux-kernel@vger.kernel.org, stable@vger.kernel.org
References: <1640147751-4777-1-git-send-email-spujar@nvidia.com>
 <1640147751-4777-2-git-send-email-spujar@nvidia.com>
 <fb8cf33f-41fb-79c0-3134-524c290e4fc1@gmail.com>
 <f734e48f-dd60-ddb8-510a-3c4f37d8fb52@nvidia.com>
 <YcQiP+MxrlLi+R94@kroah.com>
From:   Sameer Pujar <spujar@nvidia.com>
Message-ID: <84e3ab9d-b351-b4b6-aa34-174725166913@nvidia.com>
Date:   Thu, 23 Dec 2021 16:54:09 +0530
User-Agent: Mozilla/5.0 (Windows NT 10.0; Win64; x64; rv:78.0) Gecko/20100101
 Thunderbird/78.12.0
In-Reply-To: <YcQiP+MxrlLi+R94@kroah.com>
Content-Type: text/plain; charset=utf-8; format=flowed
Content-Transfer-Encoding: 8bit
Content-Language: en-GB
MIME-Version: 1.0
X-MS-Exchange-AntiSpam-MessageData-ChunkCount: 1
X-MS-Exchange-AntiSpam-MessageData-0: =?utf-8?B?SldwM0UyZExwOVo2WVgxdW5JQnRrV1JQK3ZtRzhTOGl6QVNobk9xZFhkZm9E?=
 =?utf-8?B?RlBkcTVVd2gwbW5lSkhocGlBVzdEQ2FxRjU4dTYzRTlOOGJ2UDlUekdGMzd2?=
 =?utf-8?B?NDRMSzRuUStveTd6a00vcEhLbkNOLzdBcnpWdnVEQzdJQnZUNkkzclVJQkMx?=
 =?utf-8?B?MERSMkhiNmIwZk9od3N4bGZjWEw5cUJlUnBTQWUveHZtUCtNcGwxem5oelUx?=
 =?utf-8?B?YnVDZUYzby9tbzhTbXI1OFBmK1ZBSm1CMVQ3UXMvMTVMRlQ2ZTZvRTVqcHVH?=
 =?utf-8?B?aVgxM29LM3EzSnQ4Y3FGd1VBQUppTE9SbzYyUElpRHU0LzZrQWtkSVJPOTZO?=
 =?utf-8?B?cjl0QVFnc3JhQTNIQlV0V3FicGZEZGQ4TWZuajREelRQaEFUSWZiWnZMM0sy?=
 =?utf-8?B?eGdqaGUyM0t2QVpmOWJRRm1HZko0enZQWTFSQS9MRk1ETnc4dVh0cnROUklX?=
 =?utf-8?B?czlXVlZydHE4SmZhZ0w0ZkhuYXJXNHJwSU5RcFZ1VC8xUDNaZUJscnMxanJo?=
 =?utf-8?B?Q05FeGM1OUdRMjRUazZ0cHk0QlFBa3lxZlhYbnF4VDZVbHpEV2Z1akd2VzZk?=
 =?utf-8?B?MEdxM01TaExoMFoyRWNFbk1lSnd4SlB4N2ZFQ0lqa25EYjBoelhyZVJrV2Zj?=
 =?utf-8?B?NnU5R2hOdGVFZi9qeW42UmR3MzFCcFJwaVFCZGFhTjJlM1JWdVpSRVorejE2?=
 =?utf-8?B?SGtSNTFkdWRJYnVXNTBaRmJoWjRYQ01zNEJUdXlwNjJuanZyMlpKQ2hZd1pH?=
 =?utf-8?B?aGJ5aWxNN0J5TW0vVjRXVzBwVm8vUG83ckhjRTVFMld1SHM2bXQ5THdXUHdn?=
 =?utf-8?B?ZGJJK2trSWl6U3NVWTFydjFhdGNva1I3L1pIdkJPU2V1bWRSSlA4d0gvdmt5?=
 =?utf-8?B?dndyQndKTTZsY085dDdpaEVKVkZIVlNTeWhXMlVqYVpMMlJreWxISGJ3R1hy?=
 =?utf-8?B?VVNpanJPVjBLTTF3a2RScWJWWG9qdkFJSEJYTTRSSVBJUEZWaFNmeTMvOXlv?=
 =?utf-8?B?dEc5dXg3TTJUcDRuK1hSeUhONTRNVkxzNG1YNVBvZDJUMUdXVTdjRWE1WllL?=
 =?utf-8?B?TkR2eTQwdDZhTGlYa3Izb1dYNElvYW5KQmh4K2JxcVNNN2FzdmE3TGpWSGNG?=
 =?utf-8?B?Y25UamY2YWdRaUlTMGpGampEUVUrczVndnNuNzBKenZ2NlhKZ0NoWVJxbU5o?=
 =?utf-8?B?QlMzTHoxMHNiQVM5S2s2ODliMVhZalNjNHJlaDFadm5aQll6cXdya1l6dkFU?=
 =?utf-8?B?dU5pZ2dBS2ZyOGJsYUFIdzlIRWhEN1Ewa2trT3JsL2Zwblo5TjRkSVVYWEtB?=
 =?utf-8?B?RXVSU21YaFRqQ25rOTJFZk13ZnF6bHVVRE42ZUovNk5tSGZtb0tOYm9JT3NX?=
 =?utf-8?B?dmtZemlGMTgwanc1ZVJESHRvdDhURGtkQkZ3NnVtS29pRjZqaFlUcFJzWXJ6?=
 =?utf-8?B?K2c0Ky9EL1YxTitCNFl4TlZWTUhaNkc3L0N2ZUNnVlpzZWVJWENTejgzQThh?=
 =?utf-8?B?OG0zd3h4QU1RbDlTUHo4d1k3bGk1ZkhVTk1DRjE3VFR4TmdWeVFSU3dlRUtp?=
 =?utf-8?B?OXFMVTAydzhWZXJjUUtBTm1DYzAvdnM2VUg2eEo5b2xVMnFLUmY2VDRWMU93?=
 =?utf-8?B?dFJXY2x0SkYrSFE0VlA3VklXZmVzcTZzWlVGNVZDVTcvOFRObWJneXB4dGFQ?=
 =?utf-8?B?SlNGV0svaGc3byt4cUt0dll4dzQ3M01vQ0JtK0QrOTJ3Z2d6NUI2NFE1ZGFD?=
 =?utf-8?B?NUpjMEVwYm1Eelh5SkVaLy9uTDU5QTd0cUVoRm1NTVlnRkVwU0Nlbm5nbTdW?=
 =?utf-8?B?RUxJN3MxbW9QaGRaRnFXZDRlZkNEMU5zeEJjVDRjVEtLNFc5K1M0QmtXSU9i?=
 =?utf-8?B?bXkwUGJEVVBCclJGeWJGcjI0NUJGdGgzaWVHYUpSYVFZdytHL1NXSGdmUTM3?=
 =?utf-8?B?eEV1dGppeGdja1A4WTNnMExFbVZTb3dqRzBscmVvUmEzS2ZqbzZ0R0ZKaFFr?=
 =?utf-8?B?TVJUclVCQ0FnZnFyUWZObHFQVkZLZURRMGNzMnJybVVlcHRxSkVtR1BieUlJ?=
 =?utf-8?B?UlhEaFZ6c0ZoMFNsMksyeDZ2QXMyNHpoTjNqYlBxY01Ma1F5VEs5YVlZUkt6?=
 =?utf-8?B?bVFqQjNzaDAyOVhDOW8xUEJsUFUrbjFESzBZQmpFbUVaQmdBTzVyQUZ4cU5I?=
 =?utf-8?Q?RyiLDmVIes0TaD3t2AcyJ4s=3D?=
X-OriginatorOrg: Nvidia.com
X-MS-Exchange-CrossTenant-Network-Message-Id: 2576212c-9b60-4a2a-5d34-08d9c606c517
X-MS-Exchange-CrossTenant-AuthSource: CY4PR12MB1576.namprd12.prod.outlook.com
X-MS-Exchange-CrossTenant-AuthAs: Internal
X-MS-Exchange-CrossTenant-OriginalArrivalTime: 23 Dec 2021 11:24:23.0351
 (UTC)
X-MS-Exchange-CrossTenant-FromEntityHeader: Hosted
X-MS-Exchange-CrossTenant-Id: 43083d15-7273-40c1-b7db-39efd9ccc17a
X-MS-Exchange-CrossTenant-MailboxType: HOSTED
X-MS-Exchange-CrossTenant-UserPrincipalName: wIHHtsJAozExGr4qvJFX+h5JYQGhl8YCFtBcFAmldBIPcHNIHw24vCYyoEY0aarvgo8GnWSw1+KbD6xXLm8EZA==
X-MS-Exchange-Transport-CrossTenantHeadersStamped: CY4PR12MB1751
Precedence: bulk
List-ID: <linux-kernel.vger.kernel.org>
X-Mailing-List: linux-kernel@vger.kernel.org


On 12/23/2021 12:46 PM, Greg KH wrote:
> On Thu, Dec 23, 2021 at 10:04:19AM +0530, Sameer Pujar wrote:
>>
>> On 12/23/2021 12:10 AM, Dmitry Osipenko wrote:
>>> 22.12.2021 07:35, Sameer Pujar пишет:
>>>> HDA regression is recently reported on Tegra194 based platforms.
>>>> This happens because "hda2codec_2x" reset does not really exist
>>>> in Tegra194 and it causes probe failure. All the HDA based audio
>>>> tests fail at the moment. This underlying issue is exposed by
>>>> commit c045ceb5a145 ("reset: tegra-bpmp: Handle errors in BPMP
>>>> response") which now checks return code of BPMP command response.
>>>> Fix this issue by skipping unavailable reset on Tegra194.
>>>>
>>>> Signed-off-by: Sameer Pujar <spujar@nvidia.com>
>>>> Cc: stable@vger.kernel.org
>>>> Depends-on: 87f0e46e7559 ("ALSA: hda/tegra: Reset hardware")
>>> Is "Depends-on" a valid tag? I can't find it in Documentation/.
>> I do find the usage of the tag in many commits though there is no reference
>> of this in doc. I always thought it would act as a reference when commits
>> get pulled to other branches. If this is not true and it does not mean
>> anything, I will drop this.
> It is not true at all, please read:
>      https://www.kernel.org/doc/html/latest/process/stable-kernel-rules.html
> for how to do this properly.

Thanks Greg for the pointer. I will drop above tag in v4.