From mboxrd@z Thu Jan 1 00:00:00 1970 From: Ben Hutchings Subject: Re: [PATCH -next] qlcnic: fail when try to setup unsupported features Date: Mon, 28 Jun 2010 15:18:44 +0100 Message-ID: <1277734724.2089.10.camel@achroite.uk.solarflarecom.com> References: <20100628113134.0c5208b0@dhcp-lab-109.englab.brq.redhat.com> <99737F4847ED0A48AECC9F4A1974A4B80F82CB7D46@MNEXMB2.qlogic.org> <20100628145819.74d22d5f@dhcp-lab-109.englab.brq.redhat.com> <99737F4847ED0A48AECC9F4A1974A4B80F82CB7D4E@MNEXMB2.qlogic.org> <20100628161412.7d9d0e4f@dhcp-lab-109.englab.brq.redhat.com> Mime-Version: 1.0 Content-Type: text/plain Content-Transfer-Encoding: 7bit Cc: Amit Salecha , "netdev@vger.kernel.org" , Amerigo Wang , Anirban Chakraborty To: Stanislaw Gruszka Return-path: Received: from mail.solarflare.com ([216.237.3.220]:38319 "EHLO exchange.solarflare.com" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1753067Ab0F1OSr (ORCPT ); Mon, 28 Jun 2010 10:18:47 -0400 In-Reply-To: <20100628161412.7d9d0e4f@dhcp-lab-109.englab.brq.redhat.com> Sender: netdev-owner@vger.kernel.org List-ID: On Mon, 2010-06-28 at 16:14 +0200, Stanislaw Gruszka wrote: [...] > My plan is something like that: > > static const struct ethtool_ops my_ethtool_ops = { > .get_flags = ethtool_op_get_flags, > .set_flags = ethtool_op_set_flags, > .supported_flags = ETH_FLAG_LRO > } > > Plus op->supported_flags check in ethtool_op_set_flags. That will allow > to define flags per driver. There is also possible to add supported_flags > to netdev, but I would like to avoid that - in such case drivers can use > custom .set_flags function. Sounds good to me. Ben. -- Ben Hutchings, Senior Software Engineer, Solarflare Communications Not speaking for my employer; that's the marketing department's job. They asked us to note that Solarflare product names are trademarked.