From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: X-Spam-Checker-Version: SpamAssassin 3.4.0 (2014-02-07) on aws-us-west-2-korg-lkml-1.web.codeaurora.org X-Spam-Level: X-Spam-Status: No, score=-2.0 required=3.0 tests=DKIM_INVALID,DKIM_SIGNED, HEADER_FROM_DIFFERENT_DOMAINS,MAILING_LIST_MULTI,SPF_HELO_NONE,SPF_PASS, USER_AGENT_SANE_1 autolearn=no autolearn_force=no version=3.4.0 Received: from mail.kernel.org (mail.kernel.org [198.145.29.99]) by smtp.lore.kernel.org (Postfix) with ESMTP id 827E5C433E0 for ; Mon, 6 Jul 2020 11:29:51 +0000 (UTC) Received: from vger.kernel.org (vger.kernel.org [23.128.96.18]) by mail.kernel.org (Postfix) with ESMTP id 5ECC42073E for ; Mon, 6 Jul 2020 11:29:51 +0000 (UTC) Authentication-Results: mail.kernel.org; dkim=fail reason="signature verification failed" (1024-bit key) header.d=mg.codeaurora.org header.i=@mg.codeaurora.org header.b="s4vd8vzZ" Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1728940AbgGFL3u (ORCPT ); Mon, 6 Jul 2020 07:29:50 -0400 Received: from mail29.static.mailgun.info ([104.130.122.29]:38740 "EHLO mail29.static.mailgun.info" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1728441AbgGFL3t (ORCPT ); Mon, 6 Jul 2020 07:29:49 -0400 DKIM-Signature: a=rsa-sha256; v=1; c=relaxed/relaxed; d=mg.codeaurora.org; q=dns/txt; s=smtp; t=1594034989; h=Content-Transfer-Encoding: Content-Type: MIME-Version: Date: Message-ID: From: References: Cc: To: Subject: Sender; bh=B4x/MNKnw7dNqtrqeARr2ZbfiWYvHwIO7a3zTHiX2RQ=; b=s4vd8vzZ13Nx8lIkGMHdLz7oVfw6+DAo1aWvp2EqzPdZ+Ale0kHQDX0rTSeV4cSOf+jiD1KS Vg/+ULPwhUXUQ+aA0F97E0n4oVnExQqJOOw8NUfMXGSTwroGz3Bb8Zgy+z5QnPn8z/DOmOyZ ZprV7a/Lj253Bn41g+paA8n17l4= X-Mailgun-Sending-Ip: 104.130.122.29 X-Mailgun-Sid: WyI0MWYwYSIsICJsaW51eC1rZXJuZWxAdmdlci5rZXJuZWwub3JnIiwgImJlOWU0YSJd Received: from smtp.codeaurora.org (ec2-35-166-182-171.us-west-2.compute.amazonaws.com [35.166.182.171]) by smtp-out-n07.prod.us-east-1.postgun.com with SMTP id 5f030b2a86de6ccd44b789fe (version=TLS1.2, cipher=TLS_ECDHE_RSA_WITH_AES_128_GCM_SHA256); Mon, 06 Jul 2020 11:29:46 GMT Received: by smtp.codeaurora.org (Postfix, from userid 1001) id 95BB1C43387; Mon, 6 Jul 2020 11:29:45 +0000 (UTC) Received: from [192.168.0.129] (unknown [183.83.142.110]) (using TLSv1.2 with cipher ECDHE-RSA-AES128-GCM-SHA256 (128/128 bits)) (No client certificate requested) (Authenticated sender: rohitkr) by smtp.codeaurora.org (Postfix) with ESMTPSA id 536EBC433C8; Mon, 6 Jul 2020 11:29:42 +0000 (UTC) DMARC-Filter: OpenDMARC Filter v1.3.2 smtp.codeaurora.org 536EBC433C8 Authentication-Results: aws-us-west-2-caf-mail-1.web.codeaurora.org; dmarc=none (p=none dis=none) header.from=codeaurora.org Authentication-Results: aws-us-west-2-caf-mail-1.web.codeaurora.org; spf=none smtp.mailfrom=rohitkr@codeaurora.org Subject: Re: [PATCH v2 1/7] Documentation: device-tree: sound: Update lpass-cpu driver binding To: Ajit Pandey , Mark Brown Cc: devicetree@vger.kernel.org, alsa-devel@alsa-project.org, bgoswami@codeaurora.org, plai@codeaurora.org, linux-kernel@vger.kernel.org, srinivas.kandagatla@linaro.org References: From: Rohit Kumar Message-ID: Date: Mon, 6 Jul 2020 16:59:39 +0530 User-Agent: Mozilla/5.0 (Windows NT 10.0; WOW64; rv:68.0) Gecko/20100101 Thunderbird/68.9.0 MIME-Version: 1.0 Content-Type: text/plain; charset=windows-1252; format=flowed Content-Transfer-Encoding: 8bit Content-Language: en-US Sender: linux-kernel-owner@vger.kernel.org Precedence: bulk List-ID: X-Mailing-List: linux-kernel@vger.kernel.org On 5/16/2020 11:07 PM, Ajit Pandey wrote: > On 5/14/2020 10:14 PM, Mark Brown wrote: >> On Thu, May 14, 2020 at 10:08:12PM +0530, Ajit Pandey wrote: >>> Done the required cleanups to update lpass-cpu binding with newer >>> yaml formats. >> Please do YAML conversions as the last thing in any patch series, there >> is quite a big backlog of reviews of YAML conversions so they can slow >> down the code development.  It's good to do them, just try to make sure >> other patches don't depend on them. >> >> Please submit patches using subject lines reflecting the style for the >> subsystem, this makes it easier for people to identify relevant patches. >> Look at what existing commits in the area you're changing are doing and >> make sure your subject lines visually resemble what they're doing. >> There's no need to resubmit to fix this alone. > > Ok.. we'll do the yaml conversion as a separate patch chain so that it > won't block > > merging of this feature in ASoC core. @Mark, I was planning to post v3 patchset series for this and keep yaml change as last patch in the series as suggested. However, we have one update in the driver to support another compatible string. If we keep Documentation change at the last, the driver change will have checkpatch errors. Can you please suggest if should make first Documentation change in text file itself before driver change and finally have a patch to convert it to yaml at the end? Thanks, Rohit -- Qualcomm INDIA, on behalf of Qualcomm Innovation Center, Inc.is a member of the Code Aurora Forum, hosted by the Linux Foundation.