On Thu, Jun 06, 2013 at 02:37:01PM +0530, Tushar Behera wrote: > On 06/04/2013 09:49 AM, Tushar Behera wrote: > > Printing low-level debug messages make an assumption that the specified > > UART port has been preconfigured by the bootloader. Incorrectly > > specified UART port results in system getting stalled while printing the > > message "Uncompressing Linux... done, booting the kernel" > > This UART port number is specified through S3C_LOWLEVEL_UART_PORT. Since > > the UART port might different for different board, it is not possible to > > specify it correctly for every board that use a common defconfig file. > > > > Calling this print subroutine only when DEBUG_LL fixes the problem. By > > disabling DEBUG_LL in default config file, we would be able to boot > > multiple boards with different default UART ports. > > > > With this current approach, we miss the print "Uncompressing Linux... > > done, booting the kernel." when DEBUG_LL is not defined. > > > > Signed-off-by: Tushar Behera <tushar.behera@xxxxxxxxxx> > > --- > > Change for v2: > > * Instead of checking value of uart_base, now check if DEBUG_LL > > is enabled. This removes the ifdef's from mach/uncompress.h > > > > > > arch/arm/plat-samsung/include/plat/uncompress.h | 10 +++++++++- > > 1 file changed, 9 insertions(+), 1 deletion(-) > > > > If there are no concerns, would it be possible to queue this patch [1/3] > for 3.10? > > The other two patches can wait for 3.11. Sure, I've applied it to fixes for 3.10. -Olof -- 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
 |