From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: from mail-ig0-f169.google.com (mail-ig0-f169.google.com [209.85.213.169]) by dpdk.org (Postfix) with ESMTP id 9A31429C7 for ; Thu, 3 Mar 2016 02:04:58 +0100 (CET) Received: by mail-ig0-f169.google.com with SMTP id g6so8101695igt.1 for ; Wed, 02 Mar 2016 17:04:58 -0800 (PST) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=broadcom.com; s=google; h=mime-version:in-reply-to:references:date:message-id:subject:from:to :cc; bh=Sw85NhUj8hHTlP0s0K85HEAJDtfj8+IAhhVASzoQJRw=; b=ALTXiZT2GMulHKVsmsjPnxJOCh6YrV//JyfI7gI+MxMOMBPnJoJwKLRXgZhX3z3zak 3RMrefiPZCCb2sOY4VzFF1ysD2t3gKSL57V5CMLFquzH0UjpUNuAgPpYG0rPbHnEfH/k UgBZKckij3TlJ9xWcj1k1JQUVOuxmvsTjXWfA= X-Google-DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=1e100.net; s=20130820; h=x-gm-message-state:mime-version:in-reply-to:references:date :message-id:subject:from:to:cc; bh=Sw85NhUj8hHTlP0s0K85HEAJDtfj8+IAhhVASzoQJRw=; b=GdFDH4uaUscb+WRJA/Jlk8YSqg8QWQ8jW591i845kgX8K2aFRBF1HCluJMc0ukZmO+ bIxPFFufzGHm81HRf/i5EQn1gCpvlyGN4EUTby92AT7kl/qpK3TmPUBmrJoqMoJZfDrm BFxp/9gGkGq+Nf99fQoRDMBhUUPU8AJhEvvp0xVIxuy7d5vdZRqtY6jr15cyFNHBKs+a RkM+0i4pbk8gQ2M+BPERY1N+dn16ARvksTteKaquoxNT9Ey/8KDBHodgeM2Up62i/Vr/ fuJDGuCblbTAUCPasxC/m3WPOS7UELHciy19EB29UHMyGCFOPmUFKRylghwwB9c4kkJs 9YRg== X-Gm-Message-State: AD7BkJJ9Pnw3CE7hQ8VZPw9QiY8ACLaZu+YIBewB0FCZACw0aUhKnD+rWCd7rJaNT6XJ0lGOi2w1jh5yfD5zr9ve MIME-Version: 1.0 X-Received: by 10.50.143.40 with SMTP id sb8mr327330igb.53.1456967097934; Wed, 02 Mar 2016 17:04:57 -0800 (PST) Received: by 10.36.104.7 with HTTP; Wed, 2 Mar 2016 17:04:57 -0800 (PST) In-Reply-To: <3312508.fJgjDDEF6G@xps13> References: <3156888.xIWalfADA6@xps13> <3312508.fJgjDDEF6G@xps13> Date: Wed, 2 Mar 2016 17:04:57 -0800 Message-ID: From: Stephen Hurd To: Thomas Monjalon Content-Type: text/plain; charset=UTF-8 X-Content-Filtered-By: Mailman/MimeDel 2.1.15 Cc: dev@dpdk.org, David Christensen , Carl Tung Subject: Re: [dpdk-dev] New driver (large patch) question. X-BeenThere: dev@dpdk.org X-Mailman-Version: 2.1.15 Precedence: list List-Id: patches and discussions about DPDK List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , X-List-Received-Date: Thu, 03 Mar 2016 01:04:59 -0000 On Wed, Mar 2, 2016 at 4:29 PM, Thomas Monjalon wrote: > > Should I split additional definitions/documentation that's not currently > > used in the driver as well? Or should it stay as only enough to document > > what the driver already does? > > I don't understand the question. > If something is not used, it should not been there. > Basically, since the header is the only current public documentation on the hardware, if someone without access to the whole header was interested in adding features to the driver, they wouldn't have the documentation to allow them to do so. I understand that a header file in DPDK is not the right way for Broadcom to release documentation on their hardware and firmware, but I was thinking it would be better than nothing. Further, as mentioned in another email, this header file is mechanically derived from one that is planned to be released separately, so it would be a slightly lower support burden to use the mechanically derived one rather than manually importing changes. No problems slimming it down though. > > It's a fairly work-intensive project to deconstruct the existing driver > > into a series of small patches that work at each step, is this a hard > > requirement? (if so, I'd better get cracking) > > There is no hard requirement. I'm just giving you some advices to get > some reviewers and make them confident when accepting your patches. > By the way, you would get more attention by introducing the device with > some web links and performance numbers in the cover letter. > It is also appreciated to provide a documentation in doc/guides/nics/. > You could also fill the (new) table in overview.rst. > Thanks for the pointers, I'll try to get as many of the suggestions done as possible and talk to marketing to find web resources for the device and its performance. -- Stephen Hurd Principal Engineer - Software Development Broadcom Corporation 949-926-8039 stephen.hurd@broadcom.com