Hi,
I have created a patch for kinetis port. The patch updates linker scripts (is fully compatible with actual trunk), updates chconf.h in Kinetis demos too and adds linker script e CMSIS file for K64F.
Attached a runnable demo for K64F which don't use HAL.
I hope it could be helpful somehow.
Ciao,
RM
A patch for Linker Script
Moderator: utzig
- RoccoMarco
- Posts: 655
- Joined: Wed Apr 24, 2013 4:11 pm
- Location: Munich (Germany)
- Has thanked: 83 times
- Been thanked: 67 times
- Contact:
A patch for Linker Script
- Attachments
-
- patchKinetis.7z
- (33.31 KiB) Downloaded 348 times
-
- RT-ARMCM4-GENERIC_K64.7z
- (8.56 KiB) Downloaded 325 times
-
- Posts: 359
- Joined: Sat Jan 07, 2012 6:22 pm
- Location: Brazil
- Has thanked: 1 time
- Been thanked: 20 times
- Contact:
Re: A patch for Linker Script
I would prefer you send me pull requests on Github as soon as I move the repo. Due to some unexpected events I was not able to do it on last weekend but I'll find some time till next week.
Please also break the patch in fixes to current demos and new demos.
Cheers,
Fabio Utzig
Please also break the patch in fixes to current demos and new demos.
Cheers,
Fabio Utzig
- RoccoMarco
- Posts: 655
- Joined: Wed Apr 24, 2013 4:11 pm
- Location: Munich (Germany)
- Has thanked: 83 times
- Been thanked: 67 times
- Contact:
Re: A patch for Linker Script
Hi. And what about the svn? Will it remain out of date in this way?
Ciao,
RM
Ciao,
RM
Last edited by RoccoMarco on Wed Mar 02, 2016 1:39 am, edited 1 time in total.
-
- Posts: 359
- Joined: Sat Jan 07, 2012 6:22 pm
- Location: Brazil
- Has thanked: 1 time
- Been thanked: 20 times
- Contact:
Re: A patch for Linker Script
When migration is done, there will be no reason to have a duplicated port so better remove it.
- RoccoMarco
- Posts: 655
- Joined: Wed Apr 24, 2013 4:11 pm
- Location: Munich (Germany)
- Has thanked: 83 times
- Been thanked: 67 times
- Contact:
Re: A patch for Linker Script
In my opinion remove the port from svn will create a lot of confusion in users. I think we should review the code trying to reach the high standards Giovanni was talking about.
-
- Posts: 359
- Joined: Sat Jan 07, 2012 6:22 pm
- Location: Brazil
- Has thanked: 1 time
- Been thanked: 20 times
- Contact:
Re: A patch for Linker Script
High standards and SVN are probably not correlated (but maybe inversely correlated, excluding some like ChibiOS). We have much better review options on Github than having to download 7zip files, unzip, check patch, comment on a forum. If people don't maintain the "high standards" when contributing is because people come and go which is normal for people who are not deriving financial rewards from the work. That is the dynamics of open source communities. About the one mentioned thread I prefer to not comment anymore because I don't want to generate entropy.
- RoccoMarco
- Posts: 655
- Joined: Wed Apr 24, 2013 4:11 pm
- Location: Munich (Germany)
- Has thanked: 83 times
- Been thanked: 67 times
- Contact:
Re: A patch for Linker Script
Indeed, you are right. Contributors are free to follow their standards according to their will and spare time.
I understand perfectly both the argumentations. I think community repo would be great for contributions. Nevertheless, it is a beautiful think that some code can be accepted into the main repository.
My proposal is that a Kinetis branch should be created into the Community Repo but I would like to spent some time to check code and import it into the main repo (trying to follow ChibiOS standards). Of course this idea requires an approval from Giovanni and community which is working so hard and bravely to offer some working support for (previously) unsupporded platforms. No one takes your efforts for granted (and no one should do this) but I neither think that ChibiOS is born to follow commercial purposes.
In my personal opinion, ChibiOS is a "philosophy", an elegant way to write beautiful code. I would to respect this ideal, even more I would respect people which are working on this project (for free) from much more time than me and I wouldn't create entropy.
Sincerely,
RM
I understand perfectly both the argumentations. I think community repo would be great for contributions. Nevertheless, it is a beautiful think that some code can be accepted into the main repository.
My proposal is that a Kinetis branch should be created into the Community Repo but I would like to spent some time to check code and import it into the main repo (trying to follow ChibiOS standards). Of course this idea requires an approval from Giovanni and community which is working so hard and bravely to offer some working support for (previously) unsupporded platforms. No one takes your efforts for granted (and no one should do this) but I neither think that ChibiOS is born to follow commercial purposes.
In my personal opinion, ChibiOS is a "philosophy", an elegant way to write beautiful code. I would to respect this ideal, even more I would respect people which are working on this project (for free) from much more time than me and I wouldn't create entropy.
Sincerely,
RM
-
- Posts: 71
- Joined: Sat Aug 22, 2015 1:22 pm
Re: A patch for Linker Script
It seems to me that the main problem with moving ports to the Contrib repo is about "image". People apparently consider Contrib to be "worse" than the main SVN, and also not really a part of ChibiOS.
This is a PR issue, and I think it should be treated as such. The code standards depend only on the people having push rights to the repo (which is I think the same set of people as for the main SVN).
I think it is only beneficial for the project as a whole if it will be easier for random people from the community to contribute code to ports/drivers which are not maintained full-time by Giovanni. (Submitting/reviewing patches and making bug reports through the forums is a bit of a pain in the backside.)
This is a PR issue, and I think it should be treated as such. The code standards depend only on the people having push rights to the repo (which is I think the same set of people as for the main SVN).
I think it is only beneficial for the project as a whole if it will be easier for random people from the community to contribute code to ports/drivers which are not maintained full-time by Giovanni. (Submitting/reviewing patches and making bug reports through the forums is a bit of a pain in the backside.)
-
- Posts: 359
- Joined: Sat Jan 07, 2012 6:22 pm
- Location: Brazil
- Has thanked: 1 time
- Been thanked: 20 times
- Contact:
Re: A patch for Linker Script
I "moved" the Kinetis port to ChibiOS-Contrib. The pull request is here: https://github.com/ChibiOS/ChibiOS-Contrib/pull/45
I think I got everything right but the only way to really be sure would be removing the Kinetis port from main repo! But I updated the makefiles, configs, cmsis related data, etc. I only added two demos for now one for freedom KL25Z and one for freedom K20D50 and I only tested K20D50 but will test KL25Z later today.
Cheers,
Fabio Utzig
I think I got everything right but the only way to really be sure would be removing the Kinetis port from main repo! But I updated the makefiles, configs, cmsis related data, etc. I only added two demos for now one for freedom KL25Z and one for freedom K20D50 and I only tested K20D50 but will test KL25Z later today.
Cheers,
Fabio Utzig
-
- Posts: 359
- Joined: Sat Jan 07, 2012 6:22 pm
- Location: Brazil
- Has thanked: 1 time
- Been thanked: 20 times
- Contact:
Re: A patch for Linker Script
So guys, how about we finish this process and start updating the contrib repo with extra work available on personal repos?
Who is online
Users browsing this forum: No registered users and 3 guests