[coreboot-gerrit] Change in coreboot[master]: sc7180: GPIO: Add I2S configuration for sc7180

[HAOUAS Elyes (Code Review)]

HAOUAS Elyes has posted comments on this change. ( https://review.coreboot.org/c/coreboot/+/38593 )

Change subject: sc7180: GPIO: Add I2S configuration for sc7180
......................................................................

Patch Set 6:

(1 comment)

https://review.coreboot.org/c/coreboot/+/38593/6/src/soc/qualcomm/sc7180/qi2... 
File src/soc/qualcomm/sc7180/qi2s.c:

https://review.coreboot.org/c/coreboot/+/38593/6/src/soc/qualcomm/sc7180/qi2... 
PS6, Line 16: #include <arch/cache.h>
            : #include <device/mmio.h>
            : #include <soc/addressmap.h>
            : #include <soc/gpio.h>
            : #include <soc/clock.h>
            : #include <symbols.h>
            : #include <assert.h>
            : #include <gpio.h>
            : #include <string.h>
            : #include <soc/qi2s.h>
Looks like you don't use all of those includes.
Please check, and include only what you use.