From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1752845AbbCGOfP (ORCPT ); Sat, 7 Mar 2015 09:35:15 -0500 Received: from mezzanine.sirena.org.uk ([106.187.55.193]:56291 "EHLO mezzanine.sirena.org.uk" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1750855AbbCGOfM (ORCPT ); Sat, 7 Mar 2015 09:35:12 -0500 Date: Sat, 7 Mar 2015 14:34:34 +0000 From: Mark Brown To: "Maciej S. Szmigiero" Cc: alsa-devel@alsa-project.org, Rob Herring , Pawel Moll , Mark Rutland , Ian Campbell , Kumar Gala , Liam Girdwood , Jaroslav Kysela , Takashi Iwai , Lars-Peter Clausen , Brian Austin , devicetree@vger.kernel.org, linux-kernel@vger.kernel.org Message-ID: <20150307143434.GL28806@sirena.org.uk> References: <54F9F81E.8080105@maciej.szmigiero.name> <20150307105201.GB28806@sirena.org.uk> <54FB041D.8010102@maciej.szmigiero.name> MIME-Version: 1.0 Content-Type: multipart/signed; micalg=pgp-sha256; protocol="application/pgp-signature"; boundary="+278g007AL/ykmV8" Content-Disposition: inline In-Reply-To: <54FB041D.8010102@maciej.szmigiero.name> X-Cookie: Editing is a rewording activity. User-Agent: Mutt/1.5.23 (2014-03-12) X-SA-Exim-Connect-IP: 94.175.94.161 X-SA-Exim-Mail-From: broonie@sirena.org.uk Subject: Re: [PATCH][ASoC]Add DT bindings for generic ASoC AC97 CODEC driver X-SA-Exim-Version: 4.2.1 (built Mon, 26 Dec 2011 16:24:06 +0000) X-SA-Exim-Scanned: Yes (on mezzanine.sirena.org.uk) Sender: linux-kernel-owner@vger.kernel.org List-ID: X-Mailing-List: linux-kernel@vger.kernel.org --+278g007AL/ykmV8 Content-Type: text/plain; charset=us-ascii Content-Disposition: inline On Sat, Mar 07, 2015 at 02:58:53PM +0100, Maciej S. Szmigiero wrote: > W dniu 07.03.2015 11:52, Mark Brown pisze: > > On Fri, Mar 06, 2015 at 07:55:26PM +0100, Maciej S. Szmigiero wrote: > > AC'97 shouldn't need DT bindings for the CODEC, it's an enumerable bus. > What this driver did / does is essentially to allow attaching AC'97 bus > to ASoC AC'97 controllers which don't do it on its own (from what I can > see only board files do it). That's true, and it's something it's easy to get away with with board files, but that doesn't mean it's something we should be doing for device tree where we're not just finding an expedient way to load things but rather defining an ABI. > This driver originally constrained rates to 8000, 11025, 22050, 44100, > 48000. > An alternative would be to remove this constraint at all from this > driver and leave such constraining for controller and AC'97 bus code. That's a much better approach. > But since this driver was originally limited to these rates I think it > would be safer to do it only when instantiated via OF. Again, DT is defining an ABI. --+278g007AL/ykmV8 Content-Type: application/pgp-signature; name="signature.asc" Content-Description: Digital signature -----BEGIN PGP SIGNATURE----- Version: GnuPG v2 iQEcBAEBCAAGBQJU+wx5AAoJECTWi3JdVIfQkKIH/1m9xSCDcLm1T19DFx9j4/t2 MC2zbugpd567aqpzuVvXdnN7XE3AgZGljTmjC3fJzZD22crr+ewoLXRqdPCLEh0g ILgDTuO3dTcy2EgS1ALNkKH9hbf0pgV5ImVmcCOk8BfuP3ELylb3dxwnJ+ewL4b5 ysBtVTtLK8WXQ34gd7YMjUZZvrYsyh32G3rCI7F6sog66C7EiYRLU5bDpB0maanP sedn3rIFPxdUcy0DzxxolFJS5qUdutJ/8Kqn8k0GjCFTBSmmnIFnjJFbl+PuQU15 PiBY3gWAzs9IoOiDZG3qgZnfmxwqxEa4YFd5zWlnmlpC/kfGx11D9J23qTIx0Wo= =/qlL -----END PGP SIGNATURE----- --+278g007AL/ykmV8--