search

phytec / doc-bsp-yocto

Yocto BSP Documentation
https://phytec.github.io/doc-bsp-yocto/
1 stars 1 forks source link

BSPIMX8M-3080 BSPIMX8M-3078 Update documentation on uuu and netboot #111

Closed BHahn42618 closed 11 months ago

BHahn42618 commented 11 months ago

Update package dependencies for uuu. Add notice on how to set the required udev rules for uuu. Update the command on how to start the bootloader with uuu. Update the command on how to flash the wic image onto emmc. Add notice to copy the overlays to /tftpboot. Add notice to set the bootenv.txt in /tftpboot.

mschwan-phytec commented 11 months ago

@BHahn42618 Make sure to set your PR as "Ready for review" (or not as a draft), so people know your changes are ready for review and inclusion. Draft PRs are usually comparable to RFCs, so are not considered ready for merging.

tremmet commented 11 months ago

@BHahn42618 Make sure to set your PR as "Ready for review" (or not as a draft), so people know your changes are ready for review and inclusion. Draft PRs are usually comparable to RFCs, so are not considered ready for merging.

@mschwan-phytec Benjamin used the approach we work with in the ptf-lib-dev project. We start with a pull request as draft and start reviewing only the team internally. Without annoying the Maintainers at first. If the team is happy the pull request will be set ready for review for the maintainers also.

mschwan-phytec commented 11 months ago

@BHahn42618 Make sure to set your PR as "Ready for review" (or not as a draft), so people know your changes are ready for review and inclusion. Draft PRs are usually comparable to RFCs, so are not considered ready for merging.

@mschwan-phytec Benjamin used the approach we work with in the ptf-lib-dev project. We start with a pull request as draft and start reviewing only the team internally. Without annoying the Maintainers at first. If the team is happy the pull request will be set ready for review for the maintainers also.

@tremmet Thanks for the explanation! I was confused a little, as @BHahn42618 didn't make previous PRs ready for review after you gave your team-internal reviews. I'll be patient then.

tremmet commented 11 months ago

@s-hemer @mschwan-phytec Please continue review here. We will have more documentation pull requests to finish up documentation for the upcoming release soon. Would be nice if we could avoid conflicts here.