Re: [PATCH v2 00/13] serial: 8250_exar: Clean up the driver

From: Parker Newman
Date: Fri May 03 2024 - 14:35:54 EST


On Fri, 3 May 2024 20:15:52 +0300
Andy Shevchenko <andriy.shevchenko@xxxxxxxxxxxxxxx> wrote:

> After a rework for CONNTECH was done, the driver may need a bit of
> love in order to become less verbose (in terms of indentation and
> code duplication) and hence easier to read.
>
> This clean up series fixes a couple of (not so critical) issues and
> cleans up the recently added code. No functional change indented by
> the cleaning up part.
>
> Parker, please test this and give your formal Tested-by tag
> (you may do it by replying to this message if all patches are
> successfully tested; more details about tags are available in
> the Submitting Patches documentation).
>

I was able to test the Connect Tech related code and everything is
work as expected. I can't test the non-CTI related changes but they
are pretty minor.

Tested-by: Parker Newman <pnewman@xxxxxxxxxxxxxxx>

> In v2:
> - fixed the EEPROM reading data loop (Ilpo, Parker)
>
> Andy Shevchenko (13):
> serial: 8250_exar: Don't return positive values as error codes
> serial: 8250_exar: Describe all parameters in kernel doc
> serial: 8250_exar: Kill CTI_PCI_DEVICE()
> serial: 8250_exar: Use PCI_SUBVENDOR_ID_IBM for subvendor ID
> serial: 8250_exar: Trivia typo fixes
> serial: 8250_exar: Extract cti_board_init_osc_freq() helper
> serial: 8250_exar: Kill unneeded ->board_init()
> serial: 8250_exar: Decrease indentation level
> serial: 8250_exar: Return directly from switch-cases
> serial: 8250_exar: Switch to use dev_err_probe()
> serial: 8250_exar: Use BIT() in exar_ee_read()
> serial: 8250_exar: Make type of bit the same in exar_ee_*_bit()
> serial: 8250_exar: Keep the includes sorted
>
> drivers/tty/serial/8250/8250_exar.c | 459 ++++++++++++----------------
> 1 file changed, 203 insertions(+), 256 deletions(-)
>