[5 PATCH 0/2] Backport fixes to zynq uart to RTEMS5
This patchset backports the commits of Kinsey Moore and myself, which fix the stdin behavior for the zynq-uart based console driver. I checked the behavior on hardware with the termios.exe application. Before, the scanf and similar functions do not wait for user input and return immediately. With the patches the application behaves as expected. A corresponding ticket can be found here: https://devel.rtems.org/ticket/4236 Small side note: I noticed that the ARM BSP docs for RTEMS5 list mostly TODO. Should I create a patchset backporting the BSP documentation or do you want to do that in one go as part of the 5.2 release preparation? Best regards, Jan Jan Sommer (1): bsps/shared: Allow setting baud rate for zynq uart Kinsey Moore (1): zynq-uart: Fix set_attributes implementation bsps/arm/include/bsp/zynq-uart.h | 7 +++ bsps/arm/shared/serial/zynq-uart-polled.c | 2 +- bsps/arm/shared/serial/zynq-uart.c| 77 --- 3 files changed, 75 insertions(+), 11 deletions(-) -- 2.17.1 ___ devel mailing list devel@rtems.org http://lists.rtems.org/mailman/listinfo/devel
Re: [5 PATCH 0/2] Backport fixes to zynq uart to RTEMS5
OK for 5. On 6/3/21 5:58 am, Jan Sommer wrote: > This patchset backports the commits of Kinsey Moore and myself, which > fix the stdin behavior for the zynq-uart based console driver. > I checked the behavior on hardware with the termios.exe application. > Before, the scanf and similar functions do not wait for user input and return > immediately. With the patches the application behaves as expected. > > A corresponding ticket can be found here: https://devel.rtems.org/ticket/4236 > > Small side note: > I noticed that the ARM BSP docs for RTEMS5 list mostly TODO. > Should I create a patchset backporting the BSP documentation or do you > want to do that in one go as part of the 5.2 release preparation? > > Best regards, > > Jan > > Jan Sommer (1): > bsps/shared: Allow setting baud rate for zynq uart > > Kinsey Moore (1): > zynq-uart: Fix set_attributes implementation > > bsps/arm/include/bsp/zynq-uart.h | 7 +++ > bsps/arm/shared/serial/zynq-uart-polled.c | 2 +- > bsps/arm/shared/serial/zynq-uart.c| 77 --- > 3 files changed, 75 insertions(+), 11 deletions(-) > ___ devel mailing list devel@rtems.org http://lists.rtems.org/mailman/listinfo/devel
Re: [5 PATCH 0/2] Backport fixes to zynq uart to RTEMS5
On Fri, Mar 5, 2021 at 11:59 AM Jan Sommer wrote: > > This patchset backports the commits of Kinsey Moore and myself, which > fix the stdin behavior for the zynq-uart based console driver. > I checked the behavior on hardware with the termios.exe application. > Before, the scanf and similar functions do not wait for user input and return > immediately. With the patches the application behaves as expected. > > A corresponding ticket can be found here: https://devel.rtems.org/ticket/4236 > > Small side note: > I noticed that the ARM BSP docs for RTEMS5 list mostly TODO. > Should I create a patchset backporting the BSP documentation or do you > want to do that in one go as part of the 5.2 release preparation? You should do this if you find it valuable. Not many of us take time to backport to the release docs, and no one will pick those changes up automatically. > > Best regards, > > Jan > > Jan Sommer (1): > bsps/shared: Allow setting baud rate for zynq uart > > Kinsey Moore (1): > zynq-uart: Fix set_attributes implementation > > bsps/arm/include/bsp/zynq-uart.h | 7 +++ > bsps/arm/shared/serial/zynq-uart-polled.c | 2 +- > bsps/arm/shared/serial/zynq-uart.c| 77 --- > 3 files changed, 75 insertions(+), 11 deletions(-) > > -- > 2.17.1 > > ___ > devel mailing list > devel@rtems.org > http://lists.rtems.org/mailman/listinfo/devel ___ devel mailing list devel@rtems.org http://lists.rtems.org/mailman/listinfo/devel
RE: [5 PATCH 0/2] Backport fixes to zynq uart to RTEMS5
> -Original Message- > From: Chris Johns > Sent: Tuesday, March 9, 2021 3:20 AM > To: Sommer, Jan ; devel@rtems.org > Subject: Re: [5 PATCH 0/2] Backport fixes to zynq uart to RTEMS5 > > OK for 5. > Thanks. Pushed to 5. > On 6/3/21 5:58 am, Jan Sommer wrote: > > This patchset backports the commits of Kinsey Moore and myself, which > > fix the stdin behavior for the zynq-uart based console driver. > > I checked the behavior on hardware with the termios.exe application. > > Before, the scanf and similar functions do not wait for user input and > > return immediately. With the patches the application behaves as expected. > > > > A corresponding ticket can be found here: > > https://devel.rtems.org/ticket/4236 > > > > Small side note: > > I noticed that the ARM BSP docs for RTEMS5 list mostly TODO. > > Should I create a patchset backporting the BSP documentation or do you > > want to do that in one go as part of the 5.2 release preparation? > > > > Best regards, > > > > Jan > > > > Jan Sommer (1): > > bsps/shared: Allow setting baud rate for zynq uart > > > > Kinsey Moore (1): > > zynq-uart: Fix set_attributes implementation > > > > bsps/arm/include/bsp/zynq-uart.h | 7 +++ > > bsps/arm/shared/serial/zynq-uart-polled.c | 2 +- > > bsps/arm/shared/serial/zynq-uart.c| 77 --- > > 3 files changed, 75 insertions(+), 11 deletions(-) > > ___ devel mailing list devel@rtems.org http://lists.rtems.org/mailman/listinfo/devel