From mboxrd@z Thu Jan 1 00:00:00 1970 From: Lars-Peter Clausen Subject: Re: [alsa-devel] [PATCH 2/5 v4] drm/i2c/adv7511: Add audio support Date: Sat, 9 Apr 2016 17:02:49 +0200 Message-ID: <57091999.4010906@metafoo.de> References: <5707D249.5040707@metafoo.de> <5707D88A.3010400@synopsys.com> Mime-Version: 1.0 Content-Type: text/plain; charset="utf-8" Content-Transfer-Encoding: base64 Return-path: In-Reply-To: <5707D88A.3010400@synopsys.com> List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , Errors-To: dri-devel-bounces@lists.freedesktop.org Sender: "dri-devel" To: Jose Abreu , linux-snps-arc@lists.infradead.org, linux-kernel@vger.kernel.org, dri-devel@lists.freedesktop.org, alsa-devel@alsa-project.org Cc: Vineet.Gupta1@synopsys.com, Alexey.Brodkin@synopsys.com, CARLOS.PALMINHA@synopsys.com, lgirdwood@gmail.com, broonie@kernel.org, tiwai@suse.com List-Id: alsa-devel@alsa-project.org T24gMDQvMDgvMjAxNiAwNjoxMiBQTSwgSm9zZSBBYnJldSB3cm90ZToKWy4uLl0KPj4KPj4gWy4u Ll0KPj4+ICstIGFkaSxlbmFibGUtYXVkaW86IElmIHNldCB0aGUgQURWNzUxMSBkcml2ZXIgd2ls bCByZWdpc3RlciBhIGNvZGVjIGludGVyZmFjZQo+Pj4gKyAgaW50byBBTFNBIFNvQy4KPj4gVGhp cyBpcyBub3QgYSBkZXNjcmlwdGlvbiBvZiB0aGUgaGFyZHdhcmUuCj4gCj4gSXMgdGhpcyBva2F5 OiAiYWRpLGVuYWJsZS1hdWRpbzogU2V0IHRoaXMgYm9vbGVhbiBwYXJhbWV0ZXIgaWYgQURWNzUx MQo+IHRyYW5zbWl0dGVyIHJvdXRlcyBhdWRpbyBzaWduYWxzIiA/CgpJIGRvbid0IHRoaW5rIHdl IG5lZWQgdGhpcyBwcm9wZXJ0eS4gVGhlcmUgaXMgbm8gcHJvYmxlbSB3aXRoIHJlZ2lzdGVyaW5n CnRoZSBhdWRpbyBwYXJ0IHVuY29uZGl0aW9uYWxseS4gQXMgbG9uZyBhcyB0aGVyZSBpcyBubyBj b25uZWN0aW9uIHdlIHdvbnQKY3JlYXRlIGEgc291bmQgY2FyZCB0aGF0IGlzIGV4cG9zZWQgdG8g dXNlcnNwYWNlLgoKX19fX19fX19fX19fX19fX19fX19fX19fX19fX19fX19fX19fX19fX19fX19f X18KZHJpLWRldmVsIG1haWxpbmcgbGlzdApkcmktZGV2ZWxAbGlzdHMuZnJlZWRlc2t0b3Aub3Jn Cmh0dHBzOi8vbGlzdHMuZnJlZWRlc2t0b3Aub3JnL21haWxtYW4vbGlzdGluZm8vZHJpLWRldmVs Cg== From mboxrd@z Thu Jan 1 00:00:00 1970 From: lars@metafoo.de (Lars-Peter Clausen) Date: Sat, 9 Apr 2016 17:02:49 +0200 Subject: [alsa-devel] [PATCH 2/5 v4] drm/i2c/adv7511: Add audio support In-Reply-To: <5707D88A.3010400@synopsys.com> References: <5707D249.5040707@metafoo.de> <5707D88A.3010400@synopsys.com> List-ID: Message-ID: <57091999.4010906@metafoo.de> To: linux-snps-arc@lists.infradead.org On 04/08/2016 06:12 PM, Jose Abreu wrote: [...] >> >> [...] >>> +- adi,enable-audio: If set the ADV7511 driver will register a codec interface >>> + into ALSA SoC. >> This is not a description of the hardware. > > Is this okay: "adi,enable-audio: Set this boolean parameter if ADV7511 > transmitter routes audio signals" ? I don't think we need this property. There is no problem with registering the audio part unconditionally. As long as there is no connection we wont create a sound card that is exposed to userspace. From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1753206AbcDIPCz (ORCPT ); Sat, 9 Apr 2016 11:02:55 -0400 Received: from www381.your-server.de ([78.46.137.84]:49264 "EHLO www381.your-server.de" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1751336AbcDIPCy (ORCPT ); Sat, 9 Apr 2016 11:02:54 -0400 Subject: Re: [alsa-devel] [PATCH 2/5 v4] drm/i2c/adv7511: Add audio support To: Jose Abreu , linux-snps-arc@lists.infradead.org, linux-kernel@vger.kernel.org, dri-devel@lists.freedesktop.org, alsa-devel@alsa-project.org References: <5707D249.5040707@metafoo.de> <5707D88A.3010400@synopsys.com> Cc: architt@codeaurora.org, airlied@linux.ie, Vineet.Gupta1@synopsys.com, Alexey.Brodkin@synopsys.com, lgirdwood@gmail.com, CARLOS.PALMINHA@synopsys.com, broonie@kernel.org, tiwai@suse.com From: Lars-Peter Clausen X-Enigmail-Draft-Status: N1110 Message-ID: <57091999.4010906@metafoo.de> Date: Sat, 9 Apr 2016 17:02:49 +0200 User-Agent: Mozilla/5.0 (X11; Linux x86_64; rv:38.0) Gecko/20100101 Icedove/38.6.0 MIME-Version: 1.0 In-Reply-To: <5707D88A.3010400@synopsys.com> Content-Type: text/plain; charset=windows-1252 Content-Transfer-Encoding: 8bit X-Authenticated-Sender: lars@metafoo.de Sender: linux-kernel-owner@vger.kernel.org List-ID: X-Mailing-List: linux-kernel@vger.kernel.org On 04/08/2016 06:12 PM, Jose Abreu wrote: [...] >> >> [...] >>> +- adi,enable-audio: If set the ADV7511 driver will register a codec interface >>> + into ALSA SoC. >> This is not a description of the hardware. > > Is this okay: "adi,enable-audio: Set this boolean parameter if ADV7511 > transmitter routes audio signals" ? I don't think we need this property. There is no problem with registering the audio part unconditionally. As long as there is no connection we wont create a sound card that is exposed to userspace.