From mboxrd@z Thu Jan 1 00:00:00 1970 From: Ferruh Yigit Subject: Re: [PATCHv6 00/33] NXP DPAA2 PMD Date: Tue, 24 Jan 2017 11:25:04 +0000 Message-ID: References: <1484832240-2048-1-git-send-email-hemant.agrawal@nxp.com> <1485172803-17288-1-git-send-email-hemant.agrawal@nxp.com> <04750e9b-5d19-f743-bd80-1994381c7e88@intel.com> Mime-Version: 1.0 Content-Type: text/plain; charset=utf-8 Content-Transfer-Encoding: 8bit Cc: thomas.monjalon@6wind.com, bruce.richardson@intel.com, shreyansh.jain@nxp.com, john.mcnamara@intel.com, jerin.jacob@caviumnetworks.com To: Hemant Agrawal , dev@dpdk.org Return-path: Received: from mga06.intel.com (mga06.intel.com [134.134.136.31]) by dpdk.org (Postfix) with ESMTP id 4EE5C108D for ; Tue, 24 Jan 2017 12:25:07 +0100 (CET) In-Reply-To: <04750e9b-5d19-f743-bd80-1994381c7e88@intel.com> List-Id: DPDK patches and discussions List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , Errors-To: dev-bounces@dpdk.org Sender: "dev" On 1/23/2017 5:58 PM, Ferruh Yigit wrote: > On 1/23/2017 11:59 AM, Hemant Agrawal wrote: > <...> > >> >> Hemant Agrawal (33): >> mk/dpaa2: add the crc support to the machine type >> drivers/common/dpaa2: adding qbman driver >> bus/fslmc: introducing fsl-mc bus driver >> bus/fslmc: introduce mc object functions >> bus/fslmc: add mc dpni object support >> bus/fslmc: add mc dpio object support >> bus/fslmc: add mc dpbp object support >> bus/fslmc: add mc dpseci object support >> eal/vfio: adding vfio utility functions in map file >> bus/fslmc: add vfio support >> bus/fslmc: scan for net and sec devices >> net/dpaa2: introducing NXP dpaa2 pmd driver >> doc: add dpaa2 nic details >> bus/fslmc: add debug log message support >> drivers/common/dpaa2: dpio portal driver >> drivers/pool/dpaa2: adding hw offloaded mempool >> drivers/common/dpaa2: dpio routine to affine to crypto threads >> net/dpaa2: adding eth ops to dpaa2 >> net/dpaa2: add rss flow distribution >> net/dpaa2: configure mac address at init >> net/dpaa2: attach the buffer pool to dpni >> net/dpaa2: add support for l3 and l4 checksum offload >> net/dpaa2: add support for promiscuous mode >> net/dpaa2: add mtu config support >> net/dpaa2: add packet rx and tx support >> net/dpaa2: rx packet parsing and packet type support >> net/dpaa2: link status update >> net/dpaa2: basic stats support >> net/dpaa2: enable stashing for LS2088A devices >> net/dpaa2: add support for non hw buffer pool packet transmit >> net/dpaa2: enabling the use of physical addresses >> bus/fslmc: add support for dmamap to ARM SMMU >> drivers/common/dpaa2: frame queue based dq storage alloc >> > > devtools/check-git-log.sh gives following errors: > > Wrong headline prefix: > bus/fslmc: add debug log message support Some of these warnings are because of the assumption that the scope of changed files are limited to the patch title tag. For example, for this patch, because of the "bus/fslmc:", it is expected that all modified files are under "drivers/bus/fslmc" folder, but this patch modifies: bus/fslmc/* common/dpaa2/* net/dpaa2/* I can guess different dpaa2 modules (bus/common/net) has dependencies to each other, and may not always be possible to separate them. This needs to be investigated per patch. But the more they are separated, easier to review / understand them. And I am aware this is easier to say than to do it. thanks, ferruh > drivers/common/dpaa2: dpio portal driver Or this one, scope is "drivers/common/dpaa2", but all the modifies files are under "/drivers/bus/*". > drivers/common/dpaa2: dpio routine to affine to crypto threads > net/dpaa2: adding eth ops to dpaa2 > net/dpaa2: attach the buffer pool to dpni > net/dpaa2: add support for l3 and l4 checksum offload > net/dpaa2: add mtu config support > net/dpaa2: add packet rx and tx support > net/dpaa2: enabling the use of physical addresses > bus/fslmc: add support for dmamap to ARM SMMU > drivers/common/dpaa2: frame queue based dq storage alloc > Wrong headline lowercase: > net/dpaa2: introducing NXP dpaa2 pmd driver > doc: add dpaa2 nic details > drivers/pool/dpaa2: adding hw offloaded mempool > net/dpaa2: add rss flow distribution > net/dpaa2: configure mac address at init > net/dpaa2: add support for l3 and l4 checksum offload > net/dpaa2: add mtu config support > net/dpaa2: add packet rx and tx support > net/dpaa2: rx packet parsing and packet type support > net/dpaa2: add support for non hw buffer pool packet transmit > Headline too long: > drivers/common/dpaa2: dpio routine to affine to crypto threads > net/dpaa2: add support for non hw buffer pool packet transmit > > <...> > > >