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 E8F86C32772 for ; Tue, 23 Aug 2022 11:46:30 +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:MIME-Version:References: Message-ID:Subject:Cc:To:From:Date:Reply-To:Content-ID:Content-Description: Resent-Date:Resent-From:Resent-Sender:Resent-To:Resent-Cc:Resent-Message-ID: List-Owner; bh=5FcAvkd2rppwOWabUQZD+AwmYSeOCmy0OBlf+AjQu9g=; b=SGvViewftYP8sY H2n1QYkCd5kRFkb+CMMqpihDRobhZkGh9oUatAk9XLepSbDn6IbTZ2d4JCoCTnQmKcfqOsVIK6vYe YJS9M/b+T12hW2YeAPJJUfHewbJVUBlT7Xu6bTVq2nvXJejwvFQtKbKJsnwEOFeg0/5hLWuxm7/93 ygqW4pONWTSD8+nM8uzX1eU5YDdqFDj+1ulBm48H2KlN/k3ulAwtKHr4I9dxdkf3L4YX4sbDBoR4z +PmGRVTZo4mex4xnXkklGoWj1sPi4WRVrbGr15zTrbEyvzQtkFJa5/ri6iOA1EW9lr19i969ECNxw LFEhgtf5CGRFFm1yyEdg==; Received: from localhost ([::1] helo=bombadil.infradead.org) by bombadil.infradead.org with esmtp (Exim 4.94.2 #2 (Red Hat Linux)) id 1oQSLW-005VvU-Pi; Tue, 23 Aug 2022 11:45:26 +0000 Received: from mail-lf1-x131.google.com ([2a00:1450:4864:20::131]) by bombadil.infradead.org with esmtps (Exim 4.94.2 #2 (Red Hat Linux)) id 1oQSLT-005VtO-Kn for linux-arm-kernel@lists.infradead.org; Tue, 23 Aug 2022 11:45:26 +0000 Received: by mail-lf1-x131.google.com with SMTP id s1so16512983lfp.6 for ; Tue, 23 Aug 2022 04:45:21 -0700 (PDT) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=gmail.com; s=20210112; h=in-reply-to:content-disposition:mime-version:references:message-id :subject:cc:to:from:date:from:to:cc; bh=9DFi8GMavWCQGHl6/b/DfpLAu8GPfBlHk4CuJF22IMY=; b=UKCmuVB+XW4lw2kUy4PfUbWfaQWfujDghz5LxBd0za58bJ6PIhTEhS7R5R1uukm3hl AziPWSsrq88P5KAGGRkLOfbahVbEDG93ff7xayS1kWQGL4/dadXylHN5WLG+miNpYPRm 9W/zhxJ4oaw+fFSGeyot1ncmdHtuCVgb8VxtxqNU+tPOjYsNmZu/xoYMoxwdMK8Xay4q r6v2P6fgy+cQGBwRRNA4CxhyAw9jd1OM0xdpM2RfAgGGztob3qor7bd/Mk3IY0doeqcM lc4+WY3tGrY53hvXzvip9MTQomIzCmv+FxHEfWqqLTPUZLoENGiyl15QTlJ6gv3sUyJf olKg== X-Google-DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=1e100.net; s=20210112; h=in-reply-to:content-disposition:mime-version:references:message-id :subject:cc:to:from:date:x-gm-message-state:from:to:cc; bh=9DFi8GMavWCQGHl6/b/DfpLAu8GPfBlHk4CuJF22IMY=; b=yNT0VFRlAtGC5mYYdoumOXgYmp/tANXm6jzIzHerrOOTD7G5oSvkxjuMC5S5eZMXLm jDscr4ePgQj/Y5vD7LCBBJAdm28LOy8MS7PdfvDOjjPEB2WR9bO+w0n0OT7Vj6ygPvMw 0bvnF5nWJSQIy16+PFGZee5hxoUew9CcRZnoDTpvKNDO/QZaqWq0fX+WEn/YZUVWga/G YB+HfBDaf2TpQIwhWedjgn2IxItF3PEeF7jC0m2JIpXZqBCG5T0eyZ3byEGotjHTplnS viDxB0eCj8ywgkC7SSwBfZr5O07dWfYi43BdMOL1L0pkEtFJucmVRjJKKgfodEIShOm+ jD6g== X-Gm-Message-State: ACgBeo1x6wHIdm7K/w7x+B2yujMcKrFJxouuryRYaFpf0sBTDTYF8a53 y0byo2PmOIKjZbiIZ0Lrgt0= X-Google-Smtp-Source: AA6agR5OnjnsYrTSUwMVA6+5CrajZGLUzWP+V59b0pBkkaaZWn8k3q650yb2pDZT7GNHJgbPFj/eHQ== X-Received: by 2002:a05:6512:3d1f:b0:48a:f6d5:644e with SMTP id d31-20020a0565123d1f00b0048af6d5644emr8131478lfv.589.1661255119618; Tue, 23 Aug 2022 04:45:19 -0700 (PDT) Received: from mobilestation ([95.79.140.178]) by smtp.gmail.com with ESMTPSA id x2-20020ac25dc2000000b0047dace7c7e5sm2456808lfq.212.2022.08.23.04.45.17 (version=TLS1_3 cipher=TLS_AES_256_GCM_SHA384 bits=256/256); Tue, 23 Aug 2022 04:45:18 -0700 (PDT) Date: Tue, 23 Aug 2022 14:45:16 +0300 From: Serge Semin To: Krzysztof Kozlowski 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 Subject: Re: [PATCH 16/20] dt-bindings: memory: snps: Detach Zynq DDRC controller support Message-ID: <20220823114516.4mcufkbmzy5gjdcr@mobilestation> 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> MIME-Version: 1.0 Content-Disposition: inline In-Reply-To: <166c0198-17c4-3b19-77fe-632d65f17cb0@linaro.org> X-CRM114-Version: 20100106-BlameMichelson ( TRE 0.8.0 (BSD) ) MR-646709E3 X-CRM114-CacheID: sfid-20220823_044524_342443_2A406F9C X-CRM114-Status: GOOD ( 34.66 ) 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 Tue, Aug 23, 2022 at 11:44:16AM +0300, Krzysztof Kozlowski wrote: > On 23/08/2022 11:32, Serge Semin wrote: > > On Tue, Aug 23, 2022 at 11:17:23AM +0300, Krzysztof Kozlowski wrote: > >> On 22/08/2022 22:07, Serge Semin wrote: > >>> The Zynq A05 DDRC controller has nothing in common with DW uMCTL2 DDRC: > >>> the CSRs layout is absolutely different and it doesn't has IRQ unlike DW > >>> uMCTL2 DDR controller of all versions (v1.x, v2.x and v3.x). Thus there is > >>> no any reason to have these controllers described by the same bindings. > >>> Thus let's split them up. > >>> > >>> While at it rename the original Synopsys uMCTL2 DT-schema file to a more > >>> descriptive - snps,dw-umctl2-ddrc.yaml and add a more detailed title and > >>> description of the device bindings. > >> > > > >> Filename should be based on compatible, so if renaming then > >> snps,ddrc-3.80a.yaml or snps,ddrc.yaml... which leads to original > >> filename anyway. Therefore nack for rename. Original name was synopsys,ddrc-ecc.yaml which doesn't match any of the compatible strings. > > > > New requirement? I've submitted not a single patch to the DT-bindings > > sources and didn't get any comment from Rob about that. > > This is not a new requirement. It has been since some time and Rob gave > such reviews. > > https://lore.kernel.org/linux-devicetree/YlhkwvGdcf4ozTzG@robh.at.kernel.org/ April 2022. So it's new. It would be nice to have it defined somewhere in docs (writing-bindings.rst?). So does the compatibles order (this was surprising to me too). > > For devices with multiple compatibles that's a bit tricky, but assuming > the bindings describe both original design from Synopsys and it's > implementations, then something closer to Synopsys makes sense. The closest name would be snps,dw-umctl2-ddrc.yaml. snps,ddrc is too generic especially for the IP-cores vendor. It doesn't have a reference to the actual IP-core the device in subject is based on. > > > > In addition > > There are DT bindings with names different from what is defined in the > > compatible name. Moreover there are tons of bindings with various > > compatible names. What name to choose then? Finally the current name > > is too generic to use for actual DW uMCTL2 DDRC controller. > > There are thousands of bugs, inconsistencies, naming differences in > kernel. I don't find these as arguments to repeat the practice...so the > bindings file name should be based on the compatible. Did I ask for an exception? I justified why the renaming was necessary. You said it goes against the practice of having the DT-schema named as the device compatible strings and just nacked. But above in this message you said > "assuming the bindings describe both original design from Synopsys > and it's implementations, then something closer to Synopsys makes sense" What I suggest makes more sense than some abstract Synopsys DDRC, which may refer to a Synopsys DDR controller other than the subject one. So I see two solutions here: 1. Adding a new generic compatible string like "snps,dw-umctl2-ddrc" and deprecate the "snps,ddrc-3.80a". It gets to be even more justified seeing the Synopsys IP-core version has been exported in the device CSRs since IP-core v3.20a. So having the version attached to the compatible string was absolutely redundant. 2. Just deprecate the generic compatible string, the new compatible devices will be supposed to use a vendor-specific compatible strings, but still rename the DT-bindings file. This makes sense since the current generic name isn't quiet well structured. It' prefix-part is too generic and at the same time it refers to a device reversion for no much reason. What do you think? * Note I've got it you'd prefer the renaming being performed in a separate patch. -Sergey > > Best regards, > Krzysztof _______________________________________________ linux-arm-kernel mailing list linux-arm-kernel@lists.infradead.org http://lists.infradead.org/mailman/listinfo/linux-arm-kernel