From mboxrd@z Thu Jan 1 00:00:00 1970 From: Marek Vasut To: Michal Suchanek Subject: Re: [PATCH 00/11] Enable access to SPI NOR flash on Samsung Snow board Date: Thu, 4 Jun 2015 17:29:35 +0200 References: <201506040053.29482.marex@denx.de> In-Reply-To: MIME-Version: 1.0 Content-Type: Text/Plain; charset="us-ascii" Content-Transfer-Encoding: 7bit Message-Id: <201506041729.35645.marex@denx.de> Cc: Mark Rutland , Krzysztof Kozlowski , Geert Uytterhoeven , linux-mtd@lists.infradead.org, Alison Chaiken , "Bean Huo =?utf-8?q?=E9=9C=8D=E6=96=8C=E6=96=8C?= \(beanhuo\)" , linux-samsung-soc , Russell King , Vinod Koul , =?utf-8?q?Rafa=C5=82_Mi=C5=82ecki?= , Kukjin Kim , Ben Hutchings , devicetree , Pawel Moll , Ian Campbell , Kumar Gala , Mark Brown , Dan Williams , "linux-arm-kernel@lists.infradead.org" , "grmoore@altera.com" , Linux Kernel Mailing List , linux-spi , Huang Shijie , Rob Herring , Han Xu , Knut Wohlrab , dmaengine , Brian Norris , David Woodhouse List-Id: Linux MTD discussion mailing list List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , On Thursday, June 04, 2015 at 06:21:32 AM, Michal Suchanek wrote: > On 4 June 2015 at 00:53, Marek Vasut wrote: > > On Wednesday, June 03, 2015 at 11:26:39 PM, Michal Suchanek wrote: > >> Hello, > > > > Hi, > > > >> this patch series makes it possible to access the SPI NOR flash in the > >> Samsung XE303 'Snow' Chromebook. > >> > >> Unfortunately not all issues are resolved. To work around an issue with > >> the pl330 dma engine I respun the patch for limiting transfer size in > >> m25p80 driver. > > > > Looks like fixing a bug at the wrong place to me ... > > > >> As the flash does not contain any sane filesystem and is only likely to > >> be accessed with mtd_debug or similar tool the limit of the dma engine > >> is easily reached. Filesystems using shorter data transfers are less > >> likely to be affected. > > > > Sounds like the DMA engine driver should be fixed. > > I looked at the pl330 datasheet and don't see anything obviusly wrong > with the pl330 driver in Linux. > > Ideally it would be fixed but I have no idea what's wrong with it. Ask maybe Catalin from ARM to help you -- he might be able to point you to the right hardware guy. Best regards, Marek Vasut