[RESEND v2 PATCH 00/16] Nokia RX-51: Fix USB TTY console and enable it

Sat Feb 20 09:20:05 CET 2021

On Fri, 19 Feb 2021 01:38:14 +0100
Pali Rohár <pali at kernel.org> wrote:

> On Tuesday 16 February 2021 09:28:06 Lukasz Majewski wrote:
> > Hi Pali,
> >   
> > > On Monday 08 February 2021 23:34:06 Pali Rohár wrote:  
> > > > On Monday 08 February 2021 23:21:38 Pali Rohár wrote:    
> > > > > On Monday 08 February 2021 23:15:33 Lukasz Majewski wrote:    
> > > > > > Hi Pali,
> > > > > >     
> > > > > > >     Resended v2 patch series with fixed commit messages
> > > > > > > 
> > > > > > > This patch series fix usbtty code (serial console via USB
> > > > > > > peripheral mode), fix underlying musb peripheral code, fix
> > > > > > > compilation of CONFIG_USB_DEVICE (used by usbtty), remove
> > > > > > > unused Nokia RX-51 code to decrease size of U-Boot binary
> > > > > > > and finally enable usbtty serial console for Nokia RX-51.
> > > > > > > 
> > > > > > > With this patch series debugging of Nokia RX-51 can be
> > > > > > > done also via USB serial console.
> > > > > > > 
> > > > > > > It fixes also stability of musb code and allows usage of
> > > > > > > file transfers via Kermit protocol on Nokia RX-51. Kermit
> > > > > > > file transfer via U-Boot loadb command is stable on Nokia
> > > > > > > N900 and gives about 52kB/s transfer rate.
> > > > > > > 
> > > > > > > On computer this serial console is accessible via
> > > > > > > /dev/ttyACM0 device.   
> > > > > > 
> > > > > > I've integrated your patchset and now it turns out that the
> > > > > > u-boot size is too big:
> > > > > > https://dev.azure.com/lukma633/U-Boot/_build/results?buildId=24&view=results
> > > > > > https://dev.azure.com/lukma633/U-Boot/_build/results?buildId=24&view=logs&j=9a06d2a9-1498-5de0-2a01-be581d48ba67&t=f9a6b761-daa3-500f-4840-65a939c5040d
> > > > > >    
> > > > > 
> > > > > Ah :-(
> > > > > In November when I sent these patches, U-Boot binary was
> > > > > smaller. 
> > > > > > The branch is
> > > > > > https://github.com/lmajewski/u-boot-dfu/tree/testing
> > > > > > 
> > > > > > Have you experienced similar issues?    
> > > > > 
> > > > > Yes, memory for U-Boot is limited. It is needed to decrease
> > > > > size of U-Boot binary and then it will work.
> > > > > 
> > > > > I will try to look at it later and find some dead code which
> > > > > can be commented or removed to decrease binary size...    
> > > > 
> > > > Just a quick test, could you try to add
> > > > 
> > > > CONFIG_OPTIMIZE_INLINING=y
> > > > 
> > > > into configs/nokia_rx51_defconfig file, if it helps?    
> > > 
> > > Lukasz, is CONFIG_OPTIMIZE_INLINING=y option fixing build for
> > > you?  
> > 
> > Could you setup the CI environment and test your patchset with it?
> > 
> > It is documented in .azure-pipelines.yml.
> > 
> > Azure build takes not more than 3h for the whole test setup.  
> 
> It helped!
> 
> https://dev.azure.com/u-boot/u-boot/_build/results?buildId=1816&view=logs&j=9a06d2a9-1498-5de0-2a01-be581d48ba67&t=f9a6b761-daa3-500f-4840-65a939c5040d&l=8777

Great. Please resend the eligible patch (with adding this variable).
And I will prepare PR ASAP.

> 
> > >   
> > > > > If you have an idea what increased U-Boot side it could help.
> > > > >     
> > > > > > > Pali Rohár (16):
> > > > > > >   serial: usbtty: Fix puts function
> > > > > > >   serial: usbtty: Send urb data in correct order
> > > > > > >   usb: musb: Fix compilation of gadget code
> > > > > > >   usb: musb: Always clear the data toggle bit when
> > > > > > > configuring ep usb: musb: Fix configuring FIFO for
> > > > > > > endpoints usb: musb: Read value of PERI_RXCSR to 16bit
> > > > > > > variable usb: musb: Fix transmission of bigger buffers
> > > > > > >   usb: musb: Fix receiving of bigger buffers
> > > > > > >   usb: musb: Fix handling interrupts for EP0 and SET
> > > > > > > ADDRESS commmand usb: musb: Ensure that we set musb
> > > > > > > dynamic FIFO buffer for every endpoint
> > > > > > >   usb: gadget: Use dbg_ep0() macro instead of
> > > > > > > serial_printf() arm: omap3: Compile lowlevel_init()
> > > > > > > function only when it is used arm: omap3: Compile
> > > > > > > s_init() function only when it is used Nokia RX-51:
> > > > > > > Remove function set_muxconf_regs() Nokia RX-51: Move
> > > > > > > content of rx51.h to rx51.c Nokia RX-51: Enable usbtty
> > > > > > > serial console by default
> > > > > > > 
> > > > > > >  Makefile                                  |   1 +
> > > > > > >  arch/arm/mach-omap2/omap3/board.c         |   3 +
> > > > > > >  arch/arm/mach-omap2/omap3/lowlevel_init.S |   6 +-
> > > > > > >  board/nokia/rx51/rx51.c                   |  28 +-
> > > > > > >  board/nokia/rx51/rx51.h                   | 377
> > > > > > > ---------------------- configs/nokia_rx51_defconfig
> > > > > > >    | 7 +- doc/README.nokia_rx51                     |  15
> > > > > > > +- drivers/serial/usbtty.c                   |  16 +-
> > > > > > >  drivers/usb/gadget/ep0.c                  |  16 +-
> > > > > > >  drivers/usb/musb/musb_core.c              |  12 +-
> > > > > > >  drivers/usb/musb/musb_udc.c               |  61 ++--
> > > > > > >  include/configs/nokia_rx51.h              |  21 +-
> > > > > > >  12 files changed, 82 insertions(+), 481 deletions(-)
> > > > > > >  delete mode 100644 board/nokia/rx51/rx51.h
> > > > > > >     
> > > > > > 
> > > > > > 
> > > > > > 
> > > > > > 
> > > > > > Best regards,
> > > > > > 
> > > > > > Lukasz Majewski
> > > > > > 
> > > > > > --
> > > > > > 
> > > > > > DENX Software Engineering GmbH,      Managing Director:
> > > > > > Wolfgang Denk HRB 165235 Munich, Office: Kirchenstr.5,
> > > > > > D-82194 Groebenzell, Germany Phone: (+49)-8142-66989-59 Fax:
> > > > > > (+49)-8142-66989-80 Email: lukma at denx.de    
> > > > > 
> > > > >     
> > 
> > 
> > 
> > 
> > Best regards,
> > 
> > Lukasz Majewski
> > 
> > --
> > 
> > DENX Software Engineering GmbH,      Managing Director: Wolfgang
> > Denk HRB 165235 Munich, Office: Kirchenstr.5, D-82194 Groebenzell,
> > Germany Phone: (+49)-8142-66989-59 Fax: (+49)-8142-66989-80 Email:
> > lukma at denx.de  
> 
> 

Best regards,

Lukasz Majewski

--

DENX Software Engineering GmbH,      Managing Director: Wolfgang Denk
HRB 165235 Munich, Office: Kirchenstr.5, D-82194 Groebenzell, Germany
Phone: (+49)-8142-66989-59 Fax: (+49)-8142-66989-80 Email: lukma at denx.de
-------------- next part --------------
A non-text attachment was scrubbed...
Name: not available
Type: application/pgp-signature
Size: 488 bytes
Desc: OpenPGP digital signature
URL: <https://lists.denx.de/pipermail/u-boot/attachments/20210220/4d9f5277/attachment.sig>