> -----Original Message-----
> From: Jakub Kicinski <k...@kernel.org>
> Sent: Thursday, July 9, 2020 5:49 AM
> To: Ooi, Joyce <joyce....@intel.com>
> Cc: Thor Thayer <thor.tha...@linux.intel.com>; David S . Miller
> <da...@davemloft.net>; netdev@vger.kernel.org; linux-
> ker...@vger.kernel.org; Dalon Westergreen
> <dalon.westergr...@linux.intel.com>; Tan, Ley Foon
> <ley.foon....@intel.com>; See, Chin Liang <chin.liang....@intel.com>;
> Nguyen, Dinh <dinh.ngu...@intel.com>; Westergreen, Dalon
> <dalon.westergr...@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.

Reply via email to