[coreboot-gerrit] [XS] Change in coreboot[main]: mb/google/brox: Select USE_UNIFIED_AP_FIRMWARE_FOR_UFS_AND_NON_UFS

[Paul Menzel (Code Review)]

Attention is currently required from: Ashish Kumar Mishra, Karthik Ramasubramanian, Shelley Chen.

Paul Menzel has posted comments on this change. ( https://review.coreboot.org/c/coreboot/+/81224?usp=email )

Change subject: mb/google/brox: Select USE_UNIFIED_AP_FIRMWARE_FOR_UFS_AND_NON_UFS
......................................................................

Patch Set 3:

(2 comments)

Commit Message:

https://review.coreboot.org/c/coreboot/+/81224/comment/1b5bdc91_fb1ad77c :
PS3, Line 9: Select USE_UNIFIED_AP_FIRMWARE_FOR_UFS_AND_NON_UFS in brox Kconfig.
           : This enables a single binary for both SKU1 and SKU2.
           : This also adds a warm boot step, only when booting from cold reset
           : to disable the UFS Controller.
Please add a blank line between paragraphs, or do not break lines just because a sentence ends.

https://review.coreboot.org/c/coreboot/+/81224/comment/3e6990f5_4026545b :
PS3, Line 11: This also adds a warm boot step, only when booting from cold reset
            : to disable the UFS Controller.
I do not understand the relation regarding to the Kconfig change at hand. Could you please elaborate?