From f293d21863c5b23dcb4b151c2fd879696830e0bc Mon Sep 17 00:00:00 2001 From: Atish Patra Date: Mon, 21 Jan 2019 00:01:23 -0800 Subject: [PATCH] docs: Typo fixes. Signed-off-by: Atish Patra --- README.md | 8 ++++---- docs/contributing.md | 12 ++++++------ docs/firmware/fw_jump.md | 14 ++++++++------ docs/firmware/fw_payload.md | 8 ++++---- 4 files changed, 22 insertions(+), 20 deletions(-) diff --git a/README.md b/README.md index 1e0152d..f63f202 100644 --- a/README.md +++ b/README.md @@ -14,7 +14,7 @@ by the RISC-V Foundation in [Github](https://github.com/riscv/riscv-sbi-doc) OpenSBI aims to provides an open-source and extensible implementation of the RISC-V SBI specification for case 1 mentioned above. OpenSBI implementation can be easily extended by RISC-V platform or System-on-Chip -vendors to fit a particular hadware configuration. +vendors to fit a particular hardware configuration. OpenSBI provides three components: @@ -99,7 +99,7 @@ license text is included in those source files. Documentation ------------- -A more detailed documenation is under the *docs* directory and organized +A more detailed documentation is under the *docs* directory and organized as follows. * *`docs/contributing.md`* - Guidelines for contributing to OpenSBI project @@ -119,7 +119,7 @@ The Doxygen can be installed on your debian build system using following: sudo apt-get install doxygen doxygen-latex doxygen-doc doxygen-gui graphviz ``` -To build a consolidated `refman.pdf` of all documenation, run: +To build a consolidated `refman.pdf` of all documentation, run: ``` make docs or @@ -128,7 +128,7 @@ make O= docs **NOTE:** `refman.pdf` will be available under `/docs/latex`. -To install a consolidated `refman.pdf` of all documenation, run: +To install a consolidated `refman.pdf` of all documentation, run: ``` make install_docs or diff --git a/docs/contributing.md b/docs/contributing.md index 6f0de1b..a17c548 100644 --- a/docs/contributing.md +++ b/docs/contributing.md @@ -1,13 +1,13 @@ OpenSBI Contributing Guidelines =============================== -All contributions to OpenSBI should be sent as GitHub Pull Requests (PRs) to the main repo avaliable at: https://github.com/riscv/opensbi +All contributions to OpenSBI should be sent as GitHub Pull Requests (PRs) to the main repository avaliable at: https://github.com/riscv/opensbi As well as that all work must follow the following rules: 1. We follow the [Linux coding style](https://www.kernel.org/doc/html/v4.10/process/coding-style.html). This means that ALL patches must include the Signed-off-by line. -2. Commit subject must have atleast one ":" prefix. Common ones are "lib:", "platform:", "firmware:", "docs:" and "top:". -3. Maintainer should prefer "Rebase and Merge" when using GitHub to merge pull requests. Avoid create extra merge commits. -4. Maintainer should avoid creating branches directly in main riscv/opensbi repo. Instead prefer fork of GitHub riscv/opensbi repo and create branches in your own fork for pull requests. +2. Commit subject must have at least one ":" prefix. Common ones are "lib:", "platform:", "firmware:", "docs:" and "top:". +3. Maintainer should use "Rebase and Merge" when using GitHub to merge pull requests to avoid unnecessary merge commits. +4. Maintainer should avoid creating branches directly in main riscv/opensbi repository. Instead prefer fork of GitHub riscv/opensbi repository and create branches in your own fork for pull requests. 5. Maintainer cannot merge his own pull requests on GitHub. -6. A pull request must get atleast one review from relevant maintainer. -7. A pull request must spend atleast 24 hours in review so that everyone gets a chance to review. +6. A pull request must get at least one review from relevant maintainer. +7. A pull request must spend at least 24 hours in review so that everyone gets a chance to review. diff --git a/docs/firmware/fw_jump.md b/docs/firmware/fw_jump.md index d45f926..c776aa9 100644 --- a/docs/firmware/fw_jump.md +++ b/docs/firmware/fw_jump.md @@ -23,15 +23,17 @@ Config Options -------------- We need more config details for **FW_JUMP** firmware to work -correctly. These config details can be passed as paramter to +correctly. These configuration details can be passed as parameters to top-level `make` or can be set in platform `config.mk`. -Following are the config options for **FW_JUMP** firmware: +Following are the configuration options for a **FW_JUMP** firmware: * **FW_JUMP_ADDR** - Address where next booting stage is located. This is a mandatory config option and will result in compile error if not provided. -* **FW_JUMP_FDT_ADDR** - Address where FDT passed by prior -booting stage will be placed before passing to next booting -stage. If this option is not provided then firmware will pass -zero as FDT address to next booting stage. + +* **FW_JUMP_FDT_ADDR** - Address where the FDT passed by the prior +booting stage will be placed before passing to the next booting +stage. If this option is not provided then the firmware will pass +zero as the FDT address to the next booting stage. +>>>>>>> 2c0dc4dc... docs: Typo fixes. diff --git a/docs/firmware/fw_payload.md b/docs/firmware/fw_payload.md index aff254d..2dee296 100644 --- a/docs/firmware/fw_payload.md +++ b/docs/firmware/fw_payload.md @@ -26,8 +26,8 @@ Configuration Options --------------------- A *FW_PAYLOAD* firmware needs to be built according to some predefined -configuation options to work correctly. These configuration details can -be passed as paramters to the top-level `make` command or can be defined +configuration options to work correctly. These configuration details can +be passed as parameters to the top-level `make` command or can be defined in a platform *config.mk* build configuration file. The following are the build configuration parameters for a *FW_PAYLOAD* @@ -39,10 +39,10 @@ binary will be linked in the final *FW_PAYLOAD* firmware binary image. This configuration parameter is mandatory if *FW_PAYLOAD_ALIGN* is not defined. Compilation errors will result from an incorrect definition of *FW_PAYLOAD_OFFSET* or *FW_PAYLOAD_ALIGN*, or if neither of these -paramreters are defined. +parameters are defined. * **FW_PAYLOAD_ALIGN** - Address alignment constraint where the payload -binary will be linked after the end of the base firmaware binary in the +binary will be linked after the end of the base firmware binary in the final *FW_PAYLOAD* firmware binary image. This configuration parameter is mandatory if *FW_PAYLOAD_OFFSET* is not defined and should not be defined otherwise.