Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

[REBASE && FF] Fix DeviceStateLib Formatting and Move Unit Test Mode to Fixed PCD #542

Commits on Aug 31, 2023

  1. Update DeviceStateLib Formatting to Match Coding Standards

    Description
    
    - [ ] Impacts functionality?
      - **Functionality** - Does the change ultimately impact how firmware functions?
      - Examples: Add a new library, publish a new PPI, update an algorithm, ...
    - [ ] Impacts security?
      - **Security** - Does the change have a direct security impact on an application,
        flow, or firmware?
      - Examples: Crypto algorithm change, buffer overflow fix, parameter
        validation improvement, ...
    - [ ] Breaking change?
      - **Breaking change** - Will anyone consuming this change experience a break
        in build or boot behavior?
      - Examples: Add a new library class, move a module to a different repo, call
        a function in a new library class in a pre-existing module, ...
    - [ ] Includes tests?
      - **Tests** - Does the change include any explicit test code?
      - Examples: Unit tests, integration tests, robot tests, ...
    - [ ] Includes documentation?
      - **Documentation** - Does the change contain explicit documentation additions
        outside direct code modifications (and comments)?
      - Examples: Update readme file, add feature readme file, link to documentation
        on an a separate Web page, ...
    
    How This Was Tested
    
    N/A
    
    Integration Instructions
    
    N/A
    TaylorBeebe committed Aug 31, 2023
    Configuration menu
    Copy the full SHA
    b2b8b05 View commit details
    Browse the repository at this point in the history
  2. Replace Unit Test Device State with Fixed PCD

    Description
    
    DeviceStateLib provides an interface to get and set various device
    state indicators (secure boot, unit test mode, etc.). This change
    removes the unit test mode setting from the DeviceStateLib interface
    and transitions it to a fixed at build PCD so it cannot be changed
    at runtime. The one instance in this repo where
    DEVICE_STATE_UNIT_TEST_MODE was referenced has been updated to
    reference the PCD.
    
    - [x] Impacts functionality?
      - **Functionality** - Does the change ultimately impact how firmware functions?
      - Examples: Add a new library, publish a new PPI, update an algorithm, ...
    - [x] Impacts security?
      - **Security** - Does the change have a direct security impact on an application,
        flow, or firmware?
      - Examples: Crypto algorithm change, buffer overflow fix, parameter
        validation improvement, ...
    - [x] Breaking change?
      - **Breaking change** - Will anyone consuming this change experience a break
        in build or boot behavior?
      - Examples: Add a new library class, move a module to a different repo, call
        a function in a new library class in a pre-existing module, ...
    - [ ] Includes tests?
      - **Tests** - Does the change include any explicit test code?
      - Examples: Unit tests, integration tests, robot tests, ...
    - [ ] Includes documentation?
      - **Documentation** - Does the change contain explicit documentation additions
        outside direct code modifications (and comments)?
      - Examples: Update readme file, add feature readme file, link to documentation
        on an a separate Web page, ...
    
    How This Was Tested
    
    Tested on Q35
    
    Integration Instructions
    
    References to DEVICE_STATE_UNIT_TEST_MODE will need to be removed
    from platform code and platforms which want to set unit test mode
    will need to set the PCD in their platform DSC file(s).
    TaylorBeebe committed Aug 31, 2023
    Configuration menu
    Copy the full SHA
    2a94654 View commit details
    Browse the repository at this point in the history