1 comment:
File src/include/device/pnp.h:
Patch Set #1, 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.
To view, visit change 45316. To unsubscribe, or for help writing mail filters, visit settings.