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). 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(-) -- 2.43.0.rc1.1336.g36b5255a03ac
