Received-SPF: pass (google.com: domain of linux-kernel-owner@vger.kernel.org designates 2620:137:e000::3:1 as permitted sender) client-ip=2620:137:e000::3:1;
Message-ID: <6e9bbea9-bd9b-4c98-a3da-73ccd8826872@amd.com>
Date:   Thu, 14 Dec 2023 18:06: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,
        "Hiregoudar, Basavaraj" <Basavaraj.Hiregoudar@amd.com>,
        "Dommati, Sunil-kumar" <Sunil-kumar.Dommati@amd.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>
From:   Venkata Prasad Potturu <venkataprasad.potturu@amd.com>
In-Reply-To: <ca828ee1-93f7-432e-b95d-e67c35cdcdb5@collabora.com>
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?ZGJNVDhvS0lhWFgyNnIyNncrR2RES1VzMUNsSi91RUJtRWNOMnJOaTRwTHdT?=
 =?utf-8?B?SFdDanNIZUxEdjNxSGVtc3MvNmh1TzJnTERZQWlERHZ2QTAzS1oxN0xyTSto?=
 =?utf-8?B?SE9VdEtnc2Y5a0FqTkFMMFpyUE5aS2NwUG5PalFGUVM3b3ZpOEVLSkYvT010?=
 =?utf-8?B?bVpoVDFjbUIrWjZ0ZU40SEFsT2N2THhJZ2lwNUxmNTYrVnRVd09scE5qdjhr?=
 =?utf-8?B?aHloaURaVzh2T2xyajBqZmFJS29CRTNla1VBendBQU8xbmdoL0RhdEhpYi9S?=
 =?utf-8?B?S1lmc2NhV05wbnVueGdDUUlhT0FuN2ZmempkUE5sd08rc211TDNtS2htRXZk?=
 =?utf-8?B?N2xob1BjS0FRb05BYzZycCtNM3lKQjJUYzZOODJXTjFSbmpyU1ZtclBoUDNR?=
 =?utf-8?B?UHRnUTJvaHBNU2dTdTRDQlhGeVYySkhaVzRDT2lLNDdTT0drS08vUEJvNlQz?=
 =?utf-8?B?bFZFMzlFMjh5WlJBMWdEM05TcU04Y0daYVpKSm9IU0s5M2p0OGlJckg5S09B?=
 =?utf-8?B?M3ltcGV1S29CVWNQYUhpZ0t6VmVnNWpmc1RFU2N3Y3NwVnQzNm5pRCs2cGJx?=
 =?utf-8?B?SHFMRUFQV0Vab2oyTGNPa0RjMnlYNkxOU3lmTWYrTi9HQjl3eko1N05PLzU4?=
 =?utf-8?B?dExmbnBKT3FXTThOY1hoNDI2V1pQNmR1ODluRllmdWZzMlZZbzY5enhjSHg1?=
 =?utf-8?B?OEJ0azhVbUZFRG95ditPL0QxdFJnNnVuTGpyMnRVT0M1TFBULzlTSmQ0WTc5?=
 =?utf-8?B?TWxoMGhOWFZYRC9rK0kxVlNtaElRazV5YnhCbEN2Y2gyb1ozWGJ4bGEvdUZw?=
 =?utf-8?B?ZHJHemxkVXVUc2UwZXFHVTd4OVhHU09UMjYrdWYvNXo0dWpHaWZFS1grdTJZ?=
 =?utf-8?B?cy9NNWo0R0ViTmYvMjVJVkl3TDl2VVFQZFJGQ3c1VjNSYWM0TG93SVVqdi9L?=
 =?utf-8?B?eWtsVm91cXdpdE9qY2lkYkMrRGR6QUFCOVhMckFYRGNkQkcxZ1IvQlVja3Bh?=
 =?utf-8?B?L0YwRVpIZnpYcVRLbEh5UkVCbXAyMC9OYkpVcTJEK0tybHU4Mk53REZHUXBz?=
 =?utf-8?B?dnJXQ3Z5Qm45SS9UbzNJdVpSVlZOVFVFL0owSjhyd0x2TldNdnpjLzVBZEdT?=
 =?utf-8?B?YWRyZElYVmNJeVNjanZxQTVQQStaSEdXY1V4L2lLVVp5bUduZW9jUUJEcnNj?=
 =?utf-8?B?STVRQzRnQ0c0NzB4ak5oRmFGaHBkcXFZZ05NV1UvUm5VeFZQSUdHMXl1UWtv?=
 =?utf-8?B?ZVYrVGVjNHA3TjN1eVdGdTVROW05MkNyQWV2cy9sNWEvYndudlhhQzNHQ2gx?=
 =?utf-8?B?cDZhZ1BBVWEvT0xxWS81OWdIaTkyY3FncEVRVU4vU0lzVEliN3dVSlNHRFlp?=
 =?utf-8?B?Q0UzekYwanhjcUVoRXlJajlDeUZBM3JHUEp6YlZzei9LZ0k3R2l6Ylk1blhu?=
 =?utf-8?B?eVVkYkVLc1ROc0tBQ29pbGhwSUx1YWZlNGdpVE1IQ0g5OFRtWjB5NVIwcU4r?=
 =?utf-8?B?NjRYR1lVUFBWU2lzdXNuMlovU3ZCU1RrdkppWVJpMWZXZGVnNnJtbGt1VjZv?=
 =?utf-8?B?MHpCQnoyc2d6WUs1NWR3NUtnUEoyQWxFQjJWU2o1SzVQQmtQSWZodmVoUW9y?=
 =?utf-8?B?Ym1NSHVpaXo3Z2JWeEloMDZqczNPUi9SWjN3TVpnd3lBWlFGNThvY1hkOVBh?=
 =?utf-8?B?dDJ4bEJ5dzJCTE1uRUFRR2tUalBONkZWbWUxS1ZqQWFSUFB2Mm9lK0J4cDFZ?=
 =?utf-8?B?Mm9LM3A2KzdlUjNxdWk3M1VYM3E3dlNaM0U3ODNDRDcwUktDNFhNbFNQeDlt?=
 =?utf-8?B?aGVMNUR5YVNsdW1JY1hHOHlOZXcwbitPWFBKTmNjcWNrR01hZ2NCZjVPbi9r?=
 =?utf-8?B?L3B0d3FOZXR3UUVDejFEUGRJcGE5Vm1KK3pDNnNZNmh6c3RSaXF2NW16M24z?=
 =?utf-8?B?c0Q4YnNmZy9xL3VhdkJWc04xZ1pLQjM0ZERDTGhDckZud2ZxSU5xbFAxOU1J?=
 =?utf-8?B?OW1pak5BSzFTZGlWTytXOUt3Z3lJRTB6cnB3MXJkdXpLNm5DTDQ2V1ErZ3JR?=
 =?utf-8?B?NWNPRURoNFMvczIyWk1RZkJtVytIOVVNZVhwTE5xN3EvUHIrTDg0eE1aNzZN?=
 =?utf-8?Q?GDQ31XGr90aTKsbl2lN+/ycxd?=
