Was mir hier gerade noch auffällt:
In MX_I2C1_Init() rufst Du die Funktion HAL_I2C_Init() auf.
In Deinem zweiten Code-Schnippsel heißt die Funktion HAL_I2C_MspInit().
Soll das so sein?