From mboxrd@z Thu Jan 1 00:00:00 1970 From: thomas.petazzoni@free-electrons.com (Thomas Petazzoni) Date: Tue, 19 Nov 2013 16:07:54 +0100 Subject: [PATCH RFC 3/9] ASoC: davinci-evm: HDMI audio support for TDA998x trough McASP I2S bus In-Reply-To: <737bab348e15c6c2aba01ecd91e84c870aafe242.1384862950.git.jsarha@ti.com> References: <737bab348e15c6c2aba01ecd91e84c870aafe242.1384862950.git.jsarha@ti.com> Message-ID: <20131119160754.7307a763@skate> To: linux-arm-kernel@lists.infradead.org List-Id: linux-arm-kernel.lists.infradead.org Dear Jyri Sarha, On Tue, 19 Nov 2013 14:12:23 +0200, Jyri Sarha wrote: > -- compatible : "ti,da830-evm-audio" : forDM365/DA8xx/OMAPL1x/AM33xx > +- compatible : "ti,da830-evm-audio" : for DM365/DA8xx/OMAPL1x/AM33xx > + "ti,am33xx-beaglebone-black" : for Beaglebone-black HDMI audio The compatible string namespace is a global namespace, and in the compatible string you've chosen, there's nothing that makes it specific to audio. "ti,am33xx-beaglebone-black" could be used for any other device, or even for the board as a whole, no? Maybe you need to suffix it with "-audio", like the existing compatible string for audio in this driver. Thomas -- Thomas Petazzoni, CTO, Free Electrons Embedded Linux, Kernel and Android engineering http://free-electrons.com