[U-Boot] [PATCH] am3517evm: change console device from ttyS2 to ttyO2

the serial device names have been changed from ttySx to ttyOx, so the console device name should be also changed to support the latest kernel versions.
Signed-off-by: Yegor Yefremov yegorslists@googlemail.com
--- include/configs/am3517_evm.h | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-)
Index: b/include/configs/am3517_evm.h =================================================================== --- a/include/configs/am3517_evm.h +++ b/include/configs/am3517_evm.h @@ -190,7 +190,7 @@
#define CONFIG_EXTRA_ENV_SETTINGS \ "loadaddr=0x82000000\0" \ - "console=ttyS2,115200n8\0" \ + "console=ttyO2,115200n8\0" \ "mmcargs=setenv bootargs console=${console} " \ "root=/dev/mmcblk0p2 rw " \ "rootfstype=ext3 rootwait\0" \

Hi Yegor,
Le 22/06/2011 10:33, Yegor Yefremov a écrit :
the serial device names have been changed from ttySx to ttyOx, so the console device name should be also changed to support the latest kernel versions.
Signed-off-by: Yegor Yefremovyegorslists@googlemail.com
This does not apply cleanly -- sorry, I should have pulled this earlier. Can you rebase and submit a V2 of the patch? Thanks.
Amicalement,

Hi Albert,
the serial device names have been changed from ttySx to ttyOx, so the console device name should be also changed to support the latest kernel versions.
Signed-off-by: Yegor Yefremovyegorslists@googlemail.com
This does not apply cleanly -- sorry, I should have pulled this earlier. Can you rebase and submit a V2 of the patch? Thanks.
What git repo are we talking about? The patch applies without problems against both:
git://git.denx.de/u-boot-ti.git git://git.denx.de/u-boot.git
Yegor

Hi Yegor,
Le 18/07/2011 09:32, Yegor Yefremov a écrit :
Hi Albert,
the serial device names have been changed from ttySx to ttyOx, so the console device name should be also changed to support the latest kernel versions.
Signed-off-by: Yegor Yefremovyegorslists@googlemail.com
This does not apply cleanly -- sorry, I should have pulled this earlier. Can you rebase and submit a V2 of the patch? Thanks.
What git repo are we talking about? The patch applies without problems against both:
git://git.denx.de/u-boot-ti.git git://git.denx.de/u-boot.git
Weird...
It has not been applied yet (current config file still has ttyS2) but does not applean cleanly on either u-boot-arm/master (which I applied it to in the first place), u-boot/master (which I tried then) or u-boot-ti/master (which I just tried). All three end up with:
Checking patch include/configs/am3517_evm.h... error: while searching for:
#define CONFIG_EXTRA_ENV_SETTINGS \ "loadaddr=0x82000000\0" \ "console=ttyS2,115200n8\0" \ "mmcargs=setenv bootargs console=${console} " \ "root=/dev/mmcblk0p2 rw " \ "rootfstype=ext3 rootwait\0" \
error: patch failed: include/configs/am3517_evm.h:190 Applying patch include/configs/am3517_evm.h with 1 rejects... Rejected hunk #1.
The master branches I used are the latest ones (git fetch done just before applying) and their respectve commit IDs are:
u-boot-arm/master: a55d23ccf6cb90acb9667a46427670add9486aec u-boot/master: 68d4230c3ccce96a72c5b99e48399bf1796fe3c6 u-boot-ti/master: 7b2fac7654f7420c2787f74ec3b1540fa3b343e9
Yegor
Amicalement,

Hi Albert,
the serial device names have been changed from ttySx to ttyOx, so the console device name should be also changed to support the latest kernel versions.
Signed-off-by: Yegor Yefremovyegorslists@googlemail.com
This does not apply cleanly -- sorry, I should have pulled this earlier. Can you rebase and submit a V2 of the patch? Thanks.
What git repo are we talking about? The patch applies without problems against both:
git://git.denx.de/u-boot-ti.git git://git.denx.de/u-boot.git
Weird...
It has not been applied yet (current config file still has ttyS2) but does not applean cleanly on either u-boot-arm/master (which I applied it to in the first place), u-boot/master (which I tried then) or u-boot-ti/master (which I just tried). All three end up with:
Checking patch include/configs/am3517_evm.h... error: while searching for:
#define CONFIG_EXTRA_ENV_SETTINGS \ "loadaddr=0x82000000\0" \ "console=ttyS2,115200n8\0" \ "mmcargs=setenv bootargs console=${console} " \ "root=/dev/mmcblk0p2 rw " \ "rootfstype=ext3 rootwait\0" \
error: patch failed: include/configs/am3517_evm.h:190 Applying patch include/configs/am3517_evm.h with 1 rejects... Rejected hunk #1.
The master branches I used are the latest ones (git fetch done just before applying) and their respectve commit IDs are:
u-boot-arm/master: a55d23ccf6cb90acb9667a46427670add9486aec u-boot/master: 68d4230c3ccce96a72c5b99e48399bf1796fe3c6 u-boot-ti/master: 7b2fac7654f7420c2787f74ec3b1540fa3b343e9
I've sent the v2 version. I think that GMail web interface could have damaged the patch.
Yegor
participants (2)
-
Albert ARIBAUD
-
Yegor Yefremov