Re: [PATCH] ARM: dts: exynos4: Enable serial controllers on Origen and SMDKV310
On Tuesday 23 of October 2012 21:03:01 Kukjin Kim wrote: > Tomasz Figa wrote: > > However I think this should be considered a fix, because without it > > serial console on Origen and SMDKV310 will be broken. > > Well, I couldn't find any broken due to miss this for v3.7. I think, > there is no usage of serial controller in mainline for Origen and > SMDKV310 now. So this should be queued for -next. If not, please let me > know. Personally I'm fine with this, but it sounds a bit off, since (correct me if I am wrong) it looks like serial console is the only way to check if the board boots. Best regards, Tomasz Figa -- To unsubscribe from this list: send the line "unsubscribe linux-samsung-soc" in the body of a message to majord...@vger.kernel.org More majordomo info at http://vger.kernel.org/majordomo-info.html
RE: [PATCH] ARM: dts: exynos4: Enable serial controllers on Origen and SMDKV310
Tomasz Figa wrote: > > However I think this should be considered a fix, because without it serial > console on Origen and SMDKV310 will be broken. > Well, I couldn't find any broken due to miss this for v3.7. I think, there is no usage of serial controller in mainline for Origen and SMDKV310 now. So this should be queued for -next. If not, please let me know. Thanks. Best regards, Kgene. -- Kukjin Kim , Senior Engineer, SW Solution Development Team, Samsung Electronics Co., Ltd. -- To unsubscribe from this list: send the line "unsubscribe linux-samsung-soc" in the body of a message to majord...@vger.kernel.org More majordomo info at http://vger.kernel.org/majordomo-info.html
Re: [PATCH] ARM: dts: exynos4: Enable serial controllers on Origen and SMDKV310
Hi Kgene, On Monday 15 of October 2012 23:23:22 Kukjin Kim wrote: > Tomasz Figa wrote: > > Hi Kgene, > > > > On Monday 08 of October 2012 15:12:48 Tomasz Figa wrote: > > > This patch adds status override of serial nodes to enable used > > > serial > > > ports on Origen and SMDKV310 board. > > > > > > Signed-off-by: Tomasz Figa > > > --- > > > > > > arch/arm/boot/dts/exynos4210-origen.dts | 16 > > > arch/arm/boot/dts/exynos4210-smdkv310.dts | 16 > > > 2 files changed, 32 insertions(+) > > > > I think you missed this patch when applying Exynos dts reorganization > > patches. It is needed to enable serial ports on Origen and SMDKV310 > > boards. > > > Could you apply it as well? > > Oops, yeah. I will apply. I saw that you applied this patch to your next/dt-samsung branch, thanks. However I think this should be considered a fix, because without it serial console on Origen and SMDKV310 will be broken. Best regards, Tomasz Figa -- To unsubscribe from this list: send the line "unsubscribe linux-samsung-soc" in the body of a message to majord...@vger.kernel.org More majordomo info at http://vger.kernel.org/majordomo-info.html
RE: [PATCH] ARM: dts: exynos4: Enable serial controllers on Origen and SMDKV310
Tomasz Figa wrote: > > Hi Kgene, > > On Monday 08 of October 2012 15:12:48 Tomasz Figa wrote: > > This patch adds status override of serial nodes to enable used serial > > ports on Origen and SMDKV310 board. > > > > Signed-off-by: Tomasz Figa > > --- > > arch/arm/boot/dts/exynos4210-origen.dts | 16 > > arch/arm/boot/dts/exynos4210-smdkv310.dts | 16 > > 2 files changed, 32 insertions(+) > > > > I think you missed this patch when applying Exynos dts reorganization > patches. It is needed to enable serial ports on Origen and SMDKV310 boards. > Could you apply it as well? > Oops, yeah. I will apply. Thanks. Best regards, Kgene. -- Kukjin Kim , Senior Engineer, SW Solution Development Team, Samsung Electronics Co., Ltd. -- To unsubscribe from this list: send the line "unsubscribe linux-samsung-soc" in the body of a message to majord...@vger.kernel.org More majordomo info at http://vger.kernel.org/majordomo-info.html
Re: [PATCH] ARM: dts: exynos4: Enable serial controllers on Origen and SMDKV310
Hi Kgene, On Monday 08 of October 2012 15:12:48 Tomasz Figa wrote: > This patch adds status override of serial nodes to enable used serial > ports on Origen and SMDKV310 board. > > Signed-off-by: Tomasz Figa > --- > arch/arm/boot/dts/exynos4210-origen.dts | 16 > arch/arm/boot/dts/exynos4210-smdkv310.dts | 16 > 2 files changed, 32 insertions(+) > I think you missed this patch when applying Exynos dts reorganization patches. It is needed to enable serial ports on Origen and SMDKV310 boards. Could you apply it as well? Best regards, -- Tomasz Figa Samsung Poland R&D Center -- To unsubscribe from this list: send the line "unsubscribe linux-samsung-soc" in the body of a message to majord...@vger.kernel.org More majordomo info at http://vger.kernel.org/majordomo-info.html
Re: [PATCH] ARM: dts: exynos4: Enable serial controllers on Origen and SMDKV310
Hi Kgene, On Saturday 08 of September 2012 11:36:30 Kukjin Kim wrote: > Tomasz Figa wrote: > > This patch adds status override of serial nodes to enable used serial > > ports > > on Origen and SMDKV310 board. > > Well...I'm not sure about the status of Origen serial, but all of the > serials are enabled on SMDKV310, is it right? According to arrays of struct s3c2410_uartcfg in mach-origen.c and mach- smdkv310.c, all the four serial ports are enabled on both boards. -- Best regards, Tomasz Figa -- To unsubscribe from this list: send the line "unsubscribe linux-samsung-soc" in the body of a message to majord...@vger.kernel.org More majordomo info at http://vger.kernel.org/majordomo-info.html
RE: [PATCH] ARM: dts: exynos4: Enable serial controllers on Origen and SMDKV310
Tomasz Figa wrote: > > This patch adds status override of serial nodes to enable used serial > ports > on Origen and SMDKV310 board. > Well...I'm not sure about the status of Origen serial, but all of the serials are enabled on SMDKV310, is it right? Thanks. Best regards, Kgene. -- Kukjin Kim , Senior Engineer, SW Solution Development Team, Samsung Electronics Co., Ltd. -- To unsubscribe from this list: send the line "unsubscribe linux-samsung-soc" in the body of a message to majord...@vger.kernel.org More majordomo info at http://vger.kernel.org/majordomo-info.html