soc/intel/pantherlake: Comply with the no typedef coding style rule

As https://doc.coreboot.org/contributing/coding_style.html#typedefs
states: "In general, a pointer, or a struct that has elements that can
reasonably be directly accessed should never be a typedef". This
commit makes the Intel Panther Lake SoC code comply with this by using
explicitly `struct soc_intel_pantherlake_config' in the
soc/intel/pantherlake code as I have been suggested to for the
`fsp_params.c' files. The rule being the rule and consistency across
a project matters more than personal preferences.

The documentation lists five exceptions and none on them cover the use
of `config_t' instead `struct soc_intel_pantherlake' but I believe it
does not make the code better for the following three reasons:

1. It is repetitive, make the line longer and the code is in
   soc/intel/pantherlake so obviously the config_t data structure is
   the pantherlake soc configuration.

2. It makes re-usability from one generation to another unnecessarily
   harder.

3. This config_t abstraction is required for and used by some common
   block code anyway. Hence, we end-up with some code using `config_t'
   and other using the final structure which break the consistency of
   the code when the project in looked as a whole.

BUG=348678529
TEST=Google fatcat mainboard compiles

Change-Id: Ibe382615db1a7c7a0841d8fe4ae43c226e2c2021
Signed-off-by: Jeremy Compostella <jeremy.compostella@intel.com>
Reviewed-on: https://review.coreboot.org/c/coreboot/+/84561
Tested-by: build bot (Jenkins) <no-reply@coreboot.org>
Reviewed-by: Subrata Banik <subratabanik@google.com>
Reviewed-by: Ronak Kanabar <ronak.kanabar@intel.com>

0 files changed, 0 insertions, 0 deletions