Hi Andre, > What do you mean, exactly? > To put *some* context into the commit message, I tried to find out what > this "vsc" device is about, and found "crosspoint switch" somewhere > (doc/README.VSC3316-3308). Is that not what it is about? Or is not a > Freescale device, but just used by Freescale SoCs? Some T208 hardware > seems to be the only user, though.
The VSC3316 is a switch manufactured by Microchip: https://www.microsemi.com/product-directory/digital-crosspoint-switches/3998-vsc3316 > Can you suggest a better description? What about: Subject: freescale: vsc3316_3308: fix header inclusion guard It seems like the header inclusion guard for the vsc3316_3308 switch header was misspelled. Make the preprocessor symbol for the #ifndef and #define lines the same, so that the double inclusion protection works as expected.