nuttx icon indicating copy to clipboard operation
nuttx copied to clipboard

Replace PL011 generic driver with utilities that can be used by driver developers

Open linguini1 opened this issue 1 year ago • 1 comments

Summary

Replace PL011 generic driver with utilities that can be used by driver developers to implement as many or as few PL011 devices as they require.

This PR closes #12901

Impact

This change allows boards to re-use the PL011 UART driver implementation in their own drivers but configure as many or as few UART interfaces/devices as they please. It also allows other board-specific configuration logic to have an impact on the PL011 device set up.

Testing

All of the changes were tested for compilation without error, with the exception of the cxd32xx board as it has no configuration to initially test.

I also tested the QEMU and FVP "boards" in their respective hypervisors using the guides provided in the NuttX documentation to verify that changes to the driver would not break existing functionality. All of the *:nsh defconfigs for these boards worked.

I was unable to test the goldfish implementations (although the changes are the exact same as QEMU) and the cxd32xx board because I do not own the board (nor is there a configuration to test).

linguini1 avatar Aug 21 '24 02:08 linguini1

I am not quite sure why the block comments are reported as different lengths; I've verified their lengths are all consistent. I managed to resolve all the other block comments are different lengths errors except this one.

linguini1 avatar Aug 21 '24 02:08 linguini1