Donate to e Foundation | Murena handsets with /e/OS | Own a part of Murena! Learn more

Commit ffff731d authored by Maciej W. Rozycki's avatar Maciej W. Rozycki Committed by Greg Kroah-Hartman
Browse files

serial: 8250: Correct the clock for EndRun PTP/1588 PCIe device 



commit 637674fa40059cddcc3ad2212728965072f62ea3 upstream.

The EndRun PTP/1588 dual serial port device is based on the Oxford
Semiconductor OXPCIe952 UART device with the PCI vendor:device ID set
for EndRun Technologies and is therefore driven by a fixed 62.5MHz clock
input derived from the 100MHz PCI Express clock.  The clock rate is
divided by the oversampling rate of 16 as it is supplied to the baud
rate generator, yielding the baud base of 3906250.

Replace the incorrect baud base of 4000000 with the right value of
3906250 then, complementing commit 6cbe45d8ac93 ("serial: 8250: Correct
the clock for OxSemi PCIe devices").

Signed-off-by: default avatarMaciej W. Rozycki <macro@orcam.me.uk>
Cc: stable <stable@kernel.org>
Fixes: 1bc8cde4 ("8250_pci: Added driver for Endrun Technologies PTP PCIe card.")
Reviewed-by: default avatarAndy Shevchenko <andy.shevchenko@gmail.com>
Link: https://lore.kernel.org/r/alpine.DEB.2.21.2204181515270.9383@angie.orcam.me.uk


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

drivers/tty/serial/8250/8250_pci.c

+4 −4
Original line number Diff line number Diff line
@@ -2915,7 +2915,7 @@ enum pci_board_num_t { 
	pbn_panacom2,

	pbn_panacom4,

	pbn_plx_romulus,

	pbn_endrun_2_4000000,

	pbn_endrun_2_3906250,

	pbn_oxsemi,

	pbn_oxsemi_1_4000000,

	pbn_oxsemi_2_4000000,
@@ -3479,10 +3479,10 @@ static struct pciserial_board pci_boards[] = { 
	* signal now many ports are available

	* 2 port 952 Uart support

	*/

	[pbn_endrun_2_4000000] = {

	[pbn_endrun_2_3906250] = {

		.flags		= FL_BASE0,

		.num_ports	= 2,

		.base_baud	= 4000000,

		.base_baud	= 3906250,

		.uart_offset	= 0x200,

		.first_offset	= 0x1000,

	},
@@ -4430,7 +4430,7 @@ static struct pci_device_id serial_pci_tbl[] = { 
	*/

	{	PCI_VENDOR_ID_ENDRUN, PCI_DEVICE_ID_ENDRUN_1588,

		PCI_ANY_ID, PCI_ANY_ID, 0, 0,

		pbn_endrun_2_4000000 },

		pbn_endrun_2_3906250 },

	/*

	 * Quatech cards. These actually have configurable clocks but for

	 * now we just use the default.