[coreboot-gerrit] Change in coreboot[master]: src: Add missing <device/pnp_def.h>

[HAOUAS Elyes (Code Review)]

HAOUAS Elyes has posted comments on this change. ( https://review.coreboot.org/c/coreboot/+/45316 )

Change subject: src: Add missing <device/pnp_def.h>
......................................................................

Patch Set 1:

(1 comment)

https://review.coreboot.org/c/coreboot/+/45316/1/src/include/device/pnp.h 
File src/include/device/pnp.h:

https://review.coreboot.org/c/coreboot/+/45316/1/src/include/device/pnp.h@a6 
PS1, Line 6: #include <device/pnp_def.h>
> I'm not really sure if removing this here is a good idea or not. […]

I like the idea to include indirectly <device/pnp_def.h> using <device/pnp.h> , but only when <device/pnp.h> is needed.
I mean, including <device/pnp.h> just for "PNP_IDX_*" isn't reasonable.

I'll update current patch and remove redundant  <device/pnp_def.h> when <device/pnp.h> is needed.