Received-SPF: pass (google.com: domain of linux-kernel+bounces-122882-linux.lists.archive=gmail.com@vger.kernel.org designates 2604:1380:40f1:3f00::1 as permitted sender) client-ip=2604:1380:40f1:3f00::1;
Message-ID: <c503a3ce-cb7a-4211-a15c-9a4ece8d1b6f@amd.com>
Date: Thu, 28 Mar 2024 17:53:57 +0530
User-Agent: Mozilla Thunderbird
Subject: Re: [PATCH 0/7] Soundwire: clean up sysfs group creation
Content-Language: en-US
To: Vinod Koul <vkoul@kernel.org>,
 Greg Kroah-Hartman <gregkh@linuxfoundation.org>
Cc: alsa-devel@alsa-project.org, linux-kernel@vger.kernel.org,
 Dan Williams <dan.j.williams@intel.com>,
 Bard Liao <yung-chuan.liao@linux.intel.com>,
 Pierre-Louis Bossart <pierre-louis.bossart@linux.intel.com>,
 Sanyog Kale <sanyog.r.kale@intel.com>,
 "Dommati, Sunil-kumar" <Sunil-kumar.Dommati@amd.com>
References: <2024013025-spoiling-exact-ad20@gregkh> <ZbpFTyW9UCZdCs_v@matsya>
 <2024032742-armless-cage-7c6c@gregkh> <ZgQWNjZ9EfhU5A9X@matsya>
From: "Mukunda,Vijendar" <vijendar.mukunda@amd.com>
In-Reply-To: <ZgQWNjZ9EfhU5A9X@matsya>
Content-Type: text/plain; charset=UTF-8
Content-Transfer-Encoding: 7bit
Precedence: bulk
MIME-Version: 1.0
X-MS-Exchange-AntiSpam-MessageData-ChunkCount: 1
X-MS-Exchange-AntiSpam-MessageData-0:
	=?utf-8?B?NlJwTkpZUHZEejQ1c2R6dDFiZDFra0ZZbDVEeHlXYlF1d29ValBSSjBDWXJ1?=
 =?utf-8?B?dUowQjFBcXhib2Npb3JzRWhBRGV6dFkwVFNkNVYrS3NZamZ6SGhDV0c4Zkg2?=
 =?utf-8?B?NFVLQlZ5K1pwV0RmVDQ2TUh3amM3SVpoeTVuMmFiOWU3Z0VmeCt0MStxRUN1?=
 =?utf-8?B?dzRFYnVYMGYycGNYS083akV6cE9RRG9xZkFTZVo1VXY4NlR4cGNEVVdKd1la?=
 =?utf-8?B?MlNESGZaM01HWXQwTmNleXJNRU53Mk1wcTBXd3lURzQrWnEvRVY3LzFuKytW?=
 =?utf-8?B?RDVZclAvOVZDQ21kRGJiY1pkNER6Sk02UmdqWmdqeUZYQkgxTC9yVmV6SjlO?=
 =?utf-8?B?SE95ZG5Idkc2ekJiZlZUWUVwQnJneldzaGRoMTl5Uno3VWFxQS9oY1VIQ0FH?=
 =?utf-8?B?aVVyakcvOGd1b2dnWHRNMVd4VGhxa2paaUN6NFpSQTR6Z0pHWk1rV3Yxdmln?=
 =?utf-8?B?UGJKMCtGandUNFdaNWNHNmxNV1FBajZadjE1QzZVeEsrdGtzQ0FkU3lIbUxo?=
 =?utf-8?B?UGZHdzRSNkhwRm80eEhMK20xbGZMZDZ0cVlhY0wwcEFUMDZlN3YvUUQxdEVD?=
 =?utf-8?B?L1Zmd2xqQ1UySC9SeTZ4N0xPRnNlQVlPMUVJZnJkV09RTVF0WlU2bHpZQTBM?=
 =?utf-8?B?WUhhVGhqOFQzdkI3bUpDeGdVS1BRYWJqN1ZtV3VQTlJybC84K3Z0Yytxd1k1?=
 =?utf-8?B?T2c5a0x0NytKNXFYcWdTNlljRFJYSVU4UGk0NmFabGNldUNyeWRwei82SU13?=
 =?utf-8?B?RXlJTjJKWUlCL3VRdDJrM0xDR3h3VUpQemt3OXBOMmpZMkV4SlFOUDlVTkUw?=
 =?utf-8?B?S2MxNTZrY0JXckprM256ZVBCUElSL0t1aDBzQUduNlR0RWpVRHhUWW9qOFMw?=
 =?utf-8?B?cWk0cHZGcWp0bmlLWUlwOWxkc1daT1Bsb3Jib2ZZME1qSSs1clpsUTd4czBr?=
 =?utf-8?B?eGVmMnNIVXFibXhCeEZaNmV0eWwzRG5OaHJ1TWdDRWJZdTVDK2N2Ukd1M0hu?=
 =?utf-8?B?TFBncFpSbzdQenpXOGJOcVFqREQvekdYUzNOQU9YSXUvaWpkcEx3OXRCTDNP?=
 =?utf-8?B?dDVxbEY1ckZzenpMaTlWd0ZlUW9tTW1GTHVHT01Qemt6QVl3QkgzdktQR1RB?=
 =?utf-8?B?ZFVMNVdFbkFwMmVJdEhZUjIzS24ybW8wVUh0TS9Hb2dkaWcwVzlKdWJDbzlR?=
 =?utf-8?B?allHbVllUEhxZVZScC9zbU90dHdoZjRXTDNVYVlHdUV2UGlWdVFOU3h5RUpW?=
 =?utf-8?B?b2Y2Szdza0d1OGU1ZzkvVW9pVS8rbk95NVVuVTlUa0YxdVFpeThuM0tXR3Zw?=
 =?utf-8?B?bThNWWlRVHRLUDlhMVNqK1VpRWdjNkN1Tk91cWhmck03ZktqeDRTWWU2R2lE?=
 =?utf-8?B?QldJZHQ1OUJLV3dWOTVKYjdqUHdHSnBNRmZVSmRGZmw3bTd1RDRjTG1RSTFB?=
 =?utf-8?B?YkdiZG5LblY2RTFLMDkvZ2Ftbm9xTFd6eVJkZllzNWtoVExDVUx4a0dmTk13?=
 =?utf-8?B?QitZZ0dOeENyOUI5Y2JEN3BhcnYyQm0vU1BSNDVwdlZYOG1XK1JvYkJrcDdm?=
 =?utf-8?B?TUxMYUkyUjBldXVKV0pLN29UN2pxYXkxVW1kMG9jZmZtRFdhWXNuSlZqcmpV?=
 =?utf-8?B?eFhLV3BjeS9oRWpKQ1pLK1RZcitnMXVjaXFhT3hwZXprZjdIMEkyM04vRm8r?=
 =?utf-8?B?U3ZjcHJpenRRMXljQ2ZJYnlmQ3dwZ2NWK1lZZXBsOEladjNDdUpBQmZMSDcv?=
 =?utf-8?B?TlpQMlhzVW53ZWZOL0JwM3NDbHFaZkxuNU1EaWNYVnpBdlJTems1VlJXVVJn?=
 =?utf-8?B?Q1ZzRUNEYVMrdUJHMks0WjR5eEhGbzN1TXR4WC9vUUxsdGJLSkMwRnpGOUxk?=
 =?utf-8?B?cC9iQ2VCcGFtR0hOUTVnalRGd0Q3SXR3dlpXMTJCV0FqVjd0ZEJ3Y2VHQlZW?=
 =?utf-8?B?c0xjV2R4WmkyY09PRFBWOGYwVXhBbCtwbmVWZlQ1NVE5VGYzSXBGck9nRjNz?=
 =?utf-8?B?aW9PZXRKSmFrcUFiZTdFRmJ6VmlXZDVoUWJmMkRGaTJueWo1d2NMeloyZ3Ju?=
 =?utf-8?B?bUViSU1NTDBiQkFyMWt0NjNBSzE5bmhKK1YzQm1WTnpIY1pwZkFrK3NzVHk3?=
 =?utf-8?Q?8zYeQolS/IOi2uDlF4aP2k/D4?=
