On Thu, Aug 13, 2009 at 05:08, Mark Brown wrote:> On Thu, Aug 13, 2009 at 11:41:13AM +0800, Barry Song wrote:>> On Wed, Aug 12, 2009 at 9:46 PM, Mark Brown wrote:>> > Just "ad1836" - the fact that it's controlled via SPI is clear from the>> > fact that this is a struct spi_driver.>>>> I agree the spi structure implies the meaning of spi. But files in>> arch/blackfin/mach... are using names like "xxx-spi" for almost all>> spi devices. How about keeping the coherence?>> Oh dear. Looks like you've essentially got a choice between being> consistent with the rest of the world and being consistent with what the> blackfin stuff is doing. it really isnt that bad. the drivers that use "-spi" suffix: .modalias = "ad1836-spi", # audio .modalias = "ad1938-spi", # audio .modalias = "ad7147-spi", # input .modalias = "ad9960-spi", # dead .modalias = "bfin-lq035q1-spi", # video .modalias = "fxo-spi", # dead .modalias = "fxs-spi", # dead dropping the dead devices leaves us with 4 drivers to fix, two ofwhich are audio. should be easy for us to fix.-mike_______________________________________________Alsa-devel mailing listAlsa-devel@xxxxxxxxxxxxxxxxxxxx://mailman.alsa-project.org/mailman/listinfo/alsa-devel