Dear Mark
Thank you for your checking patch.
At first, I will re-send v2 patches as "bug fix" and "add XXX".
+#define SH_FSI_FMT_MONO_DELAY (2 << 0) +#define SH_FSI_FMT_PCM (3 << 0) +#define SH_FSI_FMT_I2S (4 << 0) +#define SH_FSI_FMT_TDM (5 << 0) +#define SH_FSI_FMT_TDM_DELAY (6 << 0)
Wouldn't it be clearer to just define these as straight numbers?
Sorry. Do you mean (1 << 0) is needed ? (1 << 0) is not changed from old one
-------------------------------- #define SH_FSI_FMT_MONO (1 << 0) <=== -#define SH_FSI_FMT_MONO_DELAY (1 << 1) -#define SH_FSI_FMT_PCM (1 << 2) -#define SH_FSI_FMT_I2S (1 << 3) -#define SH_FSI_FMT_TDM (1 << 4) -#define SH_FSI_FMT_TDM_DELAY (1 << 5) +#define SH_FSI_FMT_MONO_DELAY (2 << 0) +#define SH_FSI_FMT_PCM (3 << 0) +#define SH_FSI_FMT_I2S (4 << 0) +#define SH_FSI_FMT_TDM (5 << 0) +#define SH_FSI_FMT_TDM_DELAY (6 << 0) --------------------------------
Best regards -- Kuninori Morimoto