Received-SPF: pass (google.com: best guess record for domain of linux-kernel-owner@vger.kernel.org designates 209.132.180.67 as permitted sender) client-ip=209.132.180.67;
Received-SPF: None (protection.outlook.com: amd.com does not designate
 permitted sender hosts)
From:   Ravulapati Vishnu vardhan rao 
        <Vishnuvardhanrao.Ravulapati@amd.com>
CC:     <Alexander.Deucher@amd.com>,
        Ravulapati Vishnu vardhan rao 
        <Vishnuvardhanrao.Ravulapati@amd.com>,
        Liam Girdwood <lgirdwood@gmail.com>,
        Mark Brown <broonie@kernel.org>,
        Jaroslav Kysela <perex@perex.cz>,
        "Takashi Iwai" <tiwai@suse.com>,
        Vijendar Mukunda <Vijendar.Mukunda@amd.com>,
        "Maruthi Srinivas Bayyavarapu" <Maruthi.Bayyavarapu@amd.com>,
        Colin Ian King <colin.king@canonical.com>,
        Dan Carpenter <dan.carpenter@oracle.com>,
        "moderated list:SOUND - SOC LAYER / DYNAMIC AUDIO POWER MANAGEM..." 
        <alsa-devel@alsa-project.org>,
        open list <linux-kernel@vger.kernel.org>
Subject: [PATCH] ASoC: amd: Missing Initialization of IRQFLAGS
Date:   Fri, 27 Sep 2019 05:34:47 +0530
Message-ID: <1569542689-25512-1-git-send-email-Vishnuvardhanrao.Ravulapati@amd.com>
In-Reply-To: <Fix for missing initialization of IRQFLAGS.>
References: <Fix for missing initialization of IRQFLAGS.>
MIME-Version: 1.0
Content-Type: text/plain
X-MS-Exchange-CrossTenant-OriginalArrivalTime: 26 Sep 2019 13:11:40.5842
 (UTC)
X-MS-Exchange-CrossTenant-Network-Message-Id: ed66ccfd-dab3-4e17-7d34-08d742831210
X-MS-Exchange-CrossTenant-Id: 3dd8961f-e488-4e60-8e11-a82d994e183d
X-MS-Exchange-CrossTenant-OriginalAttributedTenantConnectingIp: TenantId=3dd8961f-e488-4e60-8e11-a82d994e183d;Ip=[165.204.84.17];Helo=[SATLEXCHOV01.amd.com]
X-MS-Exchange-CrossTenant-FromEntityHeader: HybridOnPrem
X-MS-Exchange-Transport-CrossTenantHeadersStamped: MWHPR1201MB0014
To:     unlisted-recipients:; (no To-header on input)
Sender: linux-kernel-owner@vger.kernel.org
Precedence: bulk
List-ID: <linux-kernel.vger.kernel.org>
X-Mailing-List: linux-kernel@vger.kernel.org

Fix for missing initialization of IRQFLAGS in
ACP-PCI driver and Missing Macro of ACP3x_DEVS.

Follow up to IDb33df346

Signed-off-by: Ravulapati Vishnu vardhan rao <Vishnuvardhanrao.Ravulapati@amd.com>
---
 sound/soc/amd/raven/pci-acp3x.c | 8 +++++---
 1 file changed, 5 insertions(+), 3 deletions(-)

diff --git a/sound/soc/amd/raven/pci-acp3x.c b/sound/soc/amd/raven/pci-acp3x.c
index 627798a..3887ea0 100644
--- a/sound/soc/amd/raven/pci-acp3x.c
+++ b/sound/soc/amd/raven/pci-acp3x.c
@@ -219,7 +219,7 @@ static int snd_acp3x_probe(struct pci_dev *pci,
 				sizeof(struct resource) * 4,
 						GFP_KERNEL);
 		adata->cell = devm_kzalloc(&pci->dev,
-				sizeof(struct mfd_cell) * 3,
+				sizeof(struct mfd_cell) * ACP3x_DEVS,
 						GFP_KERNEL);
 		if (!adata->cell) {
 			ret = -ENOMEM;
@@ -260,6 +260,7 @@ static int snd_acp3x_probe(struct pci_dev *pci,
 		adata->res[3].flags	= IORESOURCE_IRQ;
 		adata->res[3].start	= pci->irq;
 		adata->res[3].end	= adata->res[3].start;
+		irqflags		= 0;
 
 		adata->acp3x_audio_mode	= ACP3x_I2S_MODE;
 
@@ -282,8 +283,9 @@ static int snd_acp3x_probe(struct pci_dev *pci,
 		adata->cell[2].platform_data	= &i2s_pdata[1];
 		adata->cell[2].pdata_size	=
 				sizeof(struct i2s_platform_data);
-		r = mfd_add_hotplug_devices(adata->parent, adata->cell,	3);
-		for (i = 0; i < 3 ; i++)
+		r = mfd_add_hotplug_devices(adata->parent, adata->cell,
+								ACP3x_DEVS);
+		for (i = 0; i < ACP3x_DEVS ; i++)
 			dev = get_mfd_cell_dev(adata->cell[i].name, i);
 		break;
 
-- 
2.7.4