Received-SPF: pass (google.com: domain of linux-kernel+bounces-712-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: <188ee32a-a201-4210-a8bc-bf3ba04e2f2b@amd.com>
Date: Fri, 15 Dec 2023 15:28:13 +0530
User-Agent: Mozilla Thunderbird
Subject: Re: [PATCH 11/11] ASoC: SOF: topology: Add new DAI type entry for
 SOF_DAI_AMD_BT
Content-Language: en-US
To: Cristian Ciocaltea <cristian.ciocaltea@collabora.com>,
 Mark Brown <broonie@kernel.org>
Cc: Liam Girdwood <lgirdwood@gmail.com>, Jaroslav Kysela <perex@perex.cz>,
 Takashi Iwai <tiwai@suse.com>,
 Pierre-Louis Bossart <pierre-louis.bossart@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>,
 Daniel Baluta <daniel.baluta@nxp.com>,
 Kai Vehmanen <kai.vehmanen@linux.intel.com>,
 Alper Nebi Yasak <alpernebiyasak@gmail.com>,
 Syed Saba Kareem <Syed.SabaKareem@amd.com>,
 Kuninori Morimoto <kuninori.morimoto.gx@renesas.com>,
 Marian Postevca <posteuca@mutex.one>,
 Vijendar Mukunda <Vijendar.Mukunda@amd.com>,
 V sujith kumar Reddy <Vsujithkumar.Reddy@amd.com>,
 Mastan Katragadda <Mastan.Katragadda@amd.com>,
 Ajit Kumar Pandey <AjitKumar.Pandey@amd.com>, linux-sound@vger.kernel.org,
 linux-kernel@vger.kernel.org, sound-open-firmware@alsa-project.org,
 kernel@collabora.com
References: <20231209205351.880797-1-cristian.ciocaltea@collabora.com>
 <20231209205351.880797-12-cristian.ciocaltea@collabora.com>
 <fad8a055-eabb-4087-94d5-9e1de00933e4@amd.com>
 <aa830670-e544-43a2-9ba9-a64f1964a9f5@collabora.com>
 <318470ce-1631-4c46-b425-755c877dda65@amd.com>
 <421128f7-6a17-4be9-a72b-272ea4017fbd@collabora.com>
 <ZXXEsyBUCrBULNgk@finisterre.sirena.org.uk>
 <5095ce7b-13bd-4805-b81e-f7565ab41b67@collabora.com>
 <dea5fb18-5fdc-4be4-9981-a6876cf531eb@amd.com>
 <ca828ee1-93f7-432e-b95d-e67c35cdcdb5@collabora.com>
 <0de8f81b-e7d4-43f6-b011-eb6ee09cc7fe@amd.com>
 <913257d4-75dc-4887-97d6-0f206c401057@collabora.com>
From: Venkata Prasad Potturu <venkataprasad.potturu@amd.com>
In-Reply-To: <913257d4-75dc-4887-97d6-0f206c401057@collabora.com>
Content-Type: text/plain; charset=UTF-8; format=flowed
Content-Transfer-Encoding: 8bit
Precedence: bulk
MIME-Version: 1.0
X-MS-Exchange-AntiSpam-MessageData-ChunkCount: 1
X-MS-Exchange-AntiSpam-MessageData-0:
	=?utf-8?B?a0FwZEE0cG5BQUlSTWFqZXBueFpIc0ZsZG9icmV1STRhUlp3OFM2OTdIc24z?=
 =?utf-8?B?NHVRempDSHNnRFhmTy9IaXhyWTNySjBsTTlVSXdPWURIdXhZQTN4ZVRIbUlj?=
 =?utf-8?B?cVNoVzJwckcrTi9YWGdQYXJ3TUVaSTlCQU5FS1Rsa2k4R1RkWXY4cEJRNExz?=
 =?utf-8?B?Q0hTTzdCVGhGVEkxVGlwWCt6Q3VMdjdjZlRwMGhsMVJqL0hlcHFkWkZVeWlG?=
 =?utf-8?B?NU1ZVlNCajlZMDhINXkwa2h0ZTJjMFRLaEw1d0ZTeWs1d2pkQXNsak9vS096?=
 =?utf-8?B?cTBRRG4zWFdPSlJZVEtVTzRpMHRtSExTbmNkWWx4cVhWOXpJS3k5Y1BIVW1Q?=
 =?utf-8?B?RCs1czBRcGtRUjZ5MHpsbVk5MHhXWUFYT2xmbCs5cktJbWtyNEMyOXdxVVRH?=
 =?utf-8?B?Sk10YnlyUVFZMzJNZHNoUWxvTjBlSmhNYVdmcVIra0Zrazd6a3N5WDdDWmlj?=
 =?utf-8?B?ZC85VFdpQXUybTM0QVl5enRxdGJST2l6NHJ5OFJKZURwbnZyN044dCs2cS9y?=
 =?utf-8?B?QWxjSCtEVlE3cEQvM0lETFczaUM5UWU5YXAvbDhZTEJHU2pwczc2aXhNbkZa?=
 =?utf-8?B?bXZDaGEwY2ZVOTdvSFFDZFYxQjdoTnNwR0MyM08rUWg0QmlMMFNXU1czZE9T?=
 =?utf-8?B?VnM3SEJHNndIa1BTbnMzNjBHSDJWK01NNjlKaGhaTFpaNzBpMlVXOUcwa0h3?=
 =?utf-8?B?c01rRlJVbHJ0SE92WVo1UGg2bzhiKzYwbUYreUNpVytsakRUUVpqZkNmUXoy?=
 =?utf-8?B?ZmF6NmZGbUhvRWM5N3RNeS9WbHo1MkJ4aFhIUU4wRW15SHZCaUJUOCs4MHh4?=
 =?utf-8?B?dDhIUTFzL0FkOG0vaWJKQUZSeVc1Si8rc0RTWGM4TEVqNHZrSXNoblR0TEJE?=
 =?utf-8?B?a0JKQUhWRzhsZFFhc2ZkWFlXWkZoU1V2aUhIUC9MdzdPU3dIaU9YNHZ0bWRZ?=
 =?utf-8?B?M1M0b1JkT1FZSGxoT1BrQmpJaUJLektBbmJtRjhPN0xnTGpRb3RKWUZYWmRi?=
 =?utf-8?B?ck5OZEZzcDMvQk1IeCtuQ01iNUNBRVpYU3ExeUlhQ0swbFpSWklXNjZybFFJ?=
 =?utf-8?B?Snc5UFFHUm5oSFFWdTlIRW5YRDJpVUQxRk85S1g4RTNrTG9TNEY5ajM0d0tB?=
 =?utf-8?B?S1VPY1lzVFBtRFJSUTl1bVFrNGE3M0xiME5GVVpKT3FaYzFvSDJyMDNybW5U?=
 =?utf-8?B?aGh2SmlVbFNjNmllY2pkekw4eFB6NHd5T2RwODJBTDdxR1dpUERsQmptZHdW?=
 =?utf-8?B?Q0JxSjdnejdvWnkrdktEZVlWS0JqN2lNTzdydFFBaGpWb2hqRDcrMUlnTllh?=
 =?utf-8?B?YXQwTVVPdnBHOVpnKytLTzk1OFgzN0wyZ3lMZVJYTnJBWGIvZzVJbzBzZHZw?=
 =?utf-8?B?aU9QbDRWNHhSNWNmajlVeU1rU0hjbVpBWU1FN1d0VEpQS2RQNWRkRUpUVURn?=
 =?utf-8?B?Y2hKR3d6K3NVMlNDNFFkUXFrT1BSNmQxOWtTQzFoRnQzTmdSMWFDaGdoS0s4?=
 =?utf-8?B?ZE9qenNpZ1lSSHArRlVGQWR1d2pRdlpjcHdrMFA5N09YSFFxbnRXVWtKdHAw?=
 =?utf-8?B?QUdXTFlGVXF0UzdTbFhNVCtKM2tBSU1hdjZqb3dPRnA3cjQ1UlZGNDgrRHNy?=
 =?utf-8?B?cVJVNVM5OFRVemI2RUZRQ1ZFQjZIWFoxM3U3UFgzVG1MdzdFWDh4ZjRQOFkz?=
 =?utf-8?B?UDRueFFsRE53TlZSZzVJbUNnWU12TXFaZm9UQktqSy90RE8zb3N1SGhHMHlh?=
 =?utf-8?B?UnNiNHM5bWNPOGVteWhvK1NxNUxkVU4vUzRVNmZ0UmJYN1RmS1NqUXYyTEU4?=
 =?utf-8?B?cFJLUHhDT1NmeU9DdFMycXU3SjRONGUyQlZoRENIRGp2QWdlOFRvMHFrbEs3?=
 =?utf-8?B?QXhhbkVZTTNQYlVBUjVCbHBrQ0U1MDJDTFhWSUtWeWpzV2I2aEg0VmMycHlp?=
 =?utf-8?B?TjZUTjI0YWFWdmFBV2NiYUpIRVBvSGlQb1c1WDRiZEo2VSs2VkhYNkpkbHdw?=
 =?utf-8?B?ZERXdGlJUHRMVFZlNlhrYkhmeU1zMTk1Nnp5cXNkTWk3MWhNOGFqZ3puQSsw?=
 =?utf-8?B?bHZuWnFvcWlXaXV3elVHZ2lCNG83UFd1REVEc1QwNU1LalFldGJiRUZEZGVS?=
 =?utf-8?Q?LgvCCmjXW3VpV6JzTyJJagsjP?=
X-OriginatorOrg: amd.com
X-MS-Exchange-CrossTenant-Network-Message-Id: 87ab0af2-e0bf-4426-b602-08dbfd5460a9
X-MS-Exchange-CrossTenant-AuthSource: PH7PR12MB5951.namprd12.prod.outlook.com
X-MS-Exchange-CrossTenant-AuthAs: Internal
X-MS-Exchange-CrossTenant-OriginalArrivalTime: 15 Dec 2023 09:58:24.7233
 (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: X+zlk1LSnf55pXvsoA+9anacTN7oBswgo2dykE8BXcLZDSJSaV063gSxjjmYX6xanoL+3ToeUu1kId1wK7k/Cw==
X-MS-Exchange-Transport-CrossTenantHeadersStamped: SJ2PR12MB7961


On 12/14/23 22:12, Cristian Ciocaltea wrote:
> On 12/14/23 15:15, Venkata Prasad Potturu wrote:
>> On 12/14/23 17:53, Cristian Ciocaltea wrote:
>>> On 12/11/23 07:58, Venkata Prasad Potturu wrote:
>>>> On 12/10/23 21:20, Cristian Ciocaltea wrote:
>>>>> On 12/10/23 16:01, Mark Brown wrote:
>>>>>> On Sun, Dec 10, 2023 at 12:12:53PM +0200, Cristian Ciocaltea wrote:
>>>>>>> On 12/10/23 11:51, Venkata Prasad Potturu wrote:
>>>>>>>> This should send to SOF git repo for rewiew, once SOF reviewers
>>>>>>>> approved
>>>>>>>> this, again need to send to broonie git.
>>>>>>>> All the changes in sound/soc/sof/ path should go to SOF git.
>>>>>>> Unfortunately I'm not familiar with the SOF dev workflow. So it's not
>>>>>>> enough to have this patch cc-ed to
>>>>>>> sound-open-firmware@alsa-project.org?
>>>>>> The SOF people basically do their own thing in github at
>>>>>>
>>>>>>       https://github.com/thesofproject/linux
>>>>>>
>>>>>> with a github workflow and submit their patches upstream in batches a
>>>>>> few times a release, however my understanding is that their
>>>>>> workflow can
>>>>>> cope with things going in directly upstream as well.
>>>>> Thanks for clarifying, Mark!  That would greatly simplify and speedup
>>>>> the whole process, at least for trivial patches like this one.
>>>> Hi Cristian,
>>>>
>>>> We have created a Pull request in SOF git hub for I2S BT support.
>>>> please hold v2 version SOF patches till below PR get's merged.
>>>> PR:- https://github.com/thesofproject/linux/pull/4742
>>> Hi Venkata,
>>>
>>> If this is going to be handled via the github workflow, this patch
>>> should be removed from the series.  Since there is no dependency on it,
>>> I cannot see a reason to put v2 on hold.
>>>
>>> Do I miss something?
>> Non-sof driver related patches can directly send to broonie git ad v2
>> series.
>> SOF driver patches should send to SOF github to avoid merge conflicts
>> as  per guidelines of SOF community.
> Honestly, I don't really see a high risk of conflicts, the patches are
> not that complex and can be simply cherry-picked when needed.  Moreover,
> as we already had people reviewing this, splitting this up will only add
> confusion and unnecessary burden.
>
> Are there any specific changes you are concerned about and cannot be
> really handled here?
This is not the concern about this patch series,
Generally sof driver patches sends to SOF git hub as a PR, these are the 
guidelines by SOF maintainers.
If you still want to send alsa devel list directly, please discuss with 
SOF maintainers.