Re: [PATCH v2 6/7] USB: serial: ftdi_sio: Fix custom_divisor and c_*speed for ASYNC_SPD_CUST

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 



On Thu, Aug 18, 2022 at 04:09:52PM +0200, Pali Rohár wrote:
> On Sunday 24 July 2022 15:08:09 Johan Hovold wrote:
> > On Sun, Jul 24, 2022 at 02:59:08PM +0200, Pali Rohár wrote:
> > > On Sunday 24 July 2022 14:54:58 Johan Hovold wrote:
> > > > On Sun, Jul 24, 2022 at 02:33:51PM +0200, Pali Rohár wrote:
> > 
> > > > > What is updated is the real baudrate in c_ispeed and c_ospeed
> > > > > extensions.
> > > > > 
> > > > > It is really wrong? I thought that c_cflag should stay unchanged at
> > > > > B38400 when ASYNC_SPD_CUST is used.
> > > > 
> > > > Yeah, cflags stay unchanged, but you shouldn't touch those fields when
> > > > using the deprecated ASYNC_SPD_CUST hack.
> > > 
> > > Hm... Why? I thought that new extended fields (c_ispeed and c_ospeed)
> > > should contain current real speed. What is the reason that c_*speed
> > > fields should have 38400 when ASYNC_SPD_CUST hack is set?
> > 
> > Because we shouldn't go adding new features built around the deprecated
> > ASYNC_SPD_CUST hack.
> 
> But this is not a new feature in the old deprecated hack. It for the
> new interface.

I think I understand what you're getting at, but no. Let's not add more
features built around ASYNC_SPD_CUST.

Johan



[Index of Archives]     [Linux Media]     [Linux Input]     [Linux Audio Users]     [Yosemite News]     [Linux Kernel]     [Linux SCSI]     [Old Linux USB Devel Archive]

  Powered by Linux