jflyper / cleanflight

Clean-code version of the baseflight flight controller firmware
http://cleanflight.com
GNU General Public License v3.0
0 stars 1 forks source link

Code tidy and addition of GPL headers #4

Closed martinbudden closed 7 years ago

martinbudden commented 7 years ago

@jflyper , PR includes the following:

  1. Addition of GPL headers
  2. Improved const correctness
  3. Moved CMS registration out of displayPort code - the displayPort should have no knowledge of CMS.
  4. Moved CMS files out of .mk files into makefile - this makes it much easier to add new targets and goes with the general rule that .mk files should only contain device drivers.
  5. Other minor cleanups

Can you merge this in, and then I'll do another PR to create the CMS directory and move the appropriate files into it.

We're almost there now.

jflyper commented 7 years ago

Look ok. I'm going to merge now.

martinbudden commented 7 years ago

Great. So I'm now going to do a PR that renames some files and moves them around. That might be a bit disruptive so I'd hold off doing any changes until I've got that one done.

jflyper commented 7 years ago

Okay, I'll be as still as a 4K old mummy.