From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: X-Spam-Checker-Version: SpamAssassin 3.4.0 (2014-02-07) on aws-us-west-2-korg-lkml-1.web.codeaurora.org Received: from bombadil.infradead.org (bombadil.infradead.org [198.137.202.133]) (using TLSv1.2 with cipher ECDHE-RSA-AES256-GCM-SHA384 (256/256 bits)) (No client certificate requested) by smtp.lore.kernel.org (Postfix) with ESMTPS id 12074E6ADEB for ; Sat, 23 Nov 2024 09:12:40 +0000 (UTC) DKIM-Signature: v=1; a=rsa-sha256; q=dns/txt; c=relaxed/relaxed; d=lists.infradead.org; s=bombadil.20210309; h=Sender:List-Subscribe:List-Help :List-Post:List-Archive:List-Unsubscribe:List-Id:In-Reply-To: Content-Transfer-Encoding:Content-Type:MIME-Version:References:Message-ID: Subject:Cc:To:From:Date:Reply-To:Content-ID:Content-Description:Resent-Date: Resent-From:Resent-Sender:Resent-To:Resent-Cc:Resent-Message-ID:List-Owner; bh=hth4LIFfFqiX8BxvmFXuMrsAdL3uE4QFJCpdQRVIJ/k=; b=MZyxz3MYypvjI0GZgVU4NSKpAc T9cGGRS2e3/2BoLYyBVqQPr6aBGd8hB8jSzXLPUerpyjPKq8BhwUKD27ANUufbpUIJfaNFOkwA6qS D95nBWJ4R4ZBqDKV9jVUp2ktFAgO1BNYCEmIquQnOIwetwZlfShK4twNJg3+PYJBk9erlGSd07dng dd+WAcnikBIwfFR5ig5LXLDn26BDO0kKvcGtl2IZJQBezy5j5hFqS5NkLn6xDEHgkeZfbZ5oIyU/E 6fqwlM1Ovw80p24pTE93Tic1IQWZII+lwf5ynAWFRo8Ta6xRIoFsr6RLgK1wVBHcpqNe5wmxeSH6L l7mxjvXQ==; Received: from localhost ([::1] helo=bombadil.infradead.org) by bombadil.infradead.org with esmtp (Exim 4.98 #2 (Red Hat Linux)) id 1tEmBq-000000040wU-1rQt; Sat, 23 Nov 2024 09:12:30 +0000 Received: from mail-pg1-x52e.google.com ([2607:f8b0:4864:20::52e]) by bombadil.infradead.org with esmtps (Exim 4.98 #2 (Red Hat Linux)) id 1tEmAc-000000040mO-161x for linux-arm-kernel@lists.infradead.org; Sat, 23 Nov 2024 09:11:15 +0000 Received: by mail-pg1-x52e.google.com with SMTP id 41be03b00d2f7-7fbbe0fb0b8so2031915a12.0 for ; Sat, 23 Nov 2024 01:11:13 -0800 (PST) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=linaro.org; s=google; t=1732353073; x=1732957873; darn=lists.infradead.org; h=in-reply-to:content-transfer-encoding:content-disposition :mime-version:references:message-id:subject:cc:to:from:date:from:to :cc:subject:date:message-id:reply-to; bh=hth4LIFfFqiX8BxvmFXuMrsAdL3uE4QFJCpdQRVIJ/k=; b=MGns4JoBnHdkzy6pswfWTDa0pAfLqX6J8I/Z0cN3zTeDKo9c4MoRyBCJy+j2I5Xk/o kPNb6WPeMYwLoHxcEa+q0Hj5TY95eEqc4qaJAD1q4N3wBGYAm8nLOifGOJ9opTm+p3CY dRkApmxqmy215zfkazhF1lYcuVDcrW3tVlP8fzReDDWZhA+GgwY1CAJJk9lpOIZh5/FM KdnNqkLssKZY7BYsCPrWLP7DzDQXrMf0yvrTpknUSe+1r8ysaaDu0ZJbvd1Hnf29TF6T bwqheXf+2IMCznZLNozHcOGc6VY+S2xntxh8NbhZqktkQDNSB06YPgNOzYPMLjQJiEwM +IIg== X-Google-DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=1e100.net; s=20230601; t=1732353073; x=1732957873; h=in-reply-to:content-transfer-encoding:content-disposition :mime-version:references:message-id:subject:cc:to:from:date :x-gm-message-state:from:to:cc:subject:date:message-id:reply-to; bh=hth4LIFfFqiX8BxvmFXuMrsAdL3uE4QFJCpdQRVIJ/k=; b=XA4SRt2GIqOYtS4QEUkTBQ0c84gOc6/HD92RxTQl61ar9KMQNitSoaTBmDC4G8K/Me qTqjNR6pqxVXqaEicoPK+zlhStbc5qfqa1n2gnnjTgu92/2KlEy70t1Ez0BZvZmsHpko InPejnMRCiYWLMLqd7gY9h0Y+jfXHF/mW/qNy5yM3QDOcqwI3FUxn52+N9VuRTKryg0Q cgJJrIr2rBQM7uPPZ5C7esVzfLbGw65XMMz/xWAvC3aUjxa/I67BHj6zWLB0viACLCPk CYV4AjD51K0YV2w5XqSUOEjaGMsm8c+mUZi24lUO3oIfgP8+b1AmoJEkDOqyryzMFV+a utSw== X-Forwarded-Encrypted: i=1; AJvYcCVx7srPZOZQEcSOmibuJTEdBAQLdpUpS7ZvB2690u1Uudqv818e5OgcvO9XcwLkPAxO8Aet2hTyFEpuF5w2UiR3@lists.infradead.org X-Gm-Message-State: AOJu0YwCy6gL1Cys0VSPx7Vwlr/3Hpyu5RHg5ca+F94FSXVsSkEWCfgW vkuELG5gbPVNLetN/FL/nCyKWMisQY69t7dyh9GtBkFU7NdOywa15iTXaq034Q== X-Gm-Gg: ASbGncu7tJpOcH9Iv5LFReILHQOS6xG+T3QNNxs1zX+enEM0KKic9DatsVgDNr4MxaY IqSrXOEO3pu5+ksrhfO3YFuEdWYerrfcgXUiQHRNAmv+p7G94p1U4kggzXtjUqcqaYuXvwO1Ytq HThw+4UofHrH89u15ZJFpuoZ8sap0Mk2JX3AEl9h1xVOUpyvlu6HLLapWPNcIuCai8Kxy6aRZcZ yjDgJ0+84y73eH5OOxx1D9vYR+3e3vU02i1nzQEwuNKvc9j6UsOxgG1dKpN1GtUqw== X-Google-Smtp-Source: AGHT+IHSwQx5I6ydm2NN+J7aExyMX6mGiatTpiQcH3JLyc+Ei93oibZfVSEA6ImzJbB21LtSvMRsqw== X-Received: by 2002:a05:6a20:a112:b0:1dc:32a:d409 with SMTP id adf61e73a8af0-1e09e5cba7dmr7832706637.39.1732353073302; Sat, 23 Nov 2024 01:11:13 -0800 (PST) Received: from thinkpad ([2409:40f2:101e:13d7:85cf:a1c4:6490:6f75]) by smtp.gmail.com with ESMTPSA id 41be03b00d2f7-7fbcbdb796esm2566484a12.0.2024.11.23.01.11.08 (version=TLS1_3 cipher=TLS_AES_256_GCM_SHA384 bits=256/256); Sat, 23 Nov 2024 01:11:12 -0800 (PST) Date: Sat, 23 Nov 2024 14:41:06 +0530 From: Manivannan Sadhasivam To: Lorenzo Bianconi Cc: Ryder Lee , Jianjun Wang , Lorenzo Pieralisi , Krzysztof =?utf-8?Q?Wilczy=C5=84ski?= , Rob Herring , Bjorn Helgaas , Philipp Zabel , Matthias Brugger , AngeloGioacchino Del Regno , Michael Turquette , Stephen Boyd , linux-pci@vger.kernel.org, linux-mediatek@lists.infradead.org, linux-arm-kernel@lists.infradead.org, linux-clk@vger.kernel.org Subject: Re: [PATCH v4 6/6] PCI: mediatek-gen3: rely on msleep() in mtk_pcie_en7581_power_up() Message-ID: <20241123091106.bpkzqjafvxa5yief@thinkpad> References: <20241118-pcie-en7581-fixes-v4-0-24bb61703ad7@kernel.org> <20241118-pcie-en7581-fixes-v4-6-24bb61703ad7@kernel.org> MIME-Version: 1.0 Content-Type: text/plain; charset=utf-8 Content-Disposition: inline Content-Transfer-Encoding: 8bit In-Reply-To: <20241118-pcie-en7581-fixes-v4-6-24bb61703ad7@kernel.org> X-CRM114-Version: 20100106-BlameMichelson ( TRE 0.8.0 (BSD) ) MR-646709E3 X-CRM114-CacheID: sfid-20241123_011114_299402_D859BD88 X-CRM114-Status: GOOD ( 18.05 ) X-BeenThere: linux-arm-kernel@lists.infradead.org X-Mailman-Version: 2.1.34 Precedence: list List-Id: List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , Sender: "linux-arm-kernel" Errors-To: linux-arm-kernel-bounces+linux-arm-kernel=archiver.kernel.org@lists.infradead.org On Mon, Nov 18, 2024 at 09:04:58AM +0100, Lorenzo Bianconi wrote: > Since mtk_pcie_en7581_power_up() runs in non-atomic context, rely on > msleep() routine instead of mdelay(). > > Reviewed-by: AngeloGioacchino Del Regno > Signed-off-by: Lorenzo Bianconi Reviewed-by: Manivannan Sadhasivam - Mani > --- > drivers/pci/controller/pcie-mediatek-gen3.c | 4 ++-- > 1 file changed, 2 insertions(+), 2 deletions(-) > > diff --git a/drivers/pci/controller/pcie-mediatek-gen3.c b/drivers/pci/controller/pcie-mediatek-gen3.c > index f47c0f2995d94ea99bf41146657bd90b87781a7c..69f3143783686e9ebcc7ce3dff1883fa6c80d0f4 100644 > --- a/drivers/pci/controller/pcie-mediatek-gen3.c > +++ b/drivers/pci/controller/pcie-mediatek-gen3.c > @@ -926,7 +926,7 @@ static int mtk_pcie_en7581_power_up(struct mtk_gen3_pcie *pcie) > * Wait for the time needed to complete the bulk assert in > * mtk_pcie_setup for EN7581 SoC. > */ > - mdelay(PCIE_EN7581_RESET_TIME_MS); > + msleep(PCIE_EN7581_RESET_TIME_MS); > > /* > * Unlike the other MediaTek Gen3 controllers, the Airoha EN7581 > @@ -954,7 +954,7 @@ static int mtk_pcie_en7581_power_up(struct mtk_gen3_pcie *pcie) > * Wait for the time needed to complete the bulk de-assert above. > * This time is specific for EN7581 SoC. > */ > - mdelay(PCIE_EN7581_RESET_TIME_MS); > + msleep(PCIE_EN7581_RESET_TIME_MS); > > /* MAC power on and enable transaction layer clocks */ > reset_control_deassert(pcie->mac_reset); > > -- > 2.47.0 > -- மணிவண்ணன் சதாசிவம்