For consisteny with how this is done everywhere else could you move onto two lines?

    //! port of the pin (e.g. GPIOC)
    GPIO_TypeDef *port;

    //! port pin number (e.g. GPIO_Pin_3)
    uint32_t pin;