Message ID | 1280962268-31407-1-git-send-email-timur@freescale.com (mailing list archive) |
---|---|
State | Accepted, archived |
Delegated to: | Kumar Gala |
Headers | show |
On Wed, Aug 04, 2010 at 05:51:08PM -0500, Timur Tabi wrote: > Add support for adding "status = disabled" to an SSI node to incidate that it > is not wired on the board. This replaces the not-so-intuitive previous method > of omitting a codec-handle property. > > Signed-off-by: Timur Tabi <timur@freescale.com> Acked-by: Mark Brown <broonie@opensource.wolfsonmicro.com>
On Thu, 2010-08-05 at 12:42 +0100, Mark Brown wrote: > On Wed, Aug 04, 2010 at 05:51:08PM -0500, Timur Tabi wrote: > > Add support for adding "status = disabled" to an SSI node to incidate that it > > is not wired on the board. This replaces the not-so-intuitive previous method > > of omitting a codec-handle property. > > > > Signed-off-by: Timur Tabi <timur@freescale.com> > > Acked-by: Mark Brown <broonie@opensource.wolfsonmicro.com> Applied. Thanks Liam
On Aug 4, 2010, at 5:51 PM, Timur Tabi wrote: > Add support for adding "status = disabled" to an SSI node to incidate that it > is not wired on the board. This replaces the not-so-intuitive previous method > of omitting a codec-handle property. > > Signed-off-by: Timur Tabi <timur@freescale.com> > --- > > Kumar, would you please ACK the device-tree portion of this patch? I want > it to go through the ALSA tree. > > arch/powerpc/boot/dts/mpc8610_hpcd.dts | 1 + > sound/soc/fsl/fsl_ssi.c | 13 ++++++++++--- > 2 files changed, 11 insertions(+), 3 deletions(-) Acked-by: Kumar Gala <galak@kernel.crashing.org> - k
diff --git a/arch/powerpc/boot/dts/mpc8610_hpcd.dts b/arch/powerpc/boot/dts/mpc8610_hpcd.dts index 9535ce6..83c3218 100644 --- a/arch/powerpc/boot/dts/mpc8610_hpcd.dts +++ b/arch/powerpc/boot/dts/mpc8610_hpcd.dts @@ -286,6 +286,7 @@ ssi@16100 { compatible = "fsl,mpc8610-ssi"; + status = "disabled"; cell-index = <1>; reg = <0x16100 0x100>; interrupt-parent = <&mpic>; diff --git a/sound/soc/fsl/fsl_ssi.c b/sound/soc/fsl/fsl_ssi.c index a0e18b8..45d27b1 100644 --- a/sound/soc/fsl/fsl_ssi.c +++ b/sound/soc/fsl/fsl_ssi.c @@ -636,12 +636,19 @@ static int __devinit fsl_ssi_probe(struct of_device *of_dev, struct resource res; char name[64]; - /* We are only interested in SSIs with a codec phandle in them, so let's - * make sure this SSI has one. + /* SSIs that are not connected on the board should have a + * status = "disabled" + * property in their device tree nodes. */ - if (!of_get_property(np, "codec-handle", NULL)) + if (!of_device_is_available(np)) return -ENODEV; + /* Check for a codec-handle property. */ + if (!of_get_property(np, "codec-handle", NULL)) { + dev_err(&of_dev->dev, "missing codec-handle property\n"); + return -ENODEV; + } + /* We only support the SSI in "I2S Slave" mode */ sprop = of_get_property(np, "fsl,mode", NULL); if (!sprop || strcmp(sprop, "i2s-slave")) {
Add support for adding "status = disabled" to an SSI node to incidate that it is not wired on the board. This replaces the not-so-intuitive previous method of omitting a codec-handle property. Signed-off-by: Timur Tabi <timur@freescale.com> --- Kumar, would you please ACK the device-tree portion of this patch? I want it to go through the ALSA tree. arch/powerpc/boot/dts/mpc8610_hpcd.dts | 1 + sound/soc/fsl/fsl_ssi.c | 13 ++++++++++--- 2 files changed, 11 insertions(+), 3 deletions(-)