Commit 5ec2952c authored by Ilpo Järvinen's avatar Ilpo Järvinen Committed by Greg Kroah-Hartman
Browse files

serial: sunsab: Remove frame size calculation dead-code 



The driver features a custom frame length calculation but the result is
never used. Remove it.

Reviewed-by: default avatarAndy Shevchenko <andy.shevchenko@gmail.com>
Signed-off-by: default avatarIlpo Järvinen <ilpo.jarvinen@linux.intel.com>
Link: https://lore.kernel.org/r/20220830084925.5608-5-ilpo.jarvinen@linux.intel.com


Signed-off-by: default avatarGreg Kroah-Hartman <gregkh@linuxfoundation.org>
parent d4be10c8

drivers/tty/serial/sunsab.c

+8 −12
Original line number Diff line number Diff line
@@ -681,27 +681,23 @@ static void sunsab_convert_to_sab(struct uart_sunsab_port *up, unsigned int cfla 
				  unsigned int quot)

{

	unsigned char dafo;

	int bits, n, m;

	int n, m;



	/* Byte size and parity */

	switch (cflag & CSIZE) {

	      case CS5: dafo = SAB82532_DAFO_CHL5; bits = 7; break;

	      case CS6: dafo = SAB82532_DAFO_CHL6; bits = 8; break;

	      case CS7: dafo = SAB82532_DAFO_CHL7; bits = 9; break;

	      case CS8: dafo = SAB82532_DAFO_CHL8; bits = 10; break;

	      case CS5: dafo = SAB82532_DAFO_CHL5; break;

	      case CS6: dafo = SAB82532_DAFO_CHL6; break;

	      case CS7: dafo = SAB82532_DAFO_CHL7; break;

	      case CS8: dafo = SAB82532_DAFO_CHL8; break;

	      /* Never happens, but GCC is too dumb to figure it out */

	      default:  dafo = SAB82532_DAFO_CHL5; bits = 7; break;

	      default:  dafo = SAB82532_DAFO_CHL5; break;

	}



	if (cflag & CSTOPB) {

	if (cflag & CSTOPB)

		dafo |= SAB82532_DAFO_STOP;

		bits++;

	}



	if (cflag & PARENB) {

	if (cflag & PARENB)

		dafo |= SAB82532_DAFO_PARE;

		bits++;

	}



	if (cflag & PARODD) {

		dafo |= SAB82532_DAFO_PAR_ODD;