-
Notifications
You must be signed in to change notification settings - Fork 4
Non intrusive changes to get some SAMD board working. #1
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
base: master
Are you sure you want to change the base?
Changes from all commits
ebf8533
cfd7917
564529d
d431370
b238ef8
5bf56a4
4857f9a
e81bcd8
061cfc5
3876d93
7744bd8
dff958d
File filter
Filter by extension
Conversations
Jump to
Diff view
Diff view
There are no files selected for viewing
| Original file line number | Diff line number | Diff line change |
|---|---|---|
|
|
@@ -73,33 +73,31 @@ | |
| <CompilerOptions> | ||
| <Version Value="11"/> | ||
| <Target> | ||
| <Filename Value="Blinky"/> | ||
| <Filename Value="bin/$(TargetCPU)-$(TargetOS)/samc21/$NameOnly($(ProjFile))"/> | ||
| </Target> | ||
| <SearchPaths> | ||
| <IncludeFiles Value="../../Source;$(ProjOutDir)"/> | ||
| <OtherUnitFiles Value="../../Source"/> | ||
| <UnitOutputDirectory Value="lib/$(TargetCPU)-$(TargetOS)"/> | ||
| <UnitOutputDirectory Value="lib/$(TargetCPU)-$(TargetOS)/samc21"/> | ||
| </SearchPaths> | ||
| <Parsing> | ||
| <SyntaxOptions> | ||
| <UseAnsiStrings Value="False"/> | ||
| </SyntaxOptions> | ||
| </Parsing> | ||
| <CodeGeneration> | ||
| <HeapSize Value="8192"/> | ||
|
Owner
There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. Is there a reason why you removed Heap/StackSize Values?
Contributor
Author
There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. Not anything particular.
Owner
There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. I am too lazy to look this up in details atm but I guess your code will sooner or later crash and burn when those are not properly set. But it may be that some defaults are applied when nothing is set, as said, too lazy to look this up atm. |
||
| <StackSize Value="8192"/> | ||
| <TargetCPU Value="arm"/> | ||
| <TargetOS Value="embedded"/> | ||
| </CodeGeneration> | ||
| <Linking> | ||
| <Debugging> | ||
| <GenerateDebugInfo Value="False"/> | ||
|
Owner
There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. Also here, any specific reason for this? Having Debug Info will not change the size of the deployed code, so why turn DebugInfo off?
Contributor
Author
There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. Again, not anything particular. |
||
| <DebugInfoType Value="dsDwarf2"/> | ||
| <UseLineInfoUnit Value="False"/> | ||
| </Debugging> | ||
| </Linking> | ||
| <Other> | ||
| <CustomOptions Value="-Cparmv6m | ||
| -XP/usr/local/bin/arm-none-eabi- | ||
| -Wpsamc21xpro | ||
| -dsamc21xpro"/> | ||
| </Other> | ||
|
|
||
| Original file line number | Diff line number | Diff line change |
|---|---|---|
| @@ -0,0 +1,116 @@ | ||
| <?xml version="1.0" encoding="UTF-8"?> | ||
|
Owner
There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. Same comments here as for the other lpi file, or did I overlook something?
Contributor
Author
There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. Correct: you did not overlook anything ... ;-) |
||
| <CONFIG> | ||
| <ProjectOptions> | ||
| <Version Value="11"/> | ||
| <General> | ||
| <Flags> | ||
| <LRSInOutputDirectory Value="False"/> | ||
| </Flags> | ||
| <MainUnit Value="0"/> | ||
| <UseAppBundle Value="False"/> | ||
| </General> | ||
| <BuildModes Count="1"> | ||
| <Item1 Name="default" Default="True"/> | ||
| </BuildModes> | ||
| <PublishOptions> | ||
| <Version Value="2"/> | ||
| </PublishOptions> | ||
| <RunParams> | ||
| <FormatVersion Value="2"/> | ||
| <Modes Count="1"> | ||
| <Mode0 Name="default"/> | ||
| </Modes> | ||
| </RunParams> | ||
| <Units Count="5"> | ||
| <Unit0> | ||
| <Filename Value="Blinky.lpr"/> | ||
| <IsPartOfProject Value="True"/> | ||
| <IsVisibleTab Value="True"/> | ||
| <TopLine Value="14"/> | ||
| <CursorPos Y="23"/> | ||
| <UsageCount Value="23"/> | ||
| <Loaded Value="True"/> | ||
| <LoadedDesigner Value="True"/> | ||
| </Unit0> | ||
| <Unit1> | ||
| <Filename Value="../../Source/MBF.SAMCD.SystemCore.pas"/> | ||
| <EditorIndex Value="2"/> | ||
| <TopLine Value="205"/> | ||
| <CursorPos X="33" Y="245"/> | ||
| <UsageCount Value="10"/> | ||
| <Loaded Value="True"/> | ||
| </Unit1> | ||
| <Unit2> | ||
| <Filename Value="../../Source/MBF.Boards.SAMD.inc"/> | ||
| <EditorIndex Value="3"/> | ||
| <TopLine Value="13"/> | ||
| <CursorPos X="22" Y="30"/> | ||
| <UsageCount Value="10"/> | ||
| <Loaded Value="True"/> | ||
| </Unit2> | ||
| <Unit3> | ||
| <Filename Value="../../Source/MBF.SAMCD.GPIO.pas"/> | ||
| <EditorIndex Value="1"/> | ||
| <TopLine Value="89"/> | ||
| <CursorPos X="38" Y="308"/> | ||
| <UsageCount Value="10"/> | ||
| <Loaded Value="True"/> | ||
| </Unit3> | ||
| <Unit4> | ||
| <Filename Value="../../Source/MBF.Config.inc"/> | ||
| <EditorIndex Value="4"/> | ||
| <TopLine Value="51"/> | ||
| <CursorPos X="22" Y="72"/> | ||
| <UsageCount Value="10"/> | ||
| <Loaded Value="True"/> | ||
| </Unit4> | ||
| </Units> | ||
| <JumpHistory Count="3" HistoryIndex="2"> | ||
| <Position1> | ||
| <Filename Value="Blinky.lpr"/> | ||
| <Caret Line="39" Column="61"/> | ||
| </Position1> | ||
| <Position2> | ||
| <Filename Value="../../Source/MBF.Boards.SAMD.inc"/> | ||
| <Caret Line="4"/> | ||
| </Position2> | ||
| <Position3> | ||
| <Filename Value="../../Source/MBF.Config.inc"/> | ||
| <Caret Line="72" Column="22" TopLine="51"/> | ||
| </Position3> | ||
| </JumpHistory> | ||
| </ProjectOptions> | ||
| <CompilerOptions> | ||
| <Version Value="11"/> | ||
| <Target> | ||
| <Filename Value="bin/$(TargetCPU)-$(TargetOS)/samd10/$NameOnly($(ProjFile))"/> | ||
| </Target> | ||
| <SearchPaths> | ||
| <IncludeFiles Value="../../Source;$(ProjOutDir)"/> | ||
| <OtherUnitFiles Value="../../Source"/> | ||
| <UnitOutputDirectory Value="lib/$(TargetCPU)-$(TargetOS)/samd10"/> | ||
| </SearchPaths> | ||
| <Parsing> | ||
| <SyntaxOptions> | ||
| <UseAnsiStrings Value="False"/> | ||
| </SyntaxOptions> | ||
| </Parsing> | ||
| <CodeGeneration> | ||
| <HeapSize Value="8192"/> | ||
| <StackSize Value="8192"/> | ||
| <TargetCPU Value="arm"/> | ||
| <TargetOS Value="embedded"/> | ||
| </CodeGeneration> | ||
| <Linking> | ||
| <Debugging> | ||
| <DebugInfoType Value="dsDwarf2"/> | ||
| <UseLineInfoUnit Value="False"/> | ||
| </Debugging> | ||
| </Linking> | ||
| <Other> | ||
| <CustomOptions Value="-Cparmv6m | ||
| -Wpsamd10xmini | ||
| -dsamd10xmini"/> | ||
| </Other> | ||
| </CompilerOptions> | ||
| </CONFIG> | ||
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.
I guess you did this because you had issues with build.
There are two problems here:
a) As we heavily use defines it is still unsave to do a simple build, better do a rebuild all from time to time
b) If we stay with this we will need a generic way to define the name of the 'samc21' directory because I also need to define the path to the executable for Debugging (Ozone)
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.
No build issues.
I always use a single lpi for multiple targets.
And use "Compile many Modes" to generate all binaries at once.
And use the paths settings to split libraries and binaries for easy comparison.