Skip to content

Modified the TARGET_GHI_MBUINO/PeripheralNames.h and targets/hal/TARGET_... #422

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

Merged
merged 5 commits into from
Aug 6, 2014

Conversation

Experiment-626
Copy link
Contributor

...NXP/TARGET_LPC11UXX/TARGET_GHI_MBUINO/PinNames.h to reflect the mBuino base hardware;

I commented out the non useful SPI pins and Serial pins from our hardware build;
For pwmout, I removed the non useful pins and added two more pins that are part of our hardware build.

As this is my first time making a platform, I am unsure if the changes I made are proper. Please advise to any errors I may have made.

…ET_NXP/TARGET_LPC11UXX/TARGET_GHI_MBUINO/PinNames.h to reflect the mBuino base hardware;

I commented out the non useful SPI pins and Serial pins from our hardware build;
For pwmout, I removed the non useful pins and added two more pins that are part of our hardware build.
@mbednotifications
Copy link

The changes to pwmout_api.c serial_api.c and spi_api.c aren't go to work as
they'll break compatibility on other boards that have those pins available.
Have a look at how this is handled for different targets. I think Erik O.
implemented this originally and it seems to scale quite well.

https://github.com/mbedmicro/mbed/blob/master/libraries/mbed/targets/hal/TARGET_Freescale/TARGET_KLXX/TARGET_KL46Z/PeripheralPins.c

You need to remove all pin to peripheral definitions from the xxx_api.c
file and add to the platform specific folders as PeripheralPins.c. This
needs to be done for all platforms that use these peripheral sources. Then
create TARGET_LPC11UXX/PeripheralPins.h and include where necessary.

-s

On Wed, Jul 30, 2014 at 12:58 PM, Aron L. Phillips <[email protected]

wrote:

...NXP/TARGET_LPC11UXX/TARGET_GHI_MBUINO/PinNames.h to reflect the mBuino
base hardware;

I commented out the non useful SPI pins and Serial pins from our hardware
build;
For pwmout, I removed the non useful pins and added two more pins that are
part of our hardware build.

As this is my first time making a platform, I am unsure if the changes I

made are proper. Please advise to any errors I may have made.

You can merge this Pull Request by running

git pull https://github.com/Experiment-626/mbed master

Or view, comment on, or merge it at:

#422
Commit Summary

  • Modified the TARGET_GHI_MBUINO/PeripheralNames.h and
    targets/hal/TARGET_NXP/TARGET_LPC11UXX/TARGET_GHI_MBUINO/PinNames.h to
    reflect the mBuino base hardware;

File Changes

Patch Links:


Reply to this email directly or view it on GitHub
#422.

You received this message because you are subscribed to the Google Groups
"mbed-devel" group.
To unsubscribe from this group and stop receiving emails from it, send an
email to [email protected].
For more options, visit https://groups.google.com/d/optout.

…hal/TARGET_NXP/TARGET_LPC11UXX/TARGET_GHI_MBUINO/PinNames.h to reflect the mBuino base hardware;"

This reverts commit 6b910f2.
…ogin, i2c, pwmout, serial, & spi) to follow an example from Erik O. from this platform: https://github.com/mbedmicro/mbed/blob/master/libraries/mbed/targets/hal/TARGET_Freescale/TARGET_KLXX/TARGET_KL46Z/PeripheralPins.c to allow for advanced compatibility of platforms;

Subsequently, the Peripheral Pin Definitions were added to the file called PeripheralPins.c to accommodate the removal of the definitions from the main xxx_api.c files and added it to each respective Target folder;
Added PeripheralPin.h to point to the PeripheralPin.c files in the target directories.
@Experiment-626
Copy link
Contributor Author

I made the changes as recommended and modified all of the target platforms to accommodate the changes. It will be in another pull request. Without the changes made to the other platforms, the other six would have been broken too.

…commodate for the specific chip that the mBuino platform uses LPC11U24FHI33/301 which is 6kB in the SRAM vs 8kB;

Added ('GHI_MBUINO',     ('ARM', 'uARM', 'GCC_ARM')), to the OFFICIAL_MBED_LIBRARY_BUILD section of build_release.py.
@Experiment-626
Copy link
Contributor Author

I worked with Sam Grove to make sure that all of the changes had been correct. I am awaiting final approval.

@0xc0170
Copy link
Contributor

0xc0170 commented Aug 4, 2014

Hello,

can you remove the commented PinMap definitions from HAL files?
Please add this target to build_travis script as it supports GCC ARM.

Can you please sign the mbed contributor agreement at https://mbed.org/contributor_agreement ?

Regards,
0xc0170

Removed the pin definition comments from the xxx_api.c files;
Added GHI_MBUINO to the build_travis.py target section.
@Experiment-626
Copy link
Contributor Author

PinMap definition comments have been removed and the build_travis.py file was updated with the GHI_MBUINO target.

@0xc0170
Copy link
Contributor

0xc0170 commented Aug 5, 2014

thanks,

have you signed the mbed contributor agreement? What's your nick on mbed.org?

@Experiment-626
Copy link
Contributor Author

Greetings Martin,

Yes, I did agree to the contributor's agreement. My nickname is Experiment626 on mbed.org and Experiment-626 on github.

Sincerely,

Aron

Sent via the Samsung GALAXY S® 5, an AT&T 4G LTE smartphone

-------- Original message --------
From: Martin Kojtal [email protected]
Date:08/05/2014 1:55 AM (GMT-05:00)
To: mbedmicro/mbed [email protected]
Cc: "Aron L. Phillips" [email protected]
Subject: Re: [mbed] Modified the TARGET_GHI_MBUINO/PeripheralNames.h and
targets/hal/TARGET_... (#422)

thanks,

have you signed the mbed contributor agreement? What's your nick on mbed.org?


Reply to this email directly or view it on GitHub.

0xc0170 added a commit that referenced this pull request Aug 6, 2014
[TARGET_LPC11UXX] PeripheralNames.h and PinMap definitions separation for LPC11UXX platforms
@0xc0170 0xc0170 merged commit 5b145e4 into ARMmbed:master Aug 6, 2014
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants