Lolin D32 Pro variant has conflicting pin names in pins_arduino.h

[tobozo]

Board: Lolin D32 Pro
IDE name: Arduino 1.8.5

I can't compile any sketch using TFT_eSPI library when LOLIN D32 Pro is selected as a board.
Conflicting pin names produce unrecoverable errors at compilation.

This issue describes very well the problem, and while the suggested solution works, it isn't appropriate for extended uses (e.g. enable/disable psram)

Note that this situation also conflicts with a few other libraries (most are Adafruit).

Editing /variants/d32_pro/pins_arduino.h file to use defines solved this for me:

#define TF_CS 4    // TF (Micro SD Card) CS pin
#define TS_CS 12   // Touch Screen CS pin
#define TFT_CS 14  // TFT CS pin
#define TFT_LED 32 // TFT backlight control pin
#define TFT_RST 33 // TFT reset pin
#define TFT_DC 27  // TFT DC pin

Is that worth a pull request?