Re: [PATCH V2 19/28] PCI: tegra: Change PRSNT_SENSE irq log to debug

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 



On Tue, Apr 23, 2019 at 02:58:16PM +0530, Manikanta Maddireddy wrote:
> PRSNT_MAP bit field is programmed to update the slot present status.
> PRSNT_SENSE irq is triggered when this bit field is programmed, which is

s/irq/IRQ/

> not an error. Add a new if condition to trap RSNT_SENSE code and print it

s/RSNT_SENSE/PRSNT_SENSE/

With that:

Acked-by: Thierry Reding <treding@xxxxxxxxxx>

> with debug log level.
> 
> Signed-off-by: Manikanta Maddireddy <mmaddireddy@xxxxxxxxxx>
> ---
> V2: If-else to switch-case conversion patch is dropped, this patch is
> rebased to stay with if-else statements
> 
>  drivers/pci/controller/pci-tegra.c | 2 +-
>  1 file changed, 1 insertion(+), 1 deletion(-)
> 
> diff --git a/drivers/pci/controller/pci-tegra.c b/drivers/pci/controller/pci-tegra.c
> index 9b841b0392ac..72634ce6d3da 100644
> --- a/drivers/pci/controller/pci-tegra.c
> +++ b/drivers/pci/controller/pci-tegra.c
> @@ -864,7 +864,7 @@ static irqreturn_t tegra_pcie_isr(int irq, void *arg)
>  	 * do not pollute kernel log with master abort reports since they
>  	 * happen a lot during enumeration
>  	 */
> -	if (code == AFI_INTR_MASTER_ABORT)
> +	if (code == AFI_INTR_MASTER_ABORT || code == AFI_INTR_PE_PRSNT_SENSE)
>  		dev_dbg(dev, "%s, signature: %08x\n", err_msg[code], signature);
>  	else
>  		dev_err(dev, "%s, signature: %08x\n", err_msg[code], signature);
> -- 
> 2.17.1
> 

Attachment: signature.asc
Description: PGP signature


[Index of Archives]     [DMA Engine]     [Linux Coverity]     [Linux USB]     [Video for Linux]     [Linux Audio Users]     [Yosemite News]     [Linux Kernel]     [Linux SCSI]     [Greybus]

  Powered by Linux