Re: [PATCH] serial: 8250_dw: Honor clk_round_rate errors in dw8250_set_termios

2017-02-28 Thread Andy Shevchenko
On Tue, Feb 28, 2017 at 5:17 AM, Heiko Stuebner wrote: > 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

Re: [PATCH] serial: 8250_dw: Honor clk_round_rate errors in dw8250_set_termios

2017-02-28 Thread Andy Shevchenko
On Tue, Feb 28, 2017 at 5:17 AM, Heiko Stuebner wrote: > 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.

[PATCH] serial: 8250_dw: Honor clk_round_rate errors in dw8250_set_termios

2017-02-27 Thread Heiko Stuebner
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

[PATCH] serial: 8250_dw: Honor clk_round_rate errors in dw8250_set_termios

2017-02-27 Thread Heiko Stuebner
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