Skip to content

DELTA_DFCM_NNN40 pull request #1002

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

Closed
wants to merge 15 commits into from
Closed

DELTA_DFCM_NNN40 pull request #1002

wants to merge 15 commits into from

Conversation

Marcomissyou
Copy link
Contributor

add rtc_api.c for MBED_16 RTC test and modify mbed_overrides.c to shut down Flash when boot-up.

@0xc0170
Copy link
Contributor

0xc0170 commented Mar 26, 2015

Can you squash all these commits into one?

@Marcomissyou
Copy link
Contributor Author

got it,
I try.
Thanks

@Marcomissyou
Copy link
Contributor Author

Hi 0xc0170,

Sorry, I try and I don't know how to squash all commits into one.
When I rebase, it shows more information to me and I don't know what to do.

I just want to pull request one commit that ID is 6d53a99.
Could you just merge this commit and ignore others?

Or do you have other suggestion?

Thanks

@@ -941,12 +941,6 @@ def __init__(self):
self.core = "Cortex-M0"
self.extra_labels = ['NORDIC', 'MCU_NRF51822', 'MCU_NORDIC_16K', 'DELTA_DFCM_NNN40']
self.MERGE_SOFT_DEVICE = False
class DELTA_DFCM_NNN40_OTA(NRF51822):
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Why did you remove this target?

Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

This target is added by me.
I added twice, so I remove one.
Thanks

@0xc0170
Copy link
Contributor

0xc0170 commented Apr 7, 2015

@Marcomissyou Isn't RTC0 used by soft device? You added and enabled RTC to the platform, but the code is not much clear if it's functional. Is it ?

@Marcomissyou
Copy link
Contributor Author

Yes,
I added rtc to my platform, because I want the rtc item of all basic test is pass.
The quality department in our company asks it pass.
I will not use it.

Thanks
Marco

@0xc0170
Copy link
Contributor

0xc0170 commented Apr 8, 2015

I am going to clean the history here a bit. I just came across one article explaining well a branch and master merge (aka do rebase), http://kristopherwilson.com/2015/02/12/stop-merging-master/

0xc0170 added a commit that referenced this pull request Apr 8, 2015
Delta target - add RTC, bugfixes in mbed_overrides and pinnames
@0xc0170
Copy link
Contributor

0xc0170 commented Apr 8, 2015

I went through this PR (=pull request) history, things are mixed between your master and upstream master. The commits should be changed, as not even cherry-pick is possible to pick relevant commits.

I merged using squash, did some cleaning. Please don't use tabs in the files and look at the changeset you are introducing before do a commit.

Review commit 331dc07.

@0xc0170 0xc0170 closed this Apr 8, 2015
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