From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1755389AbbFQMTd (ORCPT ); Wed, 17 Jun 2015 08:19:33 -0400 Received: from atrey.karlin.mff.cuni.cz ([195.113.26.193]:36033 "EHLO atrey.karlin.mff.cuni.cz" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1754650AbbFQMTX (ORCPT ); Wed, 17 Jun 2015 08:19:23 -0400 Date: Wed, 17 Jun 2015 14:19:20 +0200 From: Pavel Machek To: Marek Vasut Cc: Krzysztof Kozlowski , Michal Suchanek , Rob Herring , Pawel Moll , Mark Rutland , Ian Campbell , Kumar Gala , Russell King , Kukjin Kim , Vinod Koul , Dan Williams , David Woodhouse , Brian Norris , Han Xu , Mark Brown , Geert Uytterhoeven , =?utf-8?B?UmFmYcWCIE1pxYJlY2tp?= , Alison Chaiken , Huang Shijie , Ben Hutchings , Knut Wohlrab , Bean Huo =?utf-8?B?6ZyN5paM5paMIChiZWFuaHVvKQ==?= , "grmoore@altera.com" , devicetree@vger.kernel.org, linux-kernel@vger.kernel.org, linux-arm-kernel@lists.infradead.org, linux-samsung-soc@vger.kernel.org, dmaengine@vger.kernel.org, linux-mtd@lists.infradead.org, linux-spi@vger.kernel.org Subject: Re: [PATCH 11/11] dt: Exynos: add Snow SPI NOR node. Message-ID: <20150617121920.GB20728@amd> References: <3234c12c90eabbeeb6d33604a324231937e309ec.1433364398.git.hramrach@gmail.com> <556FB222.2070406@samsung.com> <201506041720.54666.marex@denx.de> MIME-Version: 1.0 Content-Type: text/plain; charset=us-ascii Content-Disposition: inline In-Reply-To: <201506041720.54666.marex@denx.de> User-Agent: Mutt/1.5.23 (2014-03-12) Sender: linux-kernel-owner@vger.kernel.org List-ID: X-Mailing-List: linux-kernel@vger.kernel.org On Thu 2015-06-04 17:20:54, Marek Vasut wrote: > On Thursday, June 04, 2015 at 04:04:18 AM, Krzysztof Kozlowski wrote: > > On 04.06.2015 06:26, Michal Suchanek wrote: > > > The Snow has onboard SPI NOR flash which contains the bootloader. > > > > > > Add DT node for this flash chip. The flash is rated 133MHz but the pl330 > > > controller can transfer only up to 128 bytes at this speed so use more > > > conservative settings. Even at 40MHz pl330 can transfer at most 64k with > > > the current driver. > > > > > > Signed-off-by: Michal Suchanek > > > --- > > > > > > arch/arm/boot/dts/exynos5250-snow.dts | 12 ++++++++++++ > > > 1 file changed, 12 insertions(+) > > > > > > diff --git a/arch/arm/boot/dts/exynos5250-snow.dts > > > b/arch/arm/boot/dts/exynos5250-snow.dts index 1fa72cf..38e4cda 100644 > > > --- a/arch/arm/boot/dts/exynos5250-snow.dts > > > +++ b/arch/arm/boot/dts/exynos5250-snow.dts > > > @@ -691,6 +691,18 @@ > > > > > > num-cs = <1>; > > > cs-gpios = <&gpa2 5 0>; > > > status = "okay"; > > > > > > + flash: m25p80@0 { > > > > The indentation looks odd. This should be at the same level as "status". > > > > > + #address-cells = <1>; > > > + #size-cells = <1>; > > > + compatible = "jedec,spi-nor"; > > > + reg = <0>; > > > + spi-max-frequency = <40000000>; > > > > So actually you wanted 133 MHz but as a workaround for DMA issue you use > > 40 MHz, right? Could you add here a small TODO note in comment about it? > > I disagree. There is a problem, the problem should actually be analyzed and > fixed, not just postponed with some TODO nonsense. Of course, that would be preferable if submitter has time. But OTOH slow SPI is probably better than no SPI at all, and debugging this can be nasty. I think that TODO is suitable. Pavel -- (english) http://www.livejournal.com/~pavelmachek (cesky, pictures) http://atrey.karlin.mff.cuni.cz/~pavel/picture/horses/blog.html