Skip to content

merge from 2.2.0 + fix up board defs#510

Merged
dhalbert merged 4 commits into
adafruit:masterfrom
dhalbert:3.0_merge_2.2.0
Jan 5, 2018
Merged

merge from 2.2.0 + fix up board defs#510
dhalbert merged 4 commits into
adafruit:masterfrom
dhalbert:3.0_merge_2.2.0

Conversation

@dhalbert
Copy link
Copy Markdown
Collaborator

@dhalbert dhalbert commented Jan 3, 2018

Merge bugfixes and additions from 2.2.0 to master.

Difficult merge. Checked a number of merges by hand. Non-Adafruit boards added but I did not try to build them.

Merged in 2.2 double-tap support in .ld files (@tannewt: you might double-check these).

CPX SPI flash support added, but not tested.

@dhalbert dhalbert requested a review from tannewt January 3, 2018 02:29
1. bidirectional SPI support (@microbuilder please check)
2. slight changes in DigitalIO types
3. placeholders (not implemented yet) for single/double-click bootloader support
@dhalbert dhalbert requested a review from microbuilder January 3, 2018 21:53
@dhalbert
Copy link
Copy Markdown
Collaborator Author

dhalbert commented Jan 3, 2018

@tannewt Fixed up things that broke the build. @microbuilder, note that there are a few changes in the nrf/ dir to make things compile, and I implemented bidirectional SPI by implementing one new routine (you can check that -- it was a guess based on the the read and write routines).

@jerryneedell
Copy link
Copy Markdown
Collaborator

FYI - I was able to merge this PR into master and build/load the code into my feather52 - Seems to be working as expected.

Copy link
Copy Markdown
Member

@tannewt tannewt left a comment

Choose a reason for hiding this comment

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

One typo and a note about board specific files.

*/
/*
* Support and FAQ: visit <a href="http://www.atmel.com/design-support/">Atmel Support</a>
*/
Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

I don't think all of these conf_* files are needed with ASF4.

Copy link
Copy Markdown
Collaborator Author

Choose a reason for hiding this comment

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

cleaned up (+more).

#LD_FILE = boards/samd21x18-bootloader.ld
USB_VID = 0x239A
USB_PID = 0x801F
USB_PRODUCT = "UGame"
Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

Small u here because its for micro.

Copy link
Copy Markdown
Collaborator Author

Choose a reason for hiding this comment

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

tnx

@dhalbert
Copy link
Copy Markdown
Collaborator Author

dhalbert commented Jan 5, 2018

ready to re-review

Copy link
Copy Markdown
Member

@tannewt tannewt left a comment

Choose a reason for hiding this comment

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

Works for me. Thanks!

@dhalbert dhalbert merged commit 5cb361c into adafruit:master Jan 5, 2018
@dhalbert dhalbert deleted the 3.0_merge_2.2.0 branch January 5, 2018 02:14
@microbuilder
Copy link
Copy Markdown

Hi Dan, sorry just saw the email alert for this. :( Looks good to me though!

@tannewt
Copy link
Copy Markdown
Member

tannewt commented Jan 5, 2018

@microbuilder Feel free to add comments to PRs after they are merged. The author can always follow up with an additional PR.

@dhalbert
Copy link
Copy Markdown
Collaborator Author

dhalbert commented Jan 5, 2018

All atmel-samd builds are going into safe mode on ctrl-D after this merge (tnx @jerryneedell). So some issue somewhere.

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.

4 participants