Marshall Dawson has posted comments on this change. ( https://review.coreboot.org/c/coreboot/+/37532 )
Change subject: northbridge: Don't use both of _ADR and _HID ......................................................................
Patch Set 2:
Patch Set 1:
Patch Set 1:
(1 comment)
If you rebase on the latest code, I think this commit should go away.
Yes, I get the comment and code from that commit... Should I redo the comment or leave it because is already in the latest code?
What I'm saying is that it looked like your starting point is out of date, and this particular change has already gone in. If you run "git fetch && git rebase origin/master", then you'll no longer need this commit at all. OTOH if I'm misreading it, then you should change the commit message to match what this commit actually does.