From mboxrd@z Thu Jan 1 00:00:00 1970 From: Thomas Monjalon Subject: Re: [PATCH 1/2] ethdev: add callback to get register size in bytes Date: Fri, 27 May 2016 16:43:51 +0200 Message-ID: <3332485.q3f48ryagT@xps13> References: <1462963714-21022-1-git-send-email-zr@semihalf.com> <1464158214-24733-1-git-send-email-zr@semihalf.com> Mime-Version: 1.0 Content-Type: text/plain; charset="us-ascii" Content-Transfer-Encoding: 7Bit Cc: zr@semihalf.com, remy.horton@intel.com, dev@dpdk.org To: Panu Matilainen Return-path: Received: from mail-wm0-f42.google.com (mail-wm0-f42.google.com [74.125.82.42]) by dpdk.org (Postfix) with ESMTP id 9A1D729B6 for ; Fri, 27 May 2016 16:43:53 +0200 (CEST) Received: by mail-wm0-f42.google.com with SMTP id n129so140305288wmn.1 for ; Fri, 27 May 2016 07:43:53 -0700 (PDT) In-Reply-To: List-Id: patches and discussions about DPDK List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , Errors-To: dev-bounces@dpdk.org Sender: "dev" 2016-05-27 13:28, Panu Matilainen: > On 05/25/2016 09:36 AM, zr@semihalf.com wrote: > > @@ -1455,6 +1458,8 @@ struct eth_dev_ops { > > > > eth_get_reg_length_t get_reg_length; > > /**< Get # of registers */ > > + eth_get_reg_width_t get_reg_width; > > + /**< Get # of bytes in register */ > > eth_get_reg_t get_reg; > > /**< Get registers */ > > eth_get_eeprom_length_t get_eeprom_length; > > This is an ABI break, but maybe it is part of that "driver > implementation API" which is exempt from the ABI/API policies. Thomas? Yes dev_ops are for drivers, not for applications. Thus it should not be impacted by the ABI policy.