From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1760541AbcCEEqQ (ORCPT ); Fri, 4 Mar 2016 23:46:16 -0500 Received: from mailout3.samsung.com ([203.254.224.33]:35262 "EHLO mailout3.samsung.com" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1759908AbcCEEqN (ORCPT ); Fri, 4 Mar 2016 23:46:13 -0500 X-AuditID: cbfee68e-f793c6d00000136c-d9-56da649378e0 Subject: Re: [RESPIN 5/6] dt-bindings: EXYNOS: Describe SROMc configuration To: Rob Herring References: <1456389222-12738-1-git-send-email-pankaj.dubey@samsung.com> <1456389222-12738-6-git-send-email-pankaj.dubey@samsung.com> <20160302175713.GB11155@rob-hp-laptop> Cc: linux-samsung-soc@vger.kernel.org, linux-arm-kernel@lists.infradead.org, linux-kernel@vger.kernel.org, kgene.kim@samsung.com, k.kozlowski@samsung.com, thomas.ab@samsung.com, olof@lixom.net, p.fedin@samsung.com, devicetree@vger.kernel.org From: "pankaj.dubey" Message-id: <56DA64ED.2030908@samsung.com> Date: Sat, 05 Mar 2016 10:17:41 +0530 User-Agent: Mozilla/5.0 (X11; Linux x86_64; rv:38.0) Gecko/20100101 Thunderbird/38.6.0 MIME-version: 1.0 In-reply-to: <20160302175713.GB11155@rob-hp-laptop> Content-type: text/plain; charset=windows-1252 Content-transfer-encoding: 7bit X-Brightmail-Tracker: H4sIAAAAAAAAA+NgFlrHIsWRmVeSWpSXmKPExsWyRsSkXXdyyq0wg+s/OCzmHznHavH6haFF 74KrbBabHl9jtbi8aw6bxYzz+5gsTl3/zGbx6tIqNov/e3awW3QsY3Tg8ti0qpPNY/OSeo8r J5pYPfq2rGL0+LxJLoA1issmJTUnsyy1SN8ugSvjwZx9zAUPFSq2T85qYPwu0cXIySEhYCLx YO43VghbTOLCvfVsXYxcHEICKxgl5q08wQxT9OzYAkaIxFJGidOHrjBBON8ZJW7vfcvSxcjB ISzgLbH8VBRIg4iAosTvtmmsEDUbGCUaTq0Ha2AGaVhx4BjYWDYBfYnpj7exgdi8AloS6979 BLNZBFQl7jdeYgKxRQUiJJ7MPckIUSMo8WPyPRYQm1PAWGLl5seMIIuZBfQk7l/UAgkzC8hL bF7zlhlkl4TAS3aJx/9vsULMFJD4NvkQ2KESArISmw5AfSYpcXDFDZYJjGKzkGyYhTB1FpKp CxiZVzGKphYkFxQnpRcZ6RUn5haX5qXrJefnbmIERuPpf8/6djDePGB9iFGAg1GJh3fCk5th QqyJZcWVuYcYTYGOmMgsJZqcD4z5vJJ4Q2MzIwtTE1NjI3NLMyVx3gSpn8FCAumJJanZqakF qUXxRaU5qcWHGJk4OKUaGLsdSq7H5s4L4HlytTHz5l8zH65tZ1hmb5Ff8eNsYbtItXxD2qs/ /1WWZQnoOJif0WGbmL1CZ1ZksIFuM/exa7Fpjc7NpvMWnjud0Tw71GxKdebfrt71BYebK7ML /1w7Mr+sarqZPqdx5OJvf3ZEpP2aXfjlotdKLvcHE163TZ748sWb2NAfS5VYijMSDbWYi4oT AcoFm6HBAgAA X-Brightmail-Tracker: H4sIAAAAAAAAA+NgFvrNIsWRmVeSWpSXmKPExsVy+t9jAd3JKbfCDE40sljMP3KO1eL1C0OL 3gVX2Sw2Pb7GanF51xw2ixnn9zFZnLr+mc3i1aVVbBb/9+xgt+hYxujA5bFpVSebx+Yl9R5X TjSxevRtWcXo8XmTXABrVAOjTUZqYkpqkUJqXnJ+SmZeuq2Sd3C8c7ypmYGhrqGlhbmSQl5i bqqtkotPgK5bZg7QTUoKZYk5pUChgMTiYiV9O0wTQkPcdC1gGiN0fUOC4HqMDNBAwjrGjAdz 9jEXPFSo2D45q4Hxu0QXIyeHhICJxLNjCxghbDGJC/fWs3UxcnEICSxllDh96AoThPOdUeL2 3rcsXYwcHMIC3hLLT0WBNIgIKEr8bpvGClGzgVGi4dR6sAZmkIYVB44xg1SxCehLTH+8jQ3E 5hXQklj37ieYzSKgKnG/8RITiC0qECHxZO5JRogaQYkfk++xgNicAsYSKzc/ZgRZzCygJ3H/ ohZImFlAXmLzmrfMExgFZiHpmIVQNQtJ1QJG5lWMEqkFyQXFSem5Rnmp5XrFibnFpXnpesn5 uZsYwRH/THoH4+Fd7ocYBTgYlXh4Xzy7GSbEmlhWXJl7iFGCg1lJhHdt9K0wId6UxMqq1KL8 +KLSnNTiQ4ymQG9MZJYSTc4HJqO8knhDYxMzI0sjMwsjE3NzJXHex//XhQkJpCeWpGanphak FsH0MXFwSjUwMt1+u95P4YhsUlFNslBXZi+P8euUvO23NkVMZ5f53hR53Udlyv7DLdXGStwr xSzX7cl7u0968+pQu+Mcp/k6S1IKt4mvzOaadkbmsV34o4Tsmjj/i14z678LOlyNldL8veF9 ZPzdh/tKr5UcZ8vZ0sQrHDdhIZdlSAvvwSg5OZGrOsmvngsqsRRnJBpqMRcVJwIAjDIUdQ4D AAA= DLP-Filter: Pass X-MTR: 20000000000000000@CPGS X-CFilter-Loop: Reflected Sender: linux-kernel-owner@vger.kernel.org List-ID: X-Mailing-List: linux-kernel@vger.kernel.org Hi, On Wednesday 02 March 2016 11:27 PM, Rob Herring wrote: > On Thu, Feb 25, 2016 at 02:03:41PM +0530, Pankaj Dubey wrote: >> From: Pavel Fedin >> >> Add documentation for new subnode properties, allowing bank configuration. >> Based on u-boot implementation, but heavily reworked. >> >> Also, fix size of SROMc mapping in the example. > > Fix it in the previous patch. OK. > >> CC: devicetree@vger.kernel.org >> Signed-off-by: Pavel Fedin >> Signed-off-by: Pankaj Dubey >> Reviewed-by: Krzysztof Kozlowski >> Acked-by: Rob Herring >> Signed-off-by: Krzysztof Kozlowski >> --- >> .../bindings/memory-controllers/exynos-srom.txt | 73 +++++++++++++++++++++- >> 1 file changed, 71 insertions(+), 2 deletions(-) >> >> diff --git a/Documentation/devicetree/bindings/memory-controllers/exynos-srom.txt b/Documentation/devicetree/bindings/memory-controllers/exynos-srom.txt >> index 33886d5..e5c18df 100644 >> --- a/Documentation/devicetree/bindings/memory-controllers/exynos-srom.txt >> +++ b/Documentation/devicetree/bindings/memory-controllers/exynos-srom.txt >> @@ -5,8 +5,77 @@ Required properties: >> >> - reg: offset and length of the register set >> >> -Example: >> +Optional properties: >> +The SROM controller can be used to attach external peripherals. In this case >> +extra properties, describing the bus behind it, should be specified as below: >> + >> +- #address-cells: Must be set to 2 to allow device address translation. >> + Address is specified as (bank#, offset). >> + >> +- #size-cells: Must be set to 1 to allow device size passing >> + >> +- ranges: Must be set up to reflect the memory layout with four integer values >> + per bank: >> + 0 >> + >> +Sub-nodes: >> +The actual device nodes should be added as subnodes to the SROMc node. These >> +subnodes, except regular device specification, should contain the following > > s/except/in addition to/ > OK. Will update this description as suggested. >> +properties, describing configuration of the relevant SROM bank: >> + >> +Required properties: >> +- reg: bank number, base address (relative to start of the bank) and size of >> + the memory mapped for the device. Note that base address will be >> + typically 0 as this is the start of the bank. >> + >> +- samsung,srom-timing : array of 6 integers, specifying bank timings in the >> + following order: Tacp, Tcah, Tcoh, Tacc, Tcos, Tacs. >> + Each value is specified in cycles and has the following >> + meaning and valid range: >> + Tacp : Page mode access cycle at Page mode (0 - 15) >> + Tcah : Address holding time after CSn (0 - 15) >> + Tcoh : Chip selection hold on OEn (0 - 15) >> + Tacc : Access cycle (0 - 31, the actual time is N + 1) >> + Tcos : Chip selection set-up before OEn (0 - 15) >> + Tacs : Address set-up before CSn (0 - 15) >> + >> +Optional properties: >> +- reg-io-width : data width in bytes (1 or 2). If omitted, default of 1 is used. >> + >> +- samsung,srom-page-mode : page mode configuration for the bank: >> + 0 - normal (one data) >> + 1 - four data >> + If omitted, default of 0 is used. > > Make this a bool instead. > I do not have strong objections to change this, but I can see doing so will increase two or three lines in driver, as such this property is not being used as bool in driver. Sorry to say this but I do not understand why these comments are coming now? Whereas you had given your "Acked-by" to the same patch when it was posted previously by Pavel and we were keeping this driver under "drivers/soc/samsung". Is it just because we are moving to "drivers/memory" and it needs to be consistent with other memory controller drivers? Thanks, Pankaj Dubey > Rob > >