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(-) >
