Angel Pons has posted comments on this change. ( https://review.coreboot.org/c/coreboot/+/42131 )
Change subject: mb/asrock/b85m_pro4: Drop spurious LPC decode ranges ......................................................................
Patch Set 11:
(1 comment)
https://review.coreboot.org/c/coreboot/+/42131/11//COMMIT_MSG Commit Message:
https://review.coreboot.org/c/coreboot/+/42131/11//COMMIT_MSG@9 PS11, Line 9: Only one generic decode range is needed for the HWM.
Please elaborate: found how? will it fix something?
I had copied the code from elsewhere but didn't check it in detail. Then I revisited it and saw I only needed one range. It doesn't make much of a difference because the dropped decode ranges were not used.
I'm not sure if this is worth adding to the commit message. Still, want me to add something?