Easy projects: Difference between revisions

From coreboot
Jump to navigation Jump to search
m (→‎Formatting and whitespace cleanup: Remove incorrect third person »s«)
 
(6 intermediate revisions by 4 users not shown)
Line 5: Line 5:
== coreboot ==
== coreboot ==


=== Formatting and whitespace cleanup ===


=== AMD 740G information gathering ===
We try to maintain the code in the [[Development_Guidelines#Coding_Style Linux style]], but occasionally white-space and other formatting issues find their way into the project. Formatting and white-space changes should be done in small groups as a separate patch from code changes. Be careful running indent/lindent. The results are not always the right thing to do and require review.
 
(This project description is not finished yet)
 
If you have a board with AMD 740G chipset, please run (as root)
 
$ '''flashrom -V'''
$ '''lspci -nnvvvxxxx'''
$ '''superiotool -deV'''
$ '''dmidecode'''
 
and mail the output to the [[Mailinglist|coreboot mailing list]] together with the exact model number/name of your board.
 
This helps us evaluate which boards are good targets for coreboot.
 
Here are some boards:
 
* http://www.czechcomputer.cz/cat_tree.jsp?bpath=Z%C3%A1kladn%C3%AD+desky\Socket+AM2%2B\AMD+740G
 
=== Use CBFS wherever possible ===


* '''src/cpu/amd/model_lx/vsmsetup.c''' is a very good candidate. The VSM blobs should be loaded from [[CBFS]].
Run 'util/lint/lint-007-checkpatch' to see a list of issues that need to be addressed. Note that these should probably be addressed in groups - Handle a group of LEADING_SPACE issues or SPACING issues together.


=== Adding copyright headers to all files ===


=== Formatting and whitespace cleanup ===
We want to have copyright headers on all .c, .h, and .asl files (and maybe others as well, but if we had them in all of these files, that would be great.)  The list of files that don't have headers can be seen by running the util/lint/lint-000-license-headers script from the coreboot directory.  To add a header to the file, you need to verify the origin of the file, so you should run a git log on it.  There may also already be information at the top of the file about what kind of license should be used.  If the file is original to coreboot, it should get the standard coreboot license header.  If it has some other license, the appropriate header should be added.


We try to maintain the code in the [[Development_Guidelines#Coding_Style Linux style]], but occasionally white-space and other formatting issues find their way into the project. Formatting and white-space changes should be done in small groups as a separate patch from code changes. Be careful running indent/lindent. The results are not always the right thing to do and require review.
Run 'util/lint/lint-000-license-headers' to see a list of files that still need headers.


== Payloads ==
== Payloads ==
Line 41: Line 24:


* Test syslinux (probably requires [[SeaBIOS]] in addition, needs to be checked).
* Test syslinux (probably requires [[SeaBIOS]] in addition, needs to be checked).
* Port [[GPXE]] to "native" coreboot (it works fine together with [[SeaBIOS]] though).


== flashrom ==
== flashrom ==


The [[flashrom]] tool can read/write coreboot/BIOS images from/to flash chips.
The [http://www.flashrom.org flashrom] tool can read/write coreboot/BIOS images from/to flash chips.


* See [http://flashrom.org/Easy_projects flashrom's Easy Projects] list for details.
* See [http://flashrom.org/Easy_projects flashrom's Easy Projects] list for details.

Latest revision as of 20:13, 20 January 2017

You probably came here trying to find a small (minutes to hours) and easy task where you can get your hands dirty and get results immediately.

If you're a coreboot or flashrom newbie, this page is for you.

coreboot

Formatting and whitespace cleanup

We try to maintain the code in the Development_Guidelines#Coding_Style Linux style, but occasionally white-space and other formatting issues find their way into the project. Formatting and white-space changes should be done in small groups as a separate patch from code changes. Be careful running indent/lindent. The results are not always the right thing to do and require review.

Run 'util/lint/lint-007-checkpatch' to see a list of issues that need to be addressed. Note that these should probably be addressed in groups - Handle a group of LEADING_SPACE issues or SPACING issues together.

Adding copyright headers to all files

We want to have copyright headers on all .c, .h, and .asl files (and maybe others as well, but if we had them in all of these files, that would be great.) The list of files that don't have headers can be seen by running the util/lint/lint-000-license-headers script from the coreboot directory. To add a header to the file, you need to verify the origin of the file, so you should run a git log on it. There may also already be information at the top of the file about what kind of license should be used. If the file is original to coreboot, it should get the standard coreboot license header. If it has some other license, the appropriate header should be added.

Run 'util/lint/lint-000-license-headers' to see a list of files that still need headers.

Payloads

coreboot can use a number of different payloads.

Add/test new supported payloads

  • Test syslinux (probably requires SeaBIOS in addition, needs to be checked).

flashrom

The flashrom tool can read/write coreboot/BIOS images from/to flash chips.

Other