Easy projects: Difference between revisions

From coreboot
Jump to navigation Jump to search
(→‎Use CBFS wherever possible: Add link to commit changing the location for referenced code.)
 
(4 intermediate revisions by 3 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 ===


* <code>src/cpu/amd/model_lx/vsmsetup.c</code> is a very good candidate. The VSM blobs should be loaded from [[CBFS]]. See [http://review.coreboot.org/gitweb?p=coreboot.git;a=commitdiff;h=9839cbd53fdcfcee52c406d9f52af924192e618d commit 9839cbd53fdcfcee52c406d9f52af924192e618d] for the new location of the code.
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.


=== Formatting and whitespace cleanup ===
=== Adding copyright headers to all files ===


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.
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.


Ideally a check/test should be integrated into the build system pointing out these issues already when committing or testing the commit. Such tests probably already exist in other projects and just need to be copied.
Run 'util/lint/lint-000-license-headers' to see a list of files that still need headers.


== Payloads ==
== Payloads ==
Line 42: 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