Heiko Stübner wrote: > > This series tries to simplify the s3c24xx debug macro, removing > dependencies > on mach/ includes, static mappings and finally moving it into > include/debug. > I think, it's good way :) > The one slightly invasive change is the need for the developer to select > the uart type by himself, which gets rid of the debug macro trying to > determine the uart type itself. > > But as usage of the debug-uart is not the common case - especially in a > multiplatform scenario - I didn't worry to much. > Yeah, I also don't want s3c24xx to support multiplatform in near future. > Based on 3.15-rc1 and tested on a S3C2442 Openmoko Freerunner (GTA02) > > changes since v1: > - do not introduce a secondary choice option, instead implement the > s3c2410 debug uarts as separate options > > Heiko Stuebner (4): > ARM: compressed/head.S: remove s3c24xx special case > ARM: S3C24XX: trim down debug uart handling > ARM: S3C24XX: use generic DEBUG_UART_PHY/_VIRT in debug macro > ARM: S3C24XX: move debug-macro.S into the common space > > arch/arm/Kconfig.debug | 54 +++++++++++- > arch/arm/boot/compressed/head.S | 5 -- > arch/arm/include/debug/s3c24xx.S | 46 +++++++++++ > arch/arm/mach-s3c24xx/Kconfig | 28 ------- > arch/arm/mach-s3c24xx/include/mach/debug-macro.S | 101 ------------------ > ----- > 5 files changed, 98 insertions(+), 136 deletions(-) > create mode 100644 arch/arm/include/debug/s3c24xx.S > delete mode 100644 arch/arm/mach-s3c24xx/include/mach/debug-macro.S > > -- > 1.9.0 Basically I'm OK on this series but need to get review from Russell? Thanks, Kukjin -- To unsubscribe from this list: send the line "unsubscribe linux-samsung-soc" in the body of a message to majordomo@xxxxxxxxxxxxxxx More majordomo info at http://vger.kernel.org/majordomo-info.html
 |