diff options
| author | Heiko Stuebner <heiko@sntech.de> | 2017-03-09 07:41:16 +0100 |
|---|---|---|
| committer | Greg Kroah-Hartman <gregkh@linuxfoundation.org> | 2017-03-14 10:48:11 +0800 |
| commit | 09498087f922bb623b66db9e89c6fd11a3799867 (patch) | |
| tree | 18c24d819f0d40d4f74dbaf187b9a9c7f4cebd4c /security/integrity | |
| parent | 4495c08e84729385774601b5146d51d9e5849f81 (diff) | |
| download | linux-09498087f922bb623b66db9e89c6fd11a3799867.tar.bz2 | |
serial: 8250_dw: Honor clk_round_rate errors in dw8250_set_termios
clk_round_rate returns a signed long and may possibly return errors
in it, for example if there is no possible rate.
Till now dw8250_set_termios ignored any error, the signednes and would
just use the value as input to clk_set_rate. This of course falls apart
if there is an actual error, so check for errors and only try to set
a rate if the value is actually valid.
This turned up on some Rockchip platforms after commit
6a171b299379 ("serial: 8250_dw: Allow hardware flow control to be used")
enabled set_termios callback in all cases, not only ACPI.
Fixes: 6a171b299379 ("serial: 8250_dw: Allow hardware flow control to be used")
Signed-off-by: Heiko Stuebner <heiko@sntech.de>
Reviewed-by: Andy Shevchenko <andy.shevchenko@gmail.com>
Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
Diffstat (limited to 'security/integrity')
0 files changed, 0 insertions, 0 deletions