From mboxrd@z Thu Jan 1 00:00:00 1970 From: Eric Nelson Date: Fri, 26 Jul 2013 17:56:07 -0700 Subject: [U-Boot] [PATCH] video: mxc_ipuv3: fix memory alignment of framebuffer In-Reply-To: <201307262138.53588.marex@denx.de> References: <1374798926-18743-1-git-send-email-eric.nelson@boundarydevices.com> <201307260627.10732.marex@denx.de> <51F28935.7000905@boundarydevices.com> <201307262138.53588.marex@denx.de> Message-ID: <51F31AA7.5040207@boundarydevices.com> List-Id: MIME-Version: 1.0 Content-Type: text/plain; charset="us-ascii" Content-Transfer-Encoding: 7bit To: u-boot@lists.denx.de On 07/26/2013 12:38 PM, Marek Vasut wrote: > Dear Eric Nelson, > >>>> - fbi->screen_base = (char *)malloc(fbi->fix.smem_len); >>>> + fbi->screen_base = (char *)memalign(ARCH_DMA_MINALIGN, >>>> + fbi->fix.smem_len); >>> >>> You might want to round-up the length too, like I did in the MXS driver. >> >> Did you really run across a display whose size wasn't a multiple of >> 32, or is this rounding just in muscle-memory for you? > > Palm devices with PXA27x (PalmT5, PalmLD, PalmTX) had 484x324 LCDs ;-) > With as much cache-enablement as you've done lately, I have a feeling you round up at the grocery store or pub ;)