arm64: dts: ti: k3-j721e: Remove UART baud rate selection

Linux kernel mirror (for testing) git.kernel.org/pub/scm/linux/kernel/git/torvalds/linux.git

kernel os linux

As described in the binding document for the "current-speed" property:

"This should only be present in case a driver has no chance to know the
baud rate of the slave device."

This is not the case for the UART used in K3 devices, the current
baud-rate can be calculated from the registers. Having this property
has the effect of actually skipping the baud-rate setup in some drivers
as it assumes it will already be set to this rate, which may not always
be the case.

It seems this property's purpose was mistaken as selecting the desired
baud-rate, which it does not. It would have been wrong to select that
here anyway as DT is not the place for configuration, especially when
there are already more standard ways to set serial baud-rates.

Signed-off-by: Andrew Davis <afd@ti.com>
Link: https://lore.kernel.org/r/20240326185441.29656-4-afd@ti.com
Signed-off-by: Nishanth Menon <nm@ti.com>

authored by

Andrew Davis and committed by

Nishanth Menon 2 years ago cef23c6b 52f02af9

-12

2 changed files

expand all

arch

arm64

boot

dts

k3-j721e-main.dtsi

k3-j721e-mcu-wakeup.dtsi

-10

arch/arm64/boot/dts/ti/k3-j721e-main.dtsi

··· 1337 1337 reg = <0x00 0x02800000 0x00 0x100>; 1338 1338 interrupts = <GIC_SPI 192 IRQ_TYPE_LEVEL_HIGH>; 1339 1339 clock-frequency = <48000000>; 1340 - current-speed = <115200>; 1341 1340 power-domains = <&k3_pds 146 TI_SCI_PD_EXCLUSIVE>; 1342 1341 clocks = <&k3_clks 146 0>; 1343 1342 clock-names = "fclk"; ··· 1348 1349 reg = <0x00 0x02810000 0x00 0x100>; 1349 1350 interrupts = <GIC_SPI 193 IRQ_TYPE_LEVEL_HIGH>; 1350 1351 clock-frequency = <48000000>; 1351 - current-speed = <115200>; 1352 1352 power-domains = <&k3_pds 278 TI_SCI_PD_EXCLUSIVE>; 1353 1353 clocks = <&k3_clks 278 0>; 1354 1354 clock-names = "fclk"; ··· 1359 1361 reg = <0x00 0x02820000 0x00 0x100>; 1360 1362 interrupts = <GIC_SPI 194 IRQ_TYPE_LEVEL_HIGH>; 1361 1363 clock-frequency = <48000000>; 1362 - current-speed = <115200>; 1363 1364 power-domains = <&k3_pds 279 TI_SCI_PD_EXCLUSIVE>; 1364 1365 clocks = <&k3_clks 279 0>; 1365 1366 clock-names = "fclk"; ··· 1370 1373 reg = <0x00 0x02830000 0x00 0x100>; 1371 1374 interrupts = <GIC_SPI 195 IRQ_TYPE_LEVEL_HIGH>; 1372 1375 clock-frequency = <48000000>; 1373 - current-speed = <115200>; 1374 1376 power-domains = <&k3_pds 280 TI_SCI_PD_EXCLUSIVE>; 1375 1377 clocks = <&k3_clks 280 0>; 1376 1378 clock-names = "fclk"; ··· 1381 1385 reg = <0x00 0x02840000 0x00 0x100>; 1382 1386 interrupts = <GIC_SPI 196 IRQ_TYPE_LEVEL_HIGH>; 1383 1387 clock-frequency = <48000000>; 1384 - current-speed = <115200>; 1385 1388 power-domains = <&k3_pds 281 TI_SCI_PD_EXCLUSIVE>; 1386 1389 clocks = <&k3_clks 281 0>; 1387 1390 clock-names = "fclk"; ··· 1392 1397 reg = <0x00 0x02850000 0x00 0x100>; 1393 1398 interrupts = <GIC_SPI 197 IRQ_TYPE_LEVEL_HIGH>; 1394 1399 clock-frequency = <48000000>; 1395 - current-speed = <115200>; 1396 1400 power-domains = <&k3_pds 282 TI_SCI_PD_EXCLUSIVE>; 1397 1401 clocks = <&k3_clks 282 0>; 1398 1402 clock-names = "fclk"; ··· 1403 1409 reg = <0x00 0x02860000 0x00 0x100>; 1404 1410 interrupts = <GIC_SPI 198 IRQ_TYPE_LEVEL_HIGH>; 1405 1411 clock-frequency = <48000000>; 1406 - current-speed = <115200>; 1407 1412 power-domains = <&k3_pds 283 TI_SCI_PD_EXCLUSIVE>; 1408 1413 clocks = <&k3_clks 283 0>; 1409 1414 clock-names = "fclk"; ··· 1414 1421 reg = <0x00 0x02870000 0x00 0x100>; 1415 1422 interrupts = <GIC_SPI 199 IRQ_TYPE_LEVEL_HIGH>; 1416 1423 clock-frequency = <48000000>; 1417 - current-speed = <115200>; 1418 1424 power-domains = <&k3_pds 284 TI_SCI_PD_EXCLUSIVE>; 1419 1425 clocks = <&k3_clks 284 0>; 1420 1426 clock-names = "fclk"; ··· 1425 1433 reg = <0x00 0x02880000 0x00 0x100>; 1426 1434 interrupts = <GIC_SPI 248 IRQ_TYPE_LEVEL_HIGH>; 1427 1435 clock-frequency = <48000000>; 1428 - current-speed = <115200>; 1429 1436 power-domains = <&k3_pds 285 TI_SCI_PD_EXCLUSIVE>; 1430 1437 clocks = <&k3_clks 285 0>; 1431 1438 clock-names = "fclk"; ··· 1436 1445 reg = <0x00 0x02890000 0x00 0x100>; 1437 1446 interrupts = <GIC_SPI 249 IRQ_TYPE_LEVEL_HIGH>; 1438 1447 clock-frequency = <48000000>; 1439 - current-speed = <115200>; 1440 1448 power-domains = <&k3_pds 286 TI_SCI_PD_EXCLUSIVE>; 1441 1449 clocks = <&k3_clks 286 0>; 1442 1450 clock-names = "fclk";

-2

arch/arm64/boot/dts/ti/k3-j721e-mcu-wakeup.dtsi

··· 243 243 reg = <0x00 0x42300000 0x00 0x100>; 244 244 interrupts = <GIC_SPI 897 IRQ_TYPE_LEVEL_HIGH>; 245 245 clock-frequency = <48000000>; 246 - current-speed = <115200>; 247 246 power-domains = <&k3_pds 287 TI_SCI_PD_EXCLUSIVE>; 248 247 clocks = <&k3_clks 287 0>; 249 248 clock-names = "fclk"; ··· 254 255 reg = <0x00 0x40a00000 0x00 0x100>; 255 256 interrupts = <GIC_SPI 846 IRQ_TYPE_LEVEL_HIGH>; 256 257 clock-frequency = <96000000>; 257 - current-speed = <115200>; 258 258 power-domains = <&k3_pds 149 TI_SCI_PD_EXCLUSIVE>; 259 259 clocks = <&k3_clks 149 0>; 260 260 clock-names = "fclk";