X-OriginatorOrg: amd.com
X-MS-Exchange-CrossTenant-Network-Message-Id: bd13ee88-9a96-44d4-d55e-08dbfca1495b
X-MS-Exchange-CrossTenant-AuthSource: DS7PR12MB5960.namprd12.prod.outlook.com
X-MS-Exchange-CrossTenant-AuthAs: Internal
X-MS-Exchange-CrossTenant-OriginalArrivalTime: 14 Dec 2023 12:36:27.0054
 (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: rBxUu9NVRgetaaJVKwTrRAHvZ6Fhid7j52SMvfO8oI49WPVwxFD1rmtpqItEYD3B8NpIbVIebk3kLsH8/0loTQ==
X-MS-Exchange-Transport-CrossTenantHeadersStamped: DS7PR12MB8203
X-Spam-Status: No, score=-0.9 required=5.0 tests=DKIMWL_WL_HIGH,DKIM_SIGNED,
	DKIM_VALID,DKIM_VALID_AU,HEADER_FROM_DIFFERENT_DOMAINS,
	MAILING_LIST_MULTI,SPF_HELO_NONE,SPF_PASS,T_SCC_BODY_TEXT_LINE
	autolearn=unavailable autolearn_force=no version=3.4.6
X-Spam-Checker-Version: SpamAssassin 3.4.6 (2021-04-09) on morse.vger.email
Precedence: bulk
List-ID: <linux-kernel.vger.kernel.org>
X-Mailing-List: linux-kernel@vger.kernel.org
X-Greylist: Sender passed SPF test, not delayed by milter-greylist-4.6.4 (morse.vger.email [0.0.0.0]); Thu, 14 Dec 2023 04:36:40 -0800 (PST)


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?
Yes, need to drop this patch.

And it's better to follow the github workflow by creating a Pull request 
in SOF github

  for all sof driver related patches, rather than sending patches to 
broonie git directly.

>
> Thanks,
> Cristian