-
Notifications
You must be signed in to change notification settings - Fork 89
Add support for LPC54114 & LPC54608 #175
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Conversation
It looks like the |
Signed-off-by: Mahadevan Mahesh <[email protected]>
Signed-off-by: Mahadevan Mahesh <[email protected]>
@bridadan @maclobdell I have changed LPC54114_M4 to LPC54114. I have also included support for LPC54608 in this PR. |
@mmahadevan108 Does the interface firmware enumerate two separate mass storage device drivers - one for each core? We allocated 1054 for this platform. 1055 is available and we can allocate it too, but I want to confirm if to unique IDs are needed. Thanks. |
@maclobdell not sure what our convention is regarding how entries in the database match with the mbed-ls listing. The names in mbedls don't match any entries in the database at the moment, is this a requirement? |
@bridadan it should match. Looks like ARMmbed/mbed-os#4290 has been updated so it matches. This platform has two targets, one for each core. |
It enumerates as 1. Initially I thought I would make it as 2 separate devices. |
the updates match the system. I approve. Thanks. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM
Signed-off-by: Mahadevan Mahesh [email protected]