Change in flashrom[master]: ichspi: Rename HSFC FDBC offset macros

[Anastasia Klimchuk (Code Review)]

Attention is currently required from: Subrata Banik, Nico Huber, Angel Pons.
Anastasia Klimchuk has posted comments on this change. ( https://review.coreboot.org/c/flashrom/+/62894 )

Change subject: ichspi: Rename HSFC FDBC offset macros
......................................................................

Patch Set 8:

(1 comment)

Commit Message:

https://review.coreboot.org/c/flashrom/+/62894/comment/5c8602c7_8a6b17a9 
PS6, Line 10: during the data portion of the SPI cycle.
> And only for `HSFC_FDBC -> HSFC_FDBC_MASK`, and not for the `_OFF` macro. `HSFC_FDBC_OFF` is the correct name anyway, and probably only changed to keep it pprint_reg() compatible.

It was me, I suggested to rename both HSFC_FDBC -> HSFC_FDBC_MASK and HSFC_FDBC_OFF -> HSFC_FDBC_MASK_OFF to be pprint_reg() compatible (in a comment earlier in this patch). I really like the idea of dropping _pprint_reg() macro, and I thought renaming both macros supported it.

But yes, not renaming any of them also is pprint_reg() compatible!
So, that means,
- this patch is not needed
- next patch CB:62892 becomes "Introduce HSFC_FDBC_VAL(n) macro" (instead of HSFC_FDBC(n) macro)

is this correct?