Hi Heinrich, On Sun, 29 Jan 2023 at 13:58, Heinrich Schuchardt <[email protected]> wrote: > > > > On 1/28/23 23:01, Simon Glass wrote: > > On Fri, 27 Jan 2023 at 17:12, Heinrich Schuchardt > > <[email protected]> wrote: > >> > >> Device name are typically longer than 8 characters. This leads to ragged > >> output. > >> Only the I and O bit of the device flags are of interest for the user. > >> Writing a hexadecimal number is just confusing. > >> > >> Before the patch the output looked like this: > >> > >> => coninfo > >> List of available devices: > >> pl011@9000000 00000007 IO stdin stdout stderr > >> serial 00000003 IO > >> usbkbd 00000001 I. > >> > >> With the patch the output looks like this: > >> > >> => coninfo > >> List of available devices > >> |-- pl011@9000000 (IO) > >> | |-- stdin > >> | |-- stdout > >> | |-- stderr > >> |-- serial (IO) > >> |-- usbkbd (I) > >> > >> Signed-off-by: Heinrich Schuchardt <[email protected]> > >> --- > >> cmd/console.c | 14 ++++++-------- > >> 1 file changed, 6 insertions(+), 8 deletions(-) > > > > Reviewed-by: Simon Glass <[email protected]> > > > > Is there a help update for this? > > The online help does not need a change. > > But a man-page in the HTML documentation with an example showing > reassignment of stdios would be helpful. > > In drv_system_init() we create a stdio device called "serial". Do we > need this dummy device if CONFIG_$(SPL_TPL_)DM_SERIAL=y? I find this > device duplicating the default serial device confusing.
I believe this is the console device for serial and that it is needed. But please take a look as I may be out of date on that. Regards, Simon
