From mboxrd@z Thu Jan  1 00:00:00 1970
From: Andrew Lunn <andrew@lunn.ch>
Subject: Re: [RFC] ethtool: Support for driver private ioctl's
Date: Fri, 6 Apr 2018 16:47:01 +0200
Message-ID: <20180406144701.GO17495@lunn.ch>
References: <cbd05f37-509b-118b-e681-0ccd0ebebd73@synopsys.com>
 <df7773c4-8695-38dd-bedd-39b88f4aaec1@gmail.com>
 <27c05ec7-1f85-8e4a-be06-70d6d80e8a10@synopsys.com>
Mime-Version: 1.0
Content-Type: text/plain; charset=us-ascii
Cc: Florian Fainelli <f.fainelli@gmail.com>,
        David Miller <davem@redhat.com>,
        Jakub Jelinek <jj@ultra.linux.cz>,
        Jeff Garzik <jgarzik@pobox.com>, Tim Hockin <thockin@sun.com>,
        Eli Kupermann <eli.kupermann@intel.com>,
        Chris Leech <christopher.leech@intel.com>,
        Scott Feldman <scott.feldman@intel.com>,
        Ben Hutchings <ben@decadent.org.uk>, netdev@vger.kernel.org,
        Joao Pinto <Joao.Pinto@synopsys.com>
To: Jose Abreu <Jose.Abreu@synopsys.com>
Return-path: <netdev-owner@vger.kernel.org>
Received: from vps0.lunn.ch ([185.16.172.187]:52431 "EHLO vps0.lunn.ch"
        rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP
        id S1752489AbeDFOr2 (ORCPT <rfc822;netdev@vger.kernel.org>);
        Fri, 6 Apr 2018 10:47:28 -0400
Content-Disposition: inline
In-Reply-To: <27c05ec7-1f85-8e4a-be06-70d6d80e8a10@synopsys.com>
Sender: netdev-owner@vger.kernel.org
List-ID: <netdev.vger.kernel.org>

On Fri, Apr 06, 2018 at 02:51:15PM +0100, Jose Abreu wrote:
> Hi Florian,
> 
> On 05-04-2018 16:50, Florian Fainelli wrote:
> >
> > On 04/05/2018 03:47 AM, Jose Abreu wrote:
> >> Hi All,
> >>
> >> I would like to know your opinion regarding adding support for
> >> driver private ioctl's in ethtool.
> >>
> >> Background: Synopsys Ethernet IP's have a certain number of
> >> features which can be reconfigured at runtime. Giving you two
> >> examples: One of the most recent one is the safety features,
> >> which can be enabled/disabled and forced at runtime.

Hi Jose

Is there a reason somebody would decide to use the Ethernet in
'unsafe' mode? Cannot you just turn it on by default?

	 Andrew