X-OriginatorOrg: amd.com
X-MS-Exchange-CrossTenant-Network-Message-Id: 878cdfae-8261-43a6-b996-08dc4f21f5f9
X-MS-Exchange-CrossTenant-AuthSource: DM6PR12MB4123.namprd12.prod.outlook.com
X-MS-Exchange-CrossTenant-AuthAs: Internal
X-MS-Exchange-CrossTenant-OriginalArrivalTime: 28 Mar 2024 12:24:06.0093
 (UTC)
X-MS-Exchange-CrossTenant-FromEntityHeader: Hosted
X-MS-Exchange-CrossTenant-Id: 3dd8961f-e488-4e60-8e11-a82d994e183d
X-MS-Exchange-CrossTenant-MailboxType: HOSTED
X-MS-Exchange-CrossTenant-UserPrincipalName: uCD+hDZtduQvRLqS7X815RDc0XWyx+G/FToynQJA+voRPZbkLEdmALCYVQNA1ZUps1kScAEuqEynTsanL4lT/Q==
X-MS-Exchange-Transport-CrossTenantHeadersStamped: PH0PR12MB7012

On 27/03/24 18:21, Vinod Koul wrote:
> On 27-03-24, 09:13, Greg Kroah-Hartman wrote:
>> On Wed, Jan 31, 2024 at 06:34:15PM +0530, Vinod Koul wrote:
>>> On 30-01-24, 10:46, Greg Kroah-Hartman wrote:
>>>> Note, this is a redone version of a very old series I wrote back in
>>>> 2022:
>>>> 	https://lore.kernel.org/r/20220824135951.3604059-1-gregkh@linuxfoundation.org
>>>> but everyone has forgotten about it now, and I've reworked it, so I'm
>>>> considering it a "new" version, and not v2.
>>>>
>>>> Here's a series that adds the functionality to the driver core to hide
>>>> entire attribute groups, in a much saner way than we have attempted in
>>>> the past (i.e. dynamically figuring it out.)  Many thanks to Dan for
>>>> this patch.  I'll also be taking this into my driver-core branch and
>>>> creating a stable tag for anyone else to pull from to get it into their
>>>> trees, as I think it will want to be in many for this development cycle.
>>>>
>>>> After the driver core change, there's cleanups to the soundwire core for
>>>> how the attribute groups are created, to remove the "manual" creation of
>>>> them, and allow the driver core to create them correctly, as needed,
>>>> when needed, which makes things much smaller for the soundwire code to
>>>> manage.
>>> The series lgtm, having the core handle these would be good. I will wait
>>> couple of days for people to test this and give a t-b and apply.
>>> I hope it is okay if patch1 goes thru sdw tree?
>> patch 1 is now in Linus's tree, so the remaining ones can go through the
>> your tree now if you want.  Or I can resend them if needed, just let me
>> know.
> Great, I was about to ask about this. If there is no conflicts I can
> pick this series (looking at folks for giving me a t-b)
Applied this patch series on top of soundwire git tree and validated SoundWire
stack on AMD platform using command line alsa utils. All use cases are working
fine. Tested-By: Vijendar Mukunda <Vijendar.Mukunda@amd.com>

>