On Wednesday, February 02, 2011 10:00 AM, Mark Brown wrote:
On Wed, Feb 02, 2011 at 10:53:15AM -0600, H Hartley Sweeten wrote:
I don't have any problem with leaving the reset management in the codec. The enable (chip-select) should be handled in the platform code. But, gpio_disable should be removed from struct cs4271_platform_data to avoid any confusion.
Is that really a chip select or is it a power control?
_From the datasheet:
Pin Name # Pin Description -------- -- ---------------------------------------------------------- AD0/nCS 13 Address Bit 0 (I2S)/Control Port Chip Select (SPI) (Input) - AD0 is a chip address pin in I2C mode; nCS is the chip signal select for SPI format.
The other pin in question:
Pin Name # Pin Description -------- -- ---------------------------------------------------------- nRST 14 Reset (Input) - The device enters a low power mode when this pin is driven low.
Regards, Hartley=