From mboxrd@z Thu Jan 1 00:00:00 1970 From: Vinod Koul Subject: Re: [PATCH 4/5] ASoC: Intel: bxtn: Assign sram address to sst structure Date: Thu, 29 Jun 2017 11:25:30 +0530 Message-ID: <20170629055530.GF19154@localhost> References: <20170629030408.4383-1-guneshwor.o.singh@intel.com> <20170629030408.4383-5-guneshwor.o.singh@intel.com> Mime-Version: 1.0 Content-Type: text/plain; charset="us-ascii" Content-Transfer-Encoding: 7bit Return-path: Received: from mga05.intel.com (mga05.intel.com [192.55.52.43]) by alsa0.perex.cz (Postfix) with ESMTP id 42E7526727D for ; Thu, 29 Jun 2017 07:52:40 +0200 (CEST) Content-Disposition: inline In-Reply-To: <20170629030408.4383-5-guneshwor.o.singh@intel.com> List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , Errors-To: alsa-devel-bounces@alsa-project.org Sender: alsa-devel-bounces@alsa-project.org To: Guneshwor Singh Cc: Takashi Iwai , liam.r.girdwood@linux.intel.com, alsa-devel@alsa-project.org, Mark Brown , Patches Audio List-Id: alsa-devel@alsa-project.org On Thu, Jun 29, 2017 at 08:34:07AM +0530, Guneshwor Singh wrote: > To read dsp registers via debugfs, assign sram address > in dsp_init(). > > Signed-off-by: Guneshwor Singh > --- > sound/soc/intel/skylake/bxt-sst.c | 4 ++++ > 1 file changed, 4 insertions(+) > > diff --git a/sound/soc/intel/skylake/bxt-sst.c b/sound/soc/intel/skylake/bxt-sst.c > index f5e7dbb1ba39..cf11b84888b9 100644 > --- a/sound/soc/intel/skylake/bxt-sst.c > +++ b/sound/soc/intel/skylake/bxt-sst.c > @@ -573,6 +573,10 @@ int bxt_sst_dsp_init(struct device *dev, void __iomem *mmio_base, int irq, > sst->fw_ops = bxt_fw_ops; > sst->addr.lpe = mmio_base; > sst->addr.shim = mmio_base; > + sst->addr.sram0_base = BXT_ADSP_SRAM0_BASE; > + sst->addr.sram1_base = BXT_ADSP_SRAM1_BASE; > + sst->addr.w0_stat_sz = SKL_ADSP_W0_STAT_SZ; > + sst->addr.w0_up_sz = SKL_ADSP_W0_UP_SZ; I think you need to modfiy the order and this and following change. With last patch applied and this not present, you would break if someone tries to read debugfs file as these adresses are NULL, so add these values first and then the usage :) > > sst_dsp_mailbox_init(sst, (BXT_ADSP_SRAM0_BASE + SKL_ADSP_W0_STAT_SZ), > SKL_ADSP_W0_UP_SZ, BXT_ADSP_SRAM1_BASE, SKL_ADSP_W1_SZ); > -- > 2.13.0 > -- ~Vinod