Hi Will,
In hal_bsp.c, someone would create this structure with pin defintions just like
we do for our HAL and thus all pin defintions would be in hal_bsp.c. They then
pass this structure in the init call. No more pin definitions in pkg.yml.
I think having everything in hal_bsp.c makes sense. It was just the
relationship between the pkg.yml and hal_bsp.c values that wasn't clear.
Another option which has been mentioned by others is to have a header file that
contains all the pin definitions and this can be in the bsp. These definitions
can be the same as the ones in nrf_drv_config.h or they can be user defined. My
preference is to have them user-defined.
I can only speak for myself, but personally I have no specific
attachment to nrf_drv_config.h and it's very Nordic-centric, so
something user defined might be smaller and more 'generic'.
K.