Skip to content

Readme - HAL structure#29

Merged
0xc0170 merged 2 commits intoARMmbed:masterfrom
0xc0170:hal-structure
Oct 13, 2015
Merged

Readme - HAL structure#29
0xc0170 merged 2 commits intoARMmbed:masterfrom
0xc0170:hal-structure

Conversation

@0xc0170
Copy link
Contributor

@0xc0170 0xc0170 commented Oct 12, 2015

Resuming #7 as I consider this vital for this repository. This should a good base for porting, or simply understanding the hal hierarchy.

I added target module to the picture, plus fixed some formatting.

@sg- @bremoran @bogdanm @rgrover

@bogdanm
Copy link
Contributor

bogdanm commented Oct 12, 2015

+1 from me, nice work! We needed this.

Copy link
Contributor

Choose a reason for hiding this comment

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

isn't there a objects.h and gpio_object.h also needed?

Copy link
Contributor Author

Choose a reason for hiding this comment

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

objects.h are currently referenced in the above layers, at least in minar, which we can remove, so objects could die and gpio_object.h should not be so then we can remove it. will update once I do proper grep

@sg-
Copy link

sg- commented Oct 12, 2015

Great start but its a bit confusing. Its a partial porting guide and partial document. I think these readmes should be scoped to the repo only and if there is a program for porting or just a repo with submodules that pull all the modules that is where the porting guide would live.

@0xc0170
Copy link
Contributor Author

0xc0170 commented Oct 12, 2015

Its a partial porting guide and partial document.

@sg- it is because there's no porting guide. Once we have one, I imagine this would have less detail (something to be defined later what) and link to a porting guide. If you got better idea, shoot here now. This mbed hal needs some more details than it contains, and I am here to write it :)

@0xc0170
Copy link
Contributor Author

0xc0170 commented Oct 13, 2015

@sg- I'll merge this as it. Please send edits, or create an issue regarding what should be in this readme.

0xc0170 added a commit that referenced this pull request Oct 13, 2015
@0xc0170 0xc0170 merged commit c3262c9 into ARMmbed:master Oct 13, 2015
@0xc0170 0xc0170 deleted the hal-structure branch October 13, 2015 08:02
@0xc0170
Copy link
Contributor Author

0xc0170 commented Oct 13, 2015

Created tracking issue referenced above for "Add a link to a porting guide and do future edits"

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.

5 participants

Comments