Am Dienstag, 6. Mai 2014, 13:16:14 schrieb Kukjin Kim: > 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? Russell pointed out a bad decision on my part in v1, so I guess he is aware of this series :-) . I've also added arm@xxxxxxxxxx now, so they can complain, if anything is done wrong [should've probably done that from the beginning]. Heiko -- 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
 |