Groups | Search | Server Info | Keyboard shortcuts | Login | Register [http] [https] [nntp] [nntps]


Groups > linux.kernel > #1403958

Re: [PATCH v7 0/3] ASoC: cygnus: Add audio support for Broadcom Cygnus SoC

Path csiph.com!aioe.org!bofh.it!news.nic.it!robomod
From Scott Branden <scott.branden@broadcom.com>
Newsgroups linux.kernel
Subject Re: [PATCH v7 0/3] ASoC: cygnus: Add audio support for Broadcom Cygnus SoC
Date Thu, 19 May 2016 22:40:01 +0200
Message-ID <rACTD-4Do-9@gated-at.bofh.it> (permalink)
References <rzTjQ-pn-7@gated-at.bofh.it>
X-Original-To Simran Rai <simran.rai@broadcom.com>, Rob Herring <robh+dt@kernel.org>, Pawel Moll <pawel.moll@arm.com>, Mark Rutland <mark.rutland@arm.com>, Ian Campbell <ijc+devicetree@hellion.org.uk>, Kumar Gala <galak@codeaurora.org>, Ray Jui <rjui@broadcom.com>, Scott Branden <sbranden@broadcom.com>, Liam Girdwood <lgirdwood@gmail.com>, Mark Brown <broonie@kernel.org>, Jaroslav Kysela <perex@perex.cz>, Takashi Iwai <tiwai@suse.com>
Dkim-Signature v=1; a=rsa-sha256; c=relaxed/relaxed; d=broadcom.com; s=google; h=subject:to:references:cc:from:message-id:date:user-agent :mime-version:in-reply-to:content-transfer-encoding; bh=1I1QyfWh9aBX6m92sb77YvscbhhqfwEi62Z2+Py9ym0=; b=gjGlZW4RDkQKhQNZ1GDmelz4rI8E4AhF5wXg76YTv+7Gl7zo4E7hVVUPR/vKUDodo/ DyHsOytxhTH5Ob5bH0MA+9Wndk+rStO9g1EvioGEUfSHslCBdGRLPyahbNy27HMIh457 9a1pZUbspit5HgLbNa5G9fZCfuoxMG2Qor6jc=
X-Google-Dkim-Signature v=1; a=rsa-sha256; c=relaxed/relaxed; d=1e100.net; s=20130820; h=x-gm-message-state:subject:to:references:cc:from:message-id:date :user-agent:mime-version:in-reply-to:content-transfer-encoding; bh=1I1QyfWh9aBX6m92sb77YvscbhhqfwEi62Z2+Py9ym0=; b=hxputSpAJhMCpr6cLAYkcVkCNryz4qw0L0NwyTbrRf87zV3Pnbp/0uV/NqFW5nZZNz 14wa6STjJqB9E820TbBvJ4tMitaUCWa3fMg3Fog/jexR9rL64kmfRT1VAViRlzow8XNR +ym+KDMS5IAyLdXnDUAPawkff3kg3yACiuzBgwI7OWazLG5FqjMaDen561ApbYJFkwjn aOQm/0LfJEHsy31eU2tAcJb+O740MOeWtohwOIhyJp5ylTRiDeDtLD6RJwjwQ4+6lun1 zlZtSKJ9kFuHKGc/TYy6FhM1FA5rvmd1rAx55OQrmAS6fNxkmtvPvo9RVFIjL5Cs8oRc GatA==
X-Gm-Message-State AOPr4FWIqt+fBlX54ux66eUIrWzPptbRnxt87lOIH6UhGWx2oxcAQp0n0tDwoqOfY5kg2itt
X-Received by 10.66.249.228 with SMTP id yx4mr22738461pac.29.1463690151538; Thu, 19 May 2016 13:35:51 -0700 (PDT)
User-Agent Mozilla/5.0 (X11; Linux x86_64; rv:38.0) Gecko/20100101 Thunderbird/38.6.0
MIME-Version 1.0
Content-Type text/plain; charset=windows-1252; format=flowed
Content-Transfer-Encoding 7bit
Sender robomod@news.nic.it
List-ID <linux-kernel.vger.kernel.org>
X-Mailing-List linux-kernel@vger.kernel.org
Approved robomod@news.nic.it
Lines 67
Organization linux.* mail to news gateway
X-Original-Cc Lori Hikichi <lhikichi@broadcom.com>, devicetree@vger.kernel.org, linux-arm-kernel@lists.infradead.org, bcm-kernel-feedback-list@broadcom.com, linux-kernel@vger.kernel.org, Arun Parameswaran <arunp@broadcom.com>, alsa-devel@alsa-project.org, Simran <ssimran@lbrmn-ssimran-lnxub64.ric.broadcom.com>
X-Original-Date Thu, 19 May 2016 13:35:46 -0700
X-Original-Message-ID <573E23A2.4020907@broadcom.com>
X-Original-References <1463514382-20930-1-git-send-email-simran.rai@broadcom.com>
X-Original-Sender linux-kernel-owner@vger.kernel.org
Xref csiph.com linux.kernel:1403958

Show key headers only | View raw


Looks good Simran.

Mark, anything else for us to do before this driver can be accepted 
upstream?

On 16-05-17 12:46 PM, Simran Rai wrote:
> From: Simran <ssimran@lbrmn-ssimran-lnxub64.ric.broadcom.com>
>
> Hi,
>
> This patchset contains audio support for Broadcom's Cygnus SoC.
> It contains DT bindings and core audio driver. The audio driver supports
> both capture and playback of Audio PCM samples over I2S/TDM interface and
> provides playback support over SPDIF interface.
>
> This patchset is derived from a previously submitted patchset:
> http://lkml.iu.edu/hypermail/linux/kernel/1503.3/05434.html
>
> This patchset has been tested on Cygnus wireless audio bcm958305K board.
> It is based on v4.6-rc1 and is available from github:
>
> repo: https://github.com/Broadcom/cygnus-linux/tree/cygnus-sound-v7
>
> Changes from v6:
> - DT bindings acknowledged by Rob Herring
> Changes from v5:
> - Fix code style, e.g. change "if" statements to "switch" statements
> - Fix SPDIF output enable register field
> - Set BUFFER_PAIR_ENABLE for both mono, stereo and TDM modes
> - Reflect PCM bit formats for SPDIF, SSP and TDM as supported by
> hardware
> Changes from v4:
> - Fix power suspend function and add power resume function
> - Move clock initialization code from audio driver to clock framework
> Changes from v3:
> - Fix the subject lines to match the style for the subsystem
> Changes from v2:
> - Split patchset 2/2 from v2 into patchsets 2/3 and 3/3.
> - Remove SND_SOC_CYGNUS_DIAG. Diagnostics can be performed using
> standard kernel trace infrastructure.
> - Fix interrupt handler. Acknowledge only those interrupts that are
> handledby ISR.
> - Modify configure_vco() and the pll_macro_entry() struct to make it
> better readable. The functionality did not change.
> - Remove casts on macros
> - Remove surround sound channel grouping from the driver.
> Changes from v1:
> - Address code review comments. Fix print format of type size_t and
> pointer.
>
> Simran Rai (3):
>    ASoC: cygnus: Add DT bindings for Broadcom Cygnus audio
>    ASoC: cygnus: Add Cygnus audio DAI driver
>    ASoC: cygnus: Add Cygnus audio DMA driver
>
>   .../bindings/sound/brcm,cygnus-audio.txt           |   67 +
>   sound/soc/bcm/Kconfig                              |    9 +
>   sound/soc/bcm/Makefile                             |    5 +
>   sound/soc/bcm/cygnus-pcm.c                         |  861 +++++++++++
>   sound/soc/bcm/cygnus-ssp.c                         | 1529 ++++++++++++++++++++
>   sound/soc/bcm/cygnus-ssp.h                         |  139 ++
>   6 files changed, 2610 insertions(+)
>   create mode 100644 Documentation/devicetree/bindings/sound/brcm,cygnus-audio.txt
>   create mode 100644 sound/soc/bcm/cygnus-pcm.c
>   create mode 100644 sound/soc/bcm/cygnus-ssp.c
>   create mode 100644 sound/soc/bcm/cygnus-ssp.h
>

Back to linux.kernel | Previous | NextNext in thread | Find similar | Unroll thread


Thread

Re: [PATCH v7 0/3] ASoC: cygnus: Add audio support for Broadcom  Cygnus SoC Scott Branden <scott.branden@broadcom.com> - 2016-05-19 22:40 +0200
  Re: [PATCH v7 0/3] ASoC: cygnus: Add audio support for Broadcom  Cygnus SoC Mark Brown <broonie@kernel.org> - 2016-05-20 12:50 +0200

csiph-web