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 Received: from bombadil.infradead.org (bombadil.infradead.org [198.137.202.133]) (using TLSv1.2 with cipher ECDHE-RSA-AES256-GCM-SHA384 (256/256 bits)) (No client certificate requested) by smtp.lore.kernel.org (Postfix) with ESMTPS id A1E67C04AA5 for ; Thu, 25 Aug 2022 06:08:28 +0000 (UTC) DKIM-Signature: v=1; a=rsa-sha256; q=dns/txt; c=relaxed/relaxed; d=lists.infradead.org; s=bombadil.20210309; h=Sender: Content-Transfer-Encoding:Content-Type:List-Subscribe:List-Help:List-Post: List-Archive:List-Unsubscribe:List-Id:In-Reply-To:From:References:Cc:To: Subject:MIME-Version:Date:Message-ID:Reply-To:Content-ID:Content-Description: Resent-Date:Resent-From:Resent-Sender:Resent-To:Resent-Cc:Resent-Message-ID: List-Owner; bh=nYfqggpasRQW+ywchJ3R4ABZj+izKWItheM9H+SZr/Y=; b=prGx4EfVDH1rvh UivgP9ORuKtN9hDXTMOUSdGbemcDQ76H6sqdHLeBQJB0WR1BgJ6pAcc7HzBdaTF+DQ5tcGz9b9H7m JbYvED7WAlyKt0EKy6Yr/929WZ3+ttGPX8hcA6rZsICxSXJR2BqtjLNCuXZ/E2uLaFzL8PSlkCPrR iL9jplhK5ICjlFq4ws/+uvgk2zZlLmGtSpKTUauhdEJqPLESfJ5BvVzt1XHAxotBps04jgvfRPYiF GxdlC1FxUMz5EVNs9wqzq3vvPTfmWj6+qcj05U7XUMTDOcAWmvOSQ5DyUEqzdpXDh/HEl4xzaP38D bSgqAb8UcW0NYRAp+K+g==; Received: from localhost ([::1] helo=bombadil.infradead.org) by bombadil.infradead.org with esmtp (Exim 4.94.2 #2 (Red Hat Linux)) id 1oR614-007kEF-CR; Thu, 25 Aug 2022 06:06:59 +0000 Received: from mail-lj1-x231.google.com ([2a00:1450:4864:20::231]) by bombadil.infradead.org with esmtps (Exim 4.94.2 #2 (Red Hat Linux)) id 1oR60t-007kBO-DN for linux-arm-kernel@lists.infradead.org; Thu, 25 Aug 2022 06:06:49 +0000 Received: by mail-lj1-x231.google.com with SMTP id d9so2344638ljl.8 for ; Wed, 24 Aug 2022 23:06:45 -0700 (PDT) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=linaro.org; s=google; h=content-transfer-encoding:in-reply-to:from:references:cc:to :content-language:subject:user-agent:mime-version:date:message-id :from:to:cc; bh=olx/ATZA2V0ZzWbXgDfIkW1Q9WuG7O6K9lzCa3iTnVQ=; b=K7hkRlQGQAhrZwDVl0snou7VU3K79DIdp+9pBegNC7RbACR4+X7EJs3kPP14UHgsyE pxVZVQb1T42Zf6Bddds/dlIvsLtZmSWY7IjO8uY8tTAaoiEIXuO/ijNF6cC4EpIBQ0kb 3nwcDAjmh7ns4PtwRfudGTkQqhe4mFeUlSt4t5FoF1ONA5nb9NpSU1B+jmvwyAwN3Wig 3qBjzJRG0U4O/p3BR0XWhrcZhZL9P+XW50t44CW53Gmtw3RaRQbMJiXweTTpRcQMhpJr 6uXstAIrkvvtxIhT43ZHV0jnuP0ujcBthinSkEz94Nu0J2ocvdibhRNJRDLEQGKyIYYr fuIQ== X-Google-DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=1e100.net; s=20210112; h=content-transfer-encoding:in-reply-to:from:references:cc:to :content-language:subject:user-agent:mime-version:date:message-id :x-gm-message-state:from:to:cc; bh=olx/ATZA2V0ZzWbXgDfIkW1Q9WuG7O6K9lzCa3iTnVQ=; b=n3c0e4N+Mawb7OwdIOEY541jjfTysm/7TgMrEW0RPQQgQ7xZFllngcNGylOu4Ye1Bp qA5sPVVJ2rejLCPipWtx/YvXxX5My83Q9IWziFN8gJx5Ip4roNoGXBfwTN/39IO2/Fdp wzGxZuMB6agwGpahrCXjAoQk16f/9nAaRu+Q/3907466oXRPVYthi386m0U1f+Doc9C0 VP+MZF8DJLF/5cCQ25pByLC9eSyvavNDfdDCtnlrjqSgJZol1jsaaKoGnO+0DywAgHZh 5svdi2txorbriX9Wjq7/3S5rpAMmPhtPeIAsXWcX7J09Cz1ZE2qh7CN51gBTOmSpqpel bkZA== X-Gm-Message-State: ACgBeo1Be5OsMRV9xlWggN74U8blOy4ucrhCazCQ1fVRPoosPamPo9Uu qxK+eVsDzPZqH/B6+PHNwlYBTw== X-Google-Smtp-Source: AA6agR5QBRKXZanSA8gmZ93Z6gOnhbSvg9XlOxZ5KCEOkCJ1wttIow4xsE77cP3aGxYyw1o/Zm4siw== X-Received: by 2002:a05:651c:4d1:b0:261:c74b:ca6a with SMTP id e17-20020a05651c04d100b00261c74bca6amr609991lji.294.1661407604483; Wed, 24 Aug 2022 23:06:44 -0700 (PDT) Received: from [192.168.0.71] (82.131.98.15.cable.starman.ee. [82.131.98.15]) by smtp.gmail.com with ESMTPSA id s16-20020a056512215000b0048b17852938sm312759lfr.162.2022.08.24.23.06.43 (version=TLS1_3 cipher=TLS_AES_128_GCM_SHA256 bits=128/128); Wed, 24 Aug 2022 23:06:43 -0700 (PDT) Message-ID: Date: Thu, 25 Aug 2022 09:06:42 +0300 MIME-Version: 1.0 User-Agent: Mozilla/5.0 (X11; Linux x86_64; rv:91.0) Gecko/20100101 Thunderbird/91.12.0 Subject: Re: [PATCH 16/20] dt-bindings: memory: snps: Detach Zynq DDRC controller support Content-Language: en-US To: Serge Semin Cc: Serge Semin , Rob Herring , Michal Simek , Borislav Petkov , Mauro Carvalho Chehab , Tony Luck , Rob Herring , Manish Narani , Alexey Malahov , Michail Ivanov , Pavel Parkhomenko , Punnaiah Choudary Kalluri , Dinh Nguyen , James Morse , Robert Richter , Krzysztof Kozlowski , devicetree@vger.kernel.org, linux-arm-kernel@lists.infradead.org, linux-edac@vger.kernel.org, linux-kernel@vger.kernel.org References: <20220822190730.27277-1-Sergey.Semin@baikalelectronics.ru> <20220822190730.27277-17-Sergey.Semin@baikalelectronics.ru> <20220823083243.aovlgu22j7uv73qv@mobilestation> <166c0198-17c4-3b19-77fe-632d65f17cb0@linaro.org> <20220823114516.4mcufkbmzy5gjdcr@mobilestation> <6661dcc1-cc93-efbb-b248-0d93f681a1bf@linaro.org> <20220824172724.ny2xpryn76h6ftv6@mobilestation> From: Krzysztof Kozlowski In-Reply-To: <20220824172724.ny2xpryn76h6ftv6@mobilestation> X-CRM114-Version: 20100106-BlameMichelson ( TRE 0.8.0 (BSD) ) MR-646709E3 X-CRM114-CacheID: sfid-20220824_230647_546724_37F41F07 X-CRM114-Status: GOOD ( 25.80 ) X-BeenThere: linux-arm-kernel@lists.infradead.org X-Mailman-Version: 2.1.34 Precedence: list List-Id: List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , Content-Type: text/plain; charset="us-ascii" Content-Transfer-Encoding: 7bit Sender: "linux-arm-kernel" Errors-To: linux-arm-kernel-bounces+linux-arm-kernel=archiver.kernel.org@lists.infradead.org On 24/08/2022 20:27, Serge Semin wrote: > > Note what Rob said concerned the generic compatible "fallback" case, > not the generic compatible string in general. It's ok to have a > generic device name defined irrespective to the platform vendor. > Moreover it's applicable in case of the DW uMCTL2 DDRC IP-core since > first IP-core version is auto-detectable starting from v3.20a and > second I managed to implement auto-detection solutions for almost > all the DDR/ECC-specific parameters. So I am more inclined to the > solution 1) suggested by me in the previous email message: > - deprecate "snps,ddrc-3.80a" string. > - add new generic "snps,dw-umctl2-ddrc" compatible string. > - rename the DT-bindings file. Sounds ok. > >> >> Here the Linux driver also binds to generic synopsys compatible, so I >> would assume it has a meaning and use case on its own. > > Please see my messages above regarding the current Synopsys DW uMCTL2 > EDAC driver implementation. > >> >>> >>> What do you think? >>> >>> * Note I've got it you'd prefer the renaming being performed in a >>> separate patch. >> >> The rename could be in the split patch as here, but then I assume the >> rename part to be detected by git and be a pure rename. However: >> 1. The git did not mark it as rename (you might need to use custom >> arguments to -M/-B/-C), > > Of course git hasn't detected it as rename, because aside with renaming > I've split the bindings up. Splitting these two updates up into two > patches will give us what you said. So to speak I suggest the next > updates for v2: > PATCH X. Detach the Zynq A05 DDRC DT-bindings to a separate schema. > PATCH X + 1. Rename the Synopsys DW uMCTL2 DDRC bindings file and add a more > descriptive generic compatible string name. > > What do you think? Regardless of the split the rename can be and should be detected by Git. That's why we have these options. If it is not detected, you changed too much during rename, so it is not a rename anymore. Relatively small amount of changes would still be detected. > >> 2. There were also changes in the process (allOf:if:then). > > Right. But this is in another patchset. I'll address your notes in there. Best regards, Krzysztof _______________________________________________ linux-arm-kernel mailing list linux-arm-kernel@lists.infradead.org http://lists.infradead.org/mailman/listinfo/linux-arm-kernel