From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: from nz-out-0506.google.com (nz-out-0506.google.com [64.233.162.225]) by ozlabs.org (Postfix) with ESMTP id 5DD45DE0F3 for ; Fri, 11 Jan 2008 07:40:00 +1100 (EST) Received: by nz-out-0506.google.com with SMTP id i1so492859nzh.39 for ; Thu, 10 Jan 2008 12:39:58 -0800 (PST) Message-ID: <9e4733910801101239n539507a8sb23d1a2177e69bd9@mail.gmail.com> Date: Thu, 10 Jan 2008 15:39:57 -0500 From: "Jon Smirl" To: "Timur Tabi" Subject: Re: [alsa-devel] [PATCH] ASoC drivers for the Freescale MPC8610 SoC In-Reply-To: <47867C80.4050408@freescale.com> MIME-Version: 1.0 Content-Type: text/plain; charset=ISO-8859-1 References: <9e4733910801010925j67192427o4e0e824b9d7e0ad0@mail.gmail.com> <20080107182853.GA17312@sirena.org.uk> <20080110034919.GI17816@localhost.localdomain> <9e4733910801092141m5d4476d0l1c081307fd5c1969@mail.gmail.com> <1199961015.19903.270.camel@localhost.localdomain> <47863C31.50309@freescale.com> <478641E3.5070404@freescale.com> <9e4733910801101210w5846bb4ds38a603a489ef0522@mail.gmail.com> <47867C80.4050408@freescale.com> Cc: alsa-devel@alsa-project.org, Liam Girdwood , linuxppc-dev@ozlabs.org, Olof Johansson , david@gibson.dropbear.id.au List-Id: Linux on PowerPC Developers Mail List List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , On 1/10/08, Timur Tabi wrote: > Jon Smirl wrote: > > > Isn't your codec is i2c controlled? Where does the main node for the > > code live, i2c bus or ssi bus? What does the link between the buses > > look like for pointing to the codec entry? > > The CS4270 driver is "old style", which means it probes all possible I2C > addresses until it finds a hit, and then stops. This has all the obvious > drawbacks, but I'm stuck with that design for now. So the codec is controlled from the i2c bus and SSI is used to supply it with data. Based on what has been said on this list, the device tree node for the codec should be on the i2c bus with a link from the ssi bus to it. > > > What about fsl,ssi being too generic for a compatible type? > > Already fixed: > > ssi@16000 { > compatible = "fsl,mpc8610-ssi"; > -- > Timur Tabi > Linux kernel developer at Freescale > -- Jon Smirl jonsmirl@gmail.com