linux-kernel - Re: [PATCH net V1] net: lan743x: Add set RFE read fifo threshold for PCI1x1x chips

lists.openwall.net		lists / announce owl-users owl-dev john-users john-dev passwdqc-users yescrypt popa3d-users / oss-security kernel-hardening musl sabotage tlsify passwords / crypt-dev xvendor / Bugtraq Full-Disclosure linux-kernel linux-netdev linux-ext4 linux-hardening linux-cve-announce PHC
Open Source and information security mailing list archives

Hash Suite: Windows password security audit tool. GUI, reports in PDF.

[<prev] [next>] [<thread-prev] [day] [month] [year] [list]

Message-ID: <20240325195330.630b7605@kernel.org>
Date: Mon, 25 Mar 2024 19:53:30 -0700
From: Jakub Kicinski <kuba@...nel.org>
To: Raju Lakkaraju <Raju.Lakkaraju@...rochip.com>
Cc: <netdev@...r.kernel.org>, <davem@...emloft.net>, <pabeni@...hat.com>,
 <edumazet@...gle.com>, <linux-kernel@...r.kernel.org>, <horms@...nel.org>,
 <bryan.whitehead@...rochip.com>, <UNGLinuxDriver@...rochip.com>
Subject: Re: [PATCH net V1] net: lan743x: Add set RFE read fifo threshold
 for PCI1x1x chips

On Fri, 22 Mar 2024 12:16:50 +0530 Raju Lakkaraju wrote:
> +static int pci11x1x_set_rfe_rd_fifo_threshold(struct lan743x_adapter *adapter)

Let's make this a void function.
It only returns 0 and the caller doesn't check for errors, anyway.

> +{
> +	u16 rev = adapter->csr.id_rev & ID_REV_CHIP_REV_MASK_;
> +
> +	if (rev == ID_REV_CHIP_REV_PCI11X1X_B0_) {
> +		u32 misc_ctl;
> +
> +		misc_ctl = lan743x_csr_read(adapter, MISC_CTL_0);
> +		misc_ctl &= ~MISC_CTL_0_RFE_READ_FIFO_MASK_;
> +		misc_ctl |= FIELD_PREP(MISC_CTL_0_RFE_READ_FIFO_MASK_,
> +				       RFE_RD_FIFO_TH_3_DWORDS);
> +		lan743x_csr_write(adapter, MISC_CTL_0, misc_ctl);
> +	}
> +
> +	return 0;
> +}
> +
>  static int lan743x_hardware_init(struct lan743x_adapter *adapter,
>  				 struct pci_dev *pdev)
>  {
> @@ -3287,6 +3306,7 @@ static int lan743x_hardware_init(struct lan743x_adapter *adapter,
>  		pci11x1x_strap_get_status(adapter);
>  		spin_lock_init(&adapter->eth_syslock_spinlock);
>  		mutex_init(&adapter->sgmii_rw_lock);
> +		pci11x1x_set_rfe_rd_fifo_threshold(adapter);
-- 
pw-bot: cr