Re: [PATCH 3/3] tty: support CIBAUD without BOTHER
From: Greg Kroah-Hartman
Date: Mon Jul 16 2018 - 06:00:37 EST
On Sun, Jul 15, 2018 at 03:39:35PM +0200, Johan Hovold wrote:
> Since commit edc6afc54968 ("[PATCH] tty: switch to ktermios and new
> framework") arbitrary baud rates can be requested using BOTHER and input
> rates can be requested using the termios CIBAUD bits (CBAUD shifted
> IBSHIFT bits).
>
> This functionality has been conditionally compiled depending on whether
> an architecture defines BOTHER and IBSHIFT respectively, but would in
> fact fail to compile unless both symbols were defined due to cross
> dependencies.
>
> Relax the IBSHIFT => BOTHER dependency so that an architecture could
> theoretically support CIBAUD without the Linux-specific BOTHER, while
> hopefully making the current conditional-compilation directives a bit
> less confusing.
>
> Note that the long-term goal is still to have all architectures support
> both features, so an alternative could just be to have the lot depend on
> BOTHER.
I thought we had all arches converted to use BOTHER already, what ones
are not yet done? It's hard to unwind the asm-generic use of termbits.h
to obviously see which ones are not doing this yet, any ideas?
Oh, and thanks for fixing this all up, odd that no one has noticed it
before.
thanks,
greg k-h