I2S Echo Sample: RX Device Misconfiguration

[rriveramcrus]

	config.word_size = SAMPLE_BIT_WIDTH;
	config.channels = NUMBER_OF_CHANNELS;
	config.format = I2S_FMT_DATA_FORMAT_I2S;
	config.options = I2S_OPT_BIT_CLK_MASTER | I2S_OPT_FRAME_CLK_MASTER;
	config.frame_clk_freq = SAMPLE_FREQUENCY;
	config.mem_slab = &mem_slab;
	config.block_size = BLOCK_SIZE;
	config.timeout = TIMEOUT;
	if (!configure_streams(i2s_dev_rx, i2s_dev_tx, &config)) {
		return 0;
	}

I noticed in the I2S Echo sample that the both transmitter device and the receiver device have the I2S_OPT_BIT_CLK_MASTER and I2S_OPT_FRAME_CLK_MASTER options set when configured. This configuration makes sense for a transceiver device, but not for discrete transmitter and receiver devices.

In other words, this causes my receiver to output a bit clock and frame clock when it should be awaiting an input from the transmitter. Am I missing something, or should I prepare a patch to correct this?

[rriveramcrus]

This is actually not a misconfiguration. I do wonder if it would be better off as a configurable option, but that is a separate issue.