From mboxrd@z Thu Jan 1 00:00:00 1970 From: Stephen Hemminger Subject: Re: [PATCH v5 1/6] ethdev: add rx interrupt enable/disable functions Date: Mon, 23 Feb 2015 08:59:18 -0800 Message-ID: <20150223085918.0b9853a9@urahara> References: <1424710542-14637-1-git-send-email-danny.zhou@intel.com> <1424710542-14637-2-git-send-email-danny.zhou@intel.com> Mime-Version: 1.0 Content-Type: text/plain; charset=US-ASCII Content-Transfer-Encoding: 7bit Cc: dev-VfR2kkLFssw@public.gmane.org To: Zhou Danny Return-path: In-Reply-To: <1424710542-14637-2-git-send-email-danny.zhou-ral2JQCrhuEAvxtiuMwx3w@public.gmane.org> List-Id: patches and discussions about DPDK List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , Errors-To: dev-bounces-VfR2kkLFssw@public.gmane.org Sender: "dev" On Tue, 24 Feb 2015 00:55:37 +0800 Zhou Danny wrote: > +int > +rte_eth_dev_rx_queue_intr_enable(uint8_t port_id, > + uint16_t queue_id) > +{ > + struct rte_eth_dev *dev; > + > + if (port_id >= nb_ports) { > + PMD_DEBUG_TRACE("Invalid port_id=%d\n", port_id); > + return (-ENODEV); Please don't use the BSD style of extra useless paren's around returns.