From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1752839AbbJENSz (ORCPT ); Mon, 5 Oct 2015 09:18:55 -0400 Received: from mailout1.w1.samsung.com ([210.118.77.11]:28665 "EHLO mailout1.w1.samsung.com" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1751831AbbJENSy (ORCPT ); Mon, 5 Oct 2015 09:18:54 -0400 X-AuditID: cbfec7f5-f794b6d000001495-aa-561278ba8385 From: Pavel Fedin To: "'Krzysztof Kozlowski'" , "'Pankaj Dubey'" , linux-samsung-soc@vger.kernel.org, linux-kernel@vger.kernel.org, linux-arm-kernel@lists.infradead.org Cc: k.kozlowski.k@gmail.com, thomas.ab@samsung.com, kgene@kernel.org, heiko@sntech.de References: <1440403348-8974-1-git-send-email-pankaj.dubey@samsung.com> <018501d0ff62$1650b410$42f21c30$@samsung.com> <5612719E.60006@samsung.com> In-reply-to: <5612719E.60006@samsung.com> Subject: RE: [PATCH v2 0/7] Add support for Exynos SROM Controller driver Date: Mon, 05 Oct 2015 16:18:46 +0300 Message-id: <01cb01d0ff70$6027b760$20772620$@samsung.com> MIME-version: 1.0 Content-type: text/plain; charset=us-ascii Content-transfer-encoding: 7bit X-Mailer: Microsoft Outlook 14.0 Thread-index: AQG13x99tKvb5gCon/JqZ+EwhdtfMADYfSp5AXEy85OegNKv0A== Content-language: ru X-Brightmail-Tracker: H4sIAAAAAAAAA+NgFupmkeLIzCtJLcpLzFFi42I5/e/4Fd1dFUJhBrO/WVv8f/Sa1eL5vx/s Fq9fGFr0P37NbLHp8TVWi8u75rBZzDi/j8li0dYv7BYdyxgdOD12zrrL7rFpVSebx+Yl9R59 W1Yxemy/No/Z4/MmuQC2KC6blNSczLLUIn27BK6MX9dOMRe84a2Y+mQ6ewPjLu4uRk4OCQET iZ6nm1ggbDGJC/fWs3UxcnEICSxllDix7DArhPOdUWJP2282kCo2AXWJ018/sIAkRAROMkrM fb6NCSTBLBAusX3aO2aIjimMEnderwFq5+DgFNCU+Hm4FMQUFvCUWHA3AqScRUBVYvKKTmaQ MK+ApcTXfeogYV4BQYkfk++xQEzUkli/8zjUdHmJzWveMkMcqiCx4+xrRhBbRMBJYk3LVKga EYlp/+4xT2AUmoVk1Cwko2YhGTULScsCRpZVjKKppckFxUnpuUZ6xYm5xaV56XrJ+bmbGCFR 9HUH49JjVocYBTgYlXh4D8QLhgmxJpYVV+YeYpTgYFYS4bV0FQoT4k1JrKxKLcqPLyrNSS0+ xCjNwaIkzjtz1/sQIYH0xJLU7NTUgtQimCwTB6dUA+PsaB3W1xGHm/YlvXYVu/Nl14Uf3nln OU96T69sCRPd4fqxrunaRoXXOhNYeS6KKUXcq+q5NoslnklN1/fq9RY59xBTYYY49q6/8zmm 39rfojxpjU30hKyfCU3l52TX2+yWDj1lZ/vExvlZV3F2js7BAyqTp30XKpxhbLJxNpcZd/JG Tx3T9UosxRmJhlrMRcWJAHifgmOeAgAA Sender: linux-kernel-owner@vger.kernel.org List-ID: X-Mailing-List: linux-kernel@vger.kernel.org Hello! > That is interesting. What do you mean by that? Although SROM controller > can access external memory (SRAM, ROM) but it is not exactly a bus. How > is the SMSC9111 "connected"? What interface or protocol is used? SROM controller actually provides a demultiplexed external address and data bus. This way you could hook up old good ROM chip to it (hence its name). However, you can plug in there anything else with the appropriate interface, and SMSC9111 uses exactly that. Just some address + control + data wires. Therefore, you can think of SROMc as a simple bus. > I am sorry but I cannot figure out what exactly are you talking about. > The Device Tree is the glue to describe the board, the hardware and its > configuration. Please elaborate a little more. Yes, exactly that. SROM controller has 4 outputs, and for every of these outputs you have to choose mode and timings. Once you set them correctly, you can use peripherial behind them. These timings actually depend on the actual peripherial. By this time i remembered that u-boot guys also did something with this. And here are links: http://git.denx.de/?p=u-boot.git;a=blob;f=arch/arm/dts/exynos5250-smdk5250.dts;h=8b695442b1a29dec4e3 53ebd3cfee0ff89f218a6;hb=HEAD#l43 - this is how connection is described http://git.denx.de/?p=u-boot.git;a=blob;f=board/samsung/common/board.c;h=d32c75de50858f131284e28db0c 7ed55883f1cb0;hb=HEAD#l184 - and this is parsing code There's only one small problem with their code. "bank" parameter is device-wide, therefore you cannot configure more than one bank. For SMDK boards it's OK, but in future you may run into problem with this. Kind regards, Pavel Fedin Expert Engineer Samsung Electronics Research center Russia