RE: [PATCH v4 09/10] net: eth: altera: add msgdma prefetcher

Ooi, Joyce
2020-07-14T14:35:25Z

> -----Original Message-----
> From: Jakub Kicinski <kuba@kernel.org>
> Sent: Thursday, July 9, 2020 5:49 AM
> To: Ooi, Joyce <joyce.ooi@intel.com>
> Cc: Thor Thayer <thor.thayer@linux.intel.com>; David S . Miller
> <davem@davemloft.net>; netdev@vger.kernel.org; linux-
> kernel@vger.kernel.org; Dalon Westergreen
> <dalon.westergreen@linux.intel.com>; Tan, Ley Foon
> <ley.foon.tan@intel.com>; See, Chin Liang <chin.liang.see@intel.com>;
> Nguyen, Dinh <dinh.nguyen@intel.com>; Westergreen, Dalon
> <dalon.westergreen@intel.com>
> Subject: Re: [PATCH v4 09/10] net: eth: altera: add msgdma prefetcher
>
> On Wed, 8 Jul 2020 15:24:00 +0800 Ooi, Joyce wrote:
> > + /* get prefetcher rx poll frequency from device tree */
> > + if (of_property_read_u32(pdev->dev.of_node, "rx-poll-
> freq",
> > + &priv->rx_poll_freq)) {
> > + dev_info(&pdev->dev, "Defaulting RX Poll Frequency
> to 128\n");
> > + priv->rx_poll_freq = 128;
> > + }
> > +
> > + /* get prefetcher rx poll frequency from device tree */
> > + if (of_property_read_u32(pdev->dev.of_node, "tx-poll-
> freq",
> > + &priv->tx_poll_freq)) {
> > + dev_info(&pdev->dev, "Defaulting TX Poll Frequency
> to 128\n");
> > + priv->tx_poll_freq = 128;
> > + }
>
> I'm no device tree expert but these look like config options rather than HW
> description. They also don't appear to be documented in the next patch.

The poll_freq are part of the msgdma prefetcher IP, whereby it specifies the frequency of descriptor polling operation.
I can add the poll_freq description in the next patch.