Hello, On Thursday, July 21, 2011 11:52 AM Kukjin Kim wrote: > > This patch series contains platform definitions for s5p-tv driver > > rebased onto latest for-next branch (including generic GIC and clockdev > > changes). > > > > Changes since v1: > > - renamed i2c8 to common i2c-hdmiphy > > - moved common vpll code to plat-s5p/clocks.c > > - added check for pll locking > > - various minor coding style fixes here and there > > > > Best regards > > -- > > Marek Szyprowski > > Samsung Poland R&D Center > > > > > > Patch summary: > > > > Tomasz Stanislawski (4): > > ARM: Samsung: add i2c hdmiphy controller definitions > > ARM: S5P: add support for tv device > > ARM: Exynos4: enable TV support on Universal_C210 board > > ARM: S5PV210: enable TV support on GONI board > > > > arch/arm/mach-exynos4/Kconfig | 2 + > > arch/arm/mach-exynos4/clock.c | 140 > > +++++++++++++++++++++- > > arch/arm/mach-exynos4/cpu.c | 4 + > > arch/arm/mach-exynos4/include/mach/iic-hdmiphy.h | 16 +++ > > arch/arm/mach-exynos4/include/mach/irqs.h | 4 + > > arch/arm/mach-exynos4/include/mach/map.h | 12 ++ > > arch/arm/mach-exynos4/include/mach/regs-pmu.h | 6 + > > arch/arm/mach-exynos4/mach-universal_c210.c | 74 ++++++++++++ > > arch/arm/mach-s5pv210/Kconfig | 1 + > > arch/arm/mach-s5pv210/clock.c | 87 ++++++++++++-- > > arch/arm/mach-s5pv210/cpu.c | 4 + > > arch/arm/mach-s5pv210/include/mach/iic-hdmiphy.h | 16 +++ > > arch/arm/mach-s5pv210/include/mach/irqs.h | 4 +- > > arch/arm/mach-s5pv210/include/mach/map.h | 13 ++ > > arch/arm/mach-s5pv210/include/mach/regs-clock.h | 9 +- > > arch/arm/mach-s5pv210/mach-goni.c | 9 ++ > > arch/arm/plat-s5p/Kconfig | 10 ++ > > arch/arm/plat-s5p/Makefile | 2 + > > arch/arm/plat-s5p/clock.c | 73 +++++++++++ > > arch/arm/plat-s5p/dev-i2c-hdmiphy.c | 59 +++++++++ > > arch/arm/plat-s5p/dev-tv.c | 100 +++++++++++++++ > > arch/arm/plat-s5p/include/plat/s5p-clock.h | 4 + > > arch/arm/plat-samsung/include/plat/devs.h | 6 + > > arch/arm/plat-samsung/include/plat/iic.h | 1 + > > arch/arm/plat-samsung/include/plat/tv-core.h | 44 +++++++ > > 25 files changed, 684 insertions(+), 16 deletions(-) > > create mode 100644 arch/arm/mach-exynos4/include/mach/iic-hdmiphy.h > > create mode 100644 arch/arm/mach-s5pv210/include/mach/iic-hdmiphy.h > > create mode 100644 arch/arm/plat-s5p/dev-i2c-hdmiphy.c > > create mode 100644 arch/arm/plat-s5p/dev-tv.c > > create mode 100644 arch/arm/plat-samsung/include/plat/tv-core.h > > > > -- > > 1.7.1.569.g6f426 > > Hi Marek, > > Looks good but happens following build failure with > s5pc100_defconfig/s5p64x0_defconfig. > Because they have no VPLL :( > > arch/arm/kernel/return_address.c:62: warning: #warning "TODO: return_address > should use unwind tables" > arch/arm/plat-s5p/clock.c: In function 's5p_vpll_set_rate': > arch/arm/plat-s5p/clock.c:237: error: 'S5P_VPLL_CON0' undeclared (first use > in this function) > arch/arm/plat-s5p/clock.c:237: error: (Each undeclared identifier is > reported only once > arch/arm/plat-s5p/clock.c:237: error: for each function it appears in.) > arch/arm/plat-s5p/clock.c:243: error: 'S5P_VPLL_CON1' undeclared (first use > in this function) > arch/arm/plat-s5p/clock.c:272: error: 'S5P_VPLLCON0_LOCKED_SHIFT' undeclared > (first use in this function) > make[1]: *** [arch/arm/plat-s5p/clock.o] Error 1 Huh... Right, I forgot about other platforms. What do you think about making this vpll code compiled conditionally only for EXYNOS4 and S5PV210? The other solution is to move it into a separate file. > First of all, I will send 'pull request' without this series... > > I'm not sure can have a chance to send 'pull request' again for this merge > window. Ok, I understand. Best regards -- Marek Szyprowski Samsung Poland R&D Center -- 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