Since the fix and merge of #621 happened so fast, I was able to test it today. Sadly, it does not work. This time the results are different - no error happens, but console output just plain stops after some diagnostic messages from the driver.
This is the last of what I see:
[ 1.198118] 5a060000.serial: ttyLP0 at MMIO 0x5a060010 (irq = 74, base_baud = 5000000) is a FSL_LPUART
[ 1.208224] 5a070000.serial: ttyLP1 at MMIO 0x5a070010 (irq = 75, base_baud = 5000000) is a FSL_LPUART
I have no clue as to why this happens. What fixes this for me is moving uart_add_one_port() before lpuart_global_reset(). So, where before #621 there were two calls to add_one_port(), and the patch removed the first one while reordering cleanup, what works for me is removing the second call and leaving cleanup as-is.
If necessary, I can create a PR later tonight.
cc @otavio @MrCry0
Since the fix and merge of #621 happened so fast, I was able to test it today. Sadly, it does not work. This time the results are different - no error happens, but console output just plain stops after some diagnostic messages from the driver.
This is the last of what I see:
I have no clue as to why this happens. What fixes this for me is moving
uart_add_one_port()beforelpuart_global_reset(). So, where before #621 there were two calls toadd_one_port(), and the patch removed the first one while reordering cleanup, what works for me is removing the second call and leaving cleanup as-is.If necessary, I can create a PR later tonight.
cc @otavio @MrCry0