[prev in list] [next in list] [prev in thread] [next in thread]
List: dpdk-dev
Subject: Re: [dpdk-dev] [PATCH 23/25] net/qede/base: semantic/formatting changes
From: "Mody, Rasesh" <Rasesh.Mody () cavium ! com>
Date: 2016-12-31 7:41:59
Message-ID: BLUPR0701MB1572EE33C9AAAB45896FC4C19F6D0 () BLUPR0701MB1572 ! namprd07 ! prod ! outlook ! com
[Download RAW message or body]
> From: Ferruh Yigit [mailto:ferruh.yigit@intel.com]
> Sent: Friday, December 23, 2016 7:42 AM
>
> On 12/3/2016 9:11 AM, Rasesh Mody wrote:
> > This patch consists of semantic/formatting changes. It also includes
> > comment additions.
>
> As far as I can see majority of the changes are formatting, but not all.
>
> Functional changes are hard to detect in this patch, what do you think
> separating formatting/comments patches into another patch, so functional
> changes can become more visible?
There are few of places(ecore_hw_bar_size(), ecore_get_hw_info() and \
ecore_init_cmd_*), where there is a bit of code refactoring. However, they are not a \
major change. We have tried to isolate most of the functional changes and made them \
part of the separate patches as fit. I think, we can include a bit of description in \
commit message to cover it in this patch. Please let me know if you think otherwise.
> >
> > Signed-off-by: Rasesh Mody <Rasesh.Mody@cavium.com>
> > ---
> <...>
[prev in list] [next in list] [prev in thread] [next in thread]
Configure |
About |
News |
Add a list |
Sponsored by KoreLogic