# Re: \[RFC 3/6\] drm: add SimpleDRM driver

- From: David Herrmann \<hidden\>
- Date: 2013-06-28 10:00:03 
- List: linux-fbdev
- Page: https://ratatoskr.run/linux-fbdev/2013/06/4074940
- Thread: https://ratatoskr.run/dri-devel/2013/06/4072197/t.md

```text
Hi

On Tue, Jun 25, 2013 at 3:05 AM, Andy Lutomirski [ref] wrote:
> On 06/24/2013 03:27 PM, David Herrmann wrote:
>> +     sdrm->fb_map = ioremap(sdrm->fb_base, sdrm->fb_size);
>
> This should probably be ioremap_wc.  Otherwise it will be *really* slow
> if used in legacy mode and it may cause conflicts with the
> pgprot_writecombine mode for mmap.

Whoops, yepp, I fixed that.

Thanks
David
```