Received: by 2002:a25:c593:0:0:0:0:0 with SMTP id v141csp848229ybe; Fri, 6 Sep 2019 08:12:03 -0700 (PDT) X-Google-Smtp-Source: APXvYqxDr0HKxN4wT3zGUIMt2D1Ys7wR5Dhqo7/WY4FrmOhb3ri1MQ0HuGVuPgWJshT3sP2yREKb X-Received: by 2002:a17:90a:890c:: with SMTP id u12mr10154033pjn.124.1567782723060; Fri, 06 Sep 2019 08:12:03 -0700 (PDT) ARC-Seal: i=1; a=rsa-sha256; t=1567782723; cv=none; d=google.com; s=arc-20160816; b=btHbnwjp3Ym4DAMqAVMHaRuRVuZlmTgBdYbSyr5WRrBvP+doMWMm7jfaZy/zQQN04N eJLTurU1hXvX0+fHhQJV+HgOkMrshgqLJgVSatz/uTgeFJRspRhzgB558f5//VCIp9ak +6l0bvTfcQXRvXhlJMZjs0FBMea7qo6Bdj8m2gHGqNXH1wU/GDqZDnDyf5Epf8WrZtxp 1QUi/tZJ3JDWtZrQcgnFeH3TVl/eeyHEk+kv2W09Y06ZFDdI1b+CmKDtV01H6GVfgOxw TEOqdwcS4HRUfUBFbayh3S65EPtgFN4dRWQVIU/0nfNSrfsw16Ku6MZvD8K/JDpICxqG O8JA== ARC-Message-Signature: i=1; a=rsa-sha256; c=relaxed/relaxed; d=google.com; s=arc-20160816; h=list-id:precedence:sender:references:cms-type :content-transfer-encoding:content-language:in-reply-to:mime-version :user-agent:date:message-id:from:cc:to:subject:dkim-signature :dkim-filter; bh=epZ2/nn61cU86rBoZ+pM3kR6r5TNU/gcgmeJYFybZcI=; b=Ay508qZUlIahXxIx4cwzk1hjIo1giXRYDBTKFEQdPSWvaEn1Gr91GhAMNPkwKwyCBX JU9BhFVstlEv+V8TV6WdeLUMImAu093+8j3PDVxFzOcsytALjbQmYLVsxDEsqm1fb4WN RPjcCusl/lUJXic6/GIDbTl4HgR7mm4Ol8w3a9Xxf8l8fpeOvCQMBjPr0N/xPEO+YK9x jFhK+ug8rxAQMHkq9g5CTujxkEiWgfvGrqP3ucZACVtXlqvD6ekCpqCiTFEtqAYJpeSa s8uuDYuCNGjM2IiBfE8yYsTtQAxzOMOLZj2Alq5XQSFjYOXbq6Y28BRGJdRk8eWTgfJv YVaQ== ARC-Authentication-Results: i=1; mx.google.com; dkim=pass header.i=@samsung.com header.s=mail20170921 header.b=gJBn6caS; spf=pass (google.com: best guess record for domain of linux-kernel-owner@vger.kernel.org designates 209.132.180.67 as permitted sender) smtp.mailfrom=linux-kernel-owner@vger.kernel.org; dmarc=pass (p=NONE sp=NONE dis=NONE) header.from=samsung.com Return-Path: Received: from vger.kernel.org (vger.kernel.org. [209.132.180.67]) by mx.google.com with ESMTP id f2si4449367pgl.74.2019.09.06.08.11.45; Fri, 06 Sep 2019 08:12:03 -0700 (PDT) Received-SPF: pass (google.com: best guess record for domain of linux-kernel-owner@vger.kernel.org designates 209.132.180.67 as permitted sender) client-ip=209.132.180.67; Authentication-Results: mx.google.com; dkim=pass header.i=@samsung.com header.s=mail20170921 header.b=gJBn6caS; spf=pass (google.com: best guess record for domain of linux-kernel-owner@vger.kernel.org designates 209.132.180.67 as permitted sender) smtp.mailfrom=linux-kernel-owner@vger.kernel.org; dmarc=pass (p=NONE sp=NONE dis=NONE) header.from=samsung.com Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S2390542AbfIFLjQ (ORCPT + 99 others); Fri, 6 Sep 2019 07:39:16 -0400 Received: from mailout2.w1.samsung.com ([210.118.77.12]:35090 "EHLO mailout2.w1.samsung.com" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S2390457AbfIFLjQ (ORCPT ); Fri, 6 Sep 2019 07:39:16 -0400 Received: from eucas1p2.samsung.com (unknown [182.198.249.207]) by mailout2.w1.samsung.com (KnoxPortal) with ESMTP id 20190906113913euoutp0238a78c1e3a288d764bb9f4a4b62537d7~B1ndsZrjV2466624666euoutp02v for ; Fri, 6 Sep 2019 11:39:13 +0000 (GMT) DKIM-Filter: OpenDKIM Filter v2.11.0 mailout2.w1.samsung.com 20190906113913euoutp0238a78c1e3a288d764bb9f4a4b62537d7~B1ndsZrjV2466624666euoutp02v DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=samsung.com; s=mail20170921; t=1567769953; bh=epZ2/nn61cU86rBoZ+pM3kR6r5TNU/gcgmeJYFybZcI=; h=Subject:To:Cc:From:Date:In-Reply-To:References:From; b=gJBn6caSNHZViW8Q+RQ60DEZY8LUZF+rwFH8aLr8GOm7Kvp3hAzQcJGXxFxYBfWqK hmjVV/WZsWxmxT/w7GPDW4TRLWAnkxo1w590a0jXRfL76IjWFAdRLTKJJtStoLPlrT bPUxUCU1hIuztzb49hCcystDJBH2wYPyHgtfl41U= Received: from eusmges2new.samsung.com (unknown [203.254.199.244]) by eucas1p1.samsung.com (KnoxPortal) with ESMTP id 20190906113912eucas1p15a6d144b485f3d7f7e6ef803aebba111~B1nc2DRO72600726007eucas1p1P; Fri, 6 Sep 2019 11:39:12 +0000 (GMT) Received: from eucas1p2.samsung.com ( [182.198.249.207]) by eusmges2new.samsung.com (EUCPMTA) with SMTP id F5.BD.04309.065427D5; Fri, 6 Sep 2019 12:39:12 +0100 (BST) Received: from eusmtrp2.samsung.com (unknown [182.198.249.139]) by eucas1p2.samsung.com (KnoxPortal) with ESMTPA id 20190906113911eucas1p25b42a5f4d3846ca4eb676dc5f6067c68~B1ncBI_rh2032920329eucas1p2v; Fri, 6 Sep 2019 11:39:11 +0000 (GMT) Received: from eusmgms2.samsung.com (unknown [182.198.249.180]) by eusmtrp2.samsung.com (KnoxPortal) with ESMTP id 20190906113911eusmtrp2df4582dc73d9c4f1f222973a160704dd~B1nby5MDL1602816028eusmtrp2P; Fri, 6 Sep 2019 11:39:11 +0000 (GMT) X-AuditID: cbfec7f4-afbff700000010d5-d0-5d724560316e Received: from eusmtip1.samsung.com ( [203.254.199.221]) by eusmgms2.samsung.com (EUCPMTA) with SMTP id D5.AC.04117.F55427D5; Fri, 6 Sep 2019 12:39:11 +0100 (BST) Received: from [106.120.51.20] (unknown [106.120.51.20]) by eusmtip1.samsung.com (KnoxPortal) with ESMTPA id 20190906113910eusmtip177b7d502ecf9cac3a446211bb2bf282c~B1nbCck9m1696716967eusmtip1S; Fri, 6 Sep 2019 11:39:10 +0000 (GMT) Subject: Re: [PATCH 3/3] dt-bindings: ddr: Add bindings for Samsung LPDDR3 memories To: Krzysztof Kozlowski Cc: devicetree@vger.kernel.org, "linux-kernel@vger.kernel.org" , linux-pm@vger.kernel.org, "linux-samsung-soc@vger.kernel.org" , linux-arm-kernel@lists.infradead.org, =?UTF-8?Q?Bart=c5=82omiej_=c5=bbo=c5=82nierkiewicz?= , kgene@kernel.org, mark.rutland@arm.com, robh+dt@kernel.org, Chanwoo Choi , kyungmin.park@samsung.com, Marek Szyprowski , s.nawrocki@samsung.com, myungjoo.ham@samsung.com, willy.mh.wolff.ml@gmail.com From: Lukasz Luba Message-ID: <52963d0d-cf48-7085-5581-a94c6e15e0bd@partner.samsung.com> Date: Fri, 6 Sep 2019 13:39:09 +0200 User-Agent: Mozilla/5.0 (X11; Linux x86_64; rv:60.0) Gecko/20100101 Thunderbird/60.8.0 MIME-Version: 1.0 In-Reply-To: Content-Language: en-US Content-Transfer-Encoding: 7bit X-Brightmail-Tracker: H4sIAAAAAAAAA02Se0hTYRjG+3Z2Lo5mX9Pam4nC8I+K1C6CH2VWdGFUdiGCqLRWHVRyajvO LgapkNZqZVFmw64khWjznq6M2pYXrKYILdIutkKwVplWStrydIz87/c97/PyPg98HKVy00Fc cmoGb0jVpWgYhbyuadgVvmuVIX5e9+tQUllkpYl7sJcmV53PaHLG85EiLlcFS57mfmJJlec5 TTptxQwZMDsRKXI9kJFy5yuWlLg7ZKQr5zZDjjU6WeL4lE+THy3v0DKsLbtShrQNllestqr0 BKOtvnlUe7qmFGkHqkI2MtsUMXv5lORM3hAZu0uRNGz1oPSKgIO55RUoG3VPMSE/DnAUPLKa WBNScCp8G0F15RAjPQYR5F+3M6JLhQcQ+C4E/tu40/uBkky3ENz4ViuXTF4EQ/eiRQ7AW+DJ yV+syIF4NrhHf9LiAoXtciiyDctMiOMYHAH1pftFjxKvhjedjUiU5TgMvC2JojwNb4Vvbx20 ZJkKrZfe/z3lhzeBvcT1NxuF1fDy/VWZxKFw11tMSTlzOOgxb5Z4JXS1FzMSB0Bfcw0rcTD4 GqRdwAJkm68jiY+A58zlcc9icDR30GI0aqyK1RYpycvBZznHiDJgf3jhnSol8IdzdRcpSVbC 8TyV5J4FNafaxw9Nh1tlhWwB0lgm9LJM6GKZ0MXy/+41JC9Fat4o6BN5YUEqfyBC0OkFY2pi xJ40fRUa+25tv5sH65FtZLcdYQ5pJivJCkO8itZlCof0dgQcpQlUWivT41XKvbpDh3lD2k6D MYUX7GgmJ9eolVmT3m5X4URdBr+P59N5w7+pjPMLykYFF9Iim6CBqo0564939IcYff2ThlpW tWdzPVl8YZ/iB/+wIDjO9rNVF5fpkJnPF35P5ksG1M4lyYuSfCmtHev6GkfC63vWL36w1F2e h0LCtoQnxAZ9iZusH124L3ptwPnor59dm9oeU4sScqNmhMo01H3VbnbDmqye3Mw5I/FzNXIh STd/DmUQdH8Aj7eSLWoDAAA= X-Brightmail-Tracker: H4sIAAAAAAAAA+NgFnrLIsWRmVeSWpSXmKPExsVy+t/xu7rxrkWxBs3/VC02zljPanH9y3NW i/lHzrFa9D9+zWxx/vwGdouzTW/YLTY9vsZqcXnXHDaLz71HGC1mnN/HZLH2yF12i6XXLzJZ 3G5cwWbRuvcIu8XhN+2sFt9OPGJ0EPBYM28No8fOWXfZPTat6mTz2Lyk3qNvyypGj8+b5ALY ovRsivJLS1IVMvKLS2yVog0tjPQMLS30jEws9QyNzWOtjEyV9O1sUlJzMstSi/TtEvQyfq5/ zFiwQbiiae0GxgbGO/xdjJwcEgImEuueP2XuYuTiEBJYyiixYkMzO0RCTGLSvu1QtrDEn2td bBBFrxklfp96zwaSEBYIlTjT/RusSERAU+L63++sIEXMAodYJD5tPc4IkhAS6GOS6H2k38XI wcEmoCexY1UhSJhXwE3i/uW9jCBhFgEVibcn0kHCogIREod3zGKEKBGUODnzCQuIzSkQKHFo 6XmwtcwCZhLzNj9khrDFJW49mc8EYctLbH87h3kCo9AsJO2zkLTMQtIyC0nLAkaWVYwiqaXF uem5xUZ6xYm5xaV56XrJ+bmbGIFRve3Yzy07GLveBR9iFOBgVOLhfeBQFCvEmlhWXJl7iFGC g1lJhHf9xoJYId6UxMqq1KL8+KLSnNTiQ4ymQL9NZJYSTc4HJpy8knhDU0NzC0tDc2NzYzML JXHeDoGDMUIC6YklqdmpqQWpRTB9TBycUg2MZzmnpBhNFl7KHljM+f/laq8XYrGFoX+NxdJN vRt4TjGL+W597/Xm3rpQkd63504b3BS0ZHW+03a9+bG2gqXihy3r13z/c3VzX9fPEk6h5P9n /7y5ad4469rfdzdtz2YstFMQ3sQ9e96zHzm5KqaL9stWRJfLvjG8MXGr69IraQtTAuSsmM0L lViKMxINtZiLihMBVcSENwADAAA= X-CMS-MailID: 20190906113911eucas1p25b42a5f4d3846ca4eb676dc5f6067c68 X-Msg-Generator: CA Content-Type: text/plain; charset="utf-8" X-RootMTR: 20190906101407eucas1p15eb0df53374b27497b4793eab24becf6 X-EPHeader: CA CMS-TYPE: 201P X-CMS-RootMailID: 20190906101407eucas1p15eb0df53374b27497b4793eab24becf6 References: <20190906101344.3535-1-l.luba@partner.samsung.com> <20190906101344.3535-4-l.luba@partner.samsung.com> Sender: linux-kernel-owner@vger.kernel.org Precedence: bulk List-ID: X-Mailing-List: linux-kernel@vger.kernel.org Hi Krzysztof, On 9/6/19 12:56 PM, Krzysztof Kozlowski wrote: > On Fri, 6 Sep 2019 at 12:14, Lukasz Luba wrote: >> >> Add description of bindings for Samsung k3qf2f20db LPDDR3 memory. >> Minor fixes in the old documentation. >> >> Signed-off-by: Lukasz Luba >> --- >> .../devicetree/bindings/ddr/lpddr3.txt | 29 +++++++++++++++++-- >> 1 file changed, 27 insertions(+), 2 deletions(-) >> >> diff --git a/Documentation/devicetree/bindings/ddr/lpddr3.txt b/Documentation/devicetree/bindings/ddr/lpddr3.txt >> index 3b2485b84b3f..de0905239767 100644 >> --- a/Documentation/devicetree/bindings/ddr/lpddr3.txt >> +++ b/Documentation/devicetree/bindings/ddr/lpddr3.txt >> @@ -40,10 +40,34 @@ Child nodes: >> a given speed-bin. Please see Documentation/devicetree/ >> bindings/ddr/lpddr3-timings.txt for more information on "lpddr3-timings" >> >> +Samsung K3QF2F20DB LPDDR3 memory >> +------------------------------------------------------------ >> + >> +This binding uses the LPDDR3 binding (described above) >> + >> +Required properties: >> +- compatible: Should be: >> + "samsung,K3QF2F20DB" >> + followed by "jedec,lpddr3" >> +- density : representing density in Mb (Mega bits) >> +- io-width : representing bus width. Possible value 32 >> +- #address-cells: Must be set to 1 >> +- #size-cells: Must be set to 0 > > If you decided to repeat all properties again, then it deserves its > own bindings file. However I though about simpler solution - just > document compatible. Exactly the same as AT24 or AT25 EEPROM bindings. > There is not much benefit from copying all these properties. OK, I see. I will add only 'compatible' and skip the rest then. So the lpddr3.txt file will get this addition: +Samsung K3QF2F20DB LPDDR3 memory +------------------------------------------------------------ + +This binding uses the LPDDR3 binding (described above) + +Required properties: +- compatible: Should be: + "samsung,K3QF2F20DB" + followed by "jedec,lpddr3" + +Optional properties: + +The optional properties are the same as in the LPDDR3 generic bindings and +values should be taken from the data-sheet. Detailed bindings are described +above. + +Child nodes: + +Detailed bindings are described in LPDDR3 generic bindings described above. + Is it OK? Regards, Lukasz