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;
Subject: Re: [PATCH 0/4] Fix kcontrol put callback in Tegra drivers
To:     Jaroslav Kysela <perex@perex.cz>, broonie@kernel.org,
        lgirdwood@gmail.com, tiwai@suse.com
Cc:     thierry.reding@gmail.com, jonathanh@nvidia.com,
        alsa-devel@alsa-project.org, linux-tegra@vger.kernel.org,
        linux-kernel@vger.kernel.org
References: <1635915897-32313-1-git-send-email-spujar@nvidia.com>
 <8a00ea68-4859-fbcc-6292-909a66b0a188@perex.cz>
From:   Sameer Pujar <spujar@nvidia.com>
Message-ID: <1b63798e-cf3f-5ef8-1950-3111273ad82d@nvidia.com>
Date:   Wed, 3 Nov 2021 16:59:08 +0530
User-Agent: Mozilla/5.0 (Windows NT 10.0; Win64; x64; rv:78.0) Gecko/20100101
 Thunderbird/78.12.0
In-Reply-To: <8a00ea68-4859-fbcc-6292-909a66b0a188@perex.cz>
Content-Type: text/plain; charset=utf-8; format=flowed
Content-Transfer-Encoding: 7bit
Content-Language: en-GB
MIME-Version: 1.0
X-MS-Exchange-AntiSpam-MessageData-ChunkCount: 1
X-MS-Exchange-AntiSpam-MessageData-0: =?utf-8?B?ejhYKzIyYklTYnNvQ09GV25uUUxGTUREWWdvUloxNWVPK2cxS0JZbU1DSG4w?=
 =?utf-8?B?QUV6MnFRc3RCZUN3cEZlL2h4N3A5cHBPYjFwcTRTb0ZOdWUyUXcyQ2YyVnhM?=
 =?utf-8?B?Uy82WU83SVUyd1BFZys3eWI0NVFUV3BnWHlqNHRheHZMblYxSWZwYnNWS2NY?=
 =?utf-8?B?M09sd0Q1N090dG4yRUNnaTJPd3ZYMEtlSjNmY1pqT1grazNiUlMzT3VtTGZz?=
 =?utf-8?B?bDBkSitmNjk2TkhRWXpkRkdXMVpKM21sUE5zRE11Zk1nWVNCTG5FVlVGSWV2?=
 =?utf-8?B?VGNPOEJVa0RtZ0tmN0pKekpCRnlxV3dVRk44c0YwZUpZc2tKb1VRZkpsWFdx?=
 =?utf-8?B?ZG1JalByd3I4dm5talFka281YzcxOVI1bnFwdkNxSVBEcGxLcWxEVE1qZUFH?=
 =?utf-8?B?ZURIZy94VFNPYWxrNlduQk9kZUM1K3hqbFZNc0RoUDZQRWRXNllxYWYwV205?=
 =?utf-8?B?ZVZ1eXZIT2Q4K2ZsTTNLNkdLZlVQUXVGWGl2WDdhQURKSFZkS0NZVFZaQXBT?=
 =?utf-8?B?bVBQTWtFazg0ZG80L3lFRGducjhRTjZ1RXdWSXh1cE5IT3EwaHJ4STB0QVlq?=
 =?utf-8?B?RW1obkpYbnE3WldYb1Z4eksxS0t1Q1ZwVkNXSys4QTVTNmZjUU4wVHMwQzhi?=
 =?utf-8?B?R3JIUytPa25Hekh3SUZwaVFUampXMWNuY2s1Mk9zVENheFhFYjhnVDJnL1Jq?=
 =?utf-8?B?NmNndVVsa3NTYW1iVXB6cFIrVzJhT0tHZEwxWDMxVHVtallTYlRPaUk4N1JF?=
 =?utf-8?B?Z2pIUlY1dGhlc1FBRXBGZ2ttWkpEcUVSRHRObjFmcGNDRTFKNVhNdXlEQjRs?=
 =?utf-8?B?YlVucGgyNGNPd3I1NWZzNVlDZVMvbW5iSVdKdTB4am1MdnRsekJNZ3FwNUNH?=
 =?utf-8?B?VEduSFdxeUREMjl0bEhrWjNlMmlOYXZURnFVcG90a0lIWGw3elRzR2grUEd2?=
 =?utf-8?B?K3BYS2x0OHczNnRVNnZrNGtlWUlESGxibEVzZkRuUTVzbXlGRXl4RmJDR1hj?=
 =?utf-8?B?UHVHaGQ5Ry9EczhQOWZ1dzdtQU9GdWN3UmlyU2NXQkorSHRZTHBUMGIvM0VS?=
 =?utf-8?B?TW96bkc3dnFaRnZCVWgvS3JSWXIzSzMvWjJRSVYwSmN5dEJUZ0laV1BuMzh5?=
 =?utf-8?B?eEtOUmcyUytoTENQZ09kcWQwVm5hTXlvSGZwczBFUER1cmRScUtRc1ZHak9X?=
 =?utf-8?B?WTUrM3RNeTNBeHJpdUUvNkNhbXFNb1ZOM0FlYnplS0YzSVJLcTEvUGcwYzcw?=
 =?utf-8?B?MEN3ak52anBHcWN0NDdEQlphY2E0Q2tlbFJ3VDFiVFlPdFRNeXViRXNxWEI3?=
 =?utf-8?B?ZWl3ckV0alpaV3F4NE9ldFhqV2xrN0wxRWdqU2tIZmtaUDV5NlJlM0NHdFk0?=
 =?utf-8?B?cjdmcXJ3OW5GWWFuV0xBNE8wa3o4MnhGWjdnOVRsNml0UmRqZVFPUDYxRWtY?=
 =?utf-8?B?RS9CaW80Z2s3em8yQlN4cmRkTVQ5Vk5EaDNlSHJwdjNJdnVKa2c0RXhEREVI?=
 =?utf-8?B?VG5tazVISG5HN1dKMU1DM0RNbGZrd1RhRXdPM003OEZlLzh1TU9FNkoxU1FT?=
 =?utf-8?B?T2V6bVhIV2xiZjRkVHlOOWZFNmN2Y2kzMElhYlFkR2x1OUhUdzgyQi9tRStP?=
 =?utf-8?B?UjZXd21LTktkODdvcWRIRVgwS3RNeDRQaDdqVU0vUElXckZySjl5dFNEWDlM?=
 =?utf-8?B?ZXRuQmo3bUxUKzR2cFFVa1B5ZkoyUDVMeGNRR0F5ZWJjbEtxd1NqSjNocDc5?=
 =?utf-8?B?bTB2cDdRblM5QncrZmJKQk04UUFVZzNHMXlXVWZrSUt0c2dRc2IwS1pLT2NV?=
 =?utf-8?B?NXJPeGFPR2gvME5GbTZ3anlyVXI4Q0psbUZBcU5XMkc0Q3kwb2FjMkhjMTVu?=
 =?utf-8?B?MVprUHJMOGVvc1MwbEpUQ0VrSGwyMWNkMWJBWG94QVhUZC9tejVQUW13d0xp?=
 =?utf-8?B?dTZMLzJoOEtNaGpVZWUyL1BybDdwNjlQRTJqQ0ZSZXIxWGtQRmFodHdBSldU?=
 =?utf-8?B?UEQzenFQL2VzRnRra0dKc2cxcFlBVEczVkRSYlIxQmR2VDBGN1B6NE9jbkt2?=
 =?utf-8?B?VVZSM3dhTEpRYUswRGxkTG9xb01IeTlGd2JZZlR1NExTdjlMUERJenlPeGhr?=
 =?utf-8?B?ZFRUYTV1WnpCNll2VU5ZU0k2dmlmc2ZTU2hLVTBVT3hLTUQvaGtZeCtHNlJG?=
 =?utf-8?Q?pJnTyXhdCyx7OzsuE5PVirs=3D?=
X-OriginatorOrg: Nvidia.com
X-MS-Exchange-CrossTenant-Network-Message-Id: 3df35c3c-9744-44cc-aed4-08d99ebd2df2
X-MS-Exchange-CrossTenant-AuthSource: BL1PR12MB5317.namprd12.prod.outlook.com
X-MS-Exchange-CrossTenant-AuthAs: Internal
X-MS-Exchange-CrossTenant-OriginalArrivalTime: 03 Nov 2021 11:29:21.0246
 (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: MT0IeejwDvE02VYMOFKg90vN91SHQO3hL3vgawG+S9asZQlPPzncaZXs7DGsX7XVMlI5bgdDtynBGTjFdqb/YQ==
X-MS-Exchange-Transport-CrossTenantHeadersStamped: BL0PR12MB5507
Precedence: bulk
List-ID: <linux-kernel.vger.kernel.org>
X-Mailing-List: linux-kernel@vger.kernel.org


On 11/3/2021 2:20 PM, Jaroslav Kysela wrote:
> External email: Use caution opening links or attachments
>
>
> On 03. 11. 21 6:04, Sameer Pujar wrote:
>> This series fixes kcontrol put callback in some of the Tegra drivers
>> which are used on platforms based on Tegra210 and later. The callback
>> is expected to return 1 whenever the HW update is done.
>
> The logic is a little bit another. The 1 means change, so you should 
> compare
> the register / cached value and return 1 only when a different value 
> is set.
> Your implementation invokes the change events for duplicate updates.

Sorry, I did not think the about duplicate updates. Thanks for the 
suggestion.