From mboxrd@z Thu Jan 1 00:00:00 1970 From: arnd@arndb.de (Arnd Bergmann) Date: Wed, 10 Oct 2012 18:25:49 +0000 Subject: [PATCH 5/7] ARM: clps711x: make all virtual addresses definition via one macro In-Reply-To: <1349883933-8881-5-git-send-email-shc_work@mail.ru> References: <1349883933-8881-1-git-send-email-shc_work@mail.ru> <1349883933-8881-5-git-send-email-shc_work@mail.ru> Message-ID: <201210101825.49391.arnd@arndb.de> To: linux-arm-kernel@lists.infradead.org List-Id: linux-arm-kernel.lists.infradead.org On Wednesday 10 October 2012, Alexander Shiyan wrote: > This patch make all virtual addresses definition via one macro. > This modification allows to avoid warning "BUG: mapping for 0x80000000 > at 0xff000000 out of vmalloc space". > > Signed-off-by: Alexander Shiyan Two comments on this one: * I wonder if we could just kill off some of the mappings if no driver relies on the registers being mapped in advance, especially for those that don't benefit from section mapping like the ethernet one that is only a page anyway. * I would recommend defining the IO_ADDRESS macro in a way that the result is of type 'void __iomem *', and you add a cast in the map_desc array. We will probably clean those up eventually and require that they are pointers at that place anyway, based on recent discussions. Arnd