Received-SPF: pass (google.com: domain of linux-kernel-owner@vger.kernel.org designates 2620:137:e000::3:4 as permitted sender) client-ip=2620:137:e000::3:4;
Message-ID: <95a7c916-dd15-624e-3cd4-f9225324df72@intel.com>
Date:   Thu, 28 Sep 2023 15:12:09 -0700
User-Agent: Mozilla/5.0 (Windows NT 10.0; Win64; x64; rv:102.0) Gecko/20100101
 Thunderbird/102.15.1
Subject: Re: [PATCH net-next v2 0/9] i40e: House-keeping and clean-up
Content-Language: en-US
To:     Ivan Vecera <ivecera@redhat.com>, <netdev@vger.kernel.org>
CC:     <poros@redhat.com>, <mschmidt@redhat.com>,
        <anthony.l.nguyen@intel.com>, <davem@davemloft.net>,
        <kuba@kernel.org>, <edumazet@google.com>, <pabeni@redhat.com>,
        <intel-wired-lan@lists.osuosl.org>, <linux-kernel@vger.kernel.org>,
        <przemyslaw.kitszel@intel.com>
References: <20230927083135.3237206-1-ivecera@redhat.com>
From:   Jesse Brandeburg <jesse.brandeburg@intel.com>
In-Reply-To: <20230927083135.3237206-1-ivecera@redhat.com>
Content-Type: text/plain; charset="UTF-8"; format=flowed
Content-Transfer-Encoding: 7bit
MIME-Version: 1.0
X-MS-Exchange-AntiSpam-MessageData-ChunkCount: 1
X-MS-Exchange-AntiSpam-MessageData-0: =?utf-8?B?THl1WUU3R1dnangwUXVOOWVramFieUlMdHBIcUtJdTZYeFJhSUZ4VllWU2l3?=
 =?utf-8?B?WFQ3dTM3dGJtR3l1VHJJa2pOLzcweWJ3a2t6dEt4S2taUHJMTVNBZXU2eHE4?=
 =?utf-8?B?RWR3a3V0L2p2Z3dXT2dSTUllbjdDaHgxNitEaFZhMmR3RlRRRG1ZclE0OGV5?=
 =?utf-8?B?OHNmVU9Xb0tmKzcwU2JWSHFhQU9UTVMxTExBZlVkTEFqTXQ5ZGdYWGk5UU9n?=
 =?utf-8?B?ekZKTHhnNytTOHNjcXpzVi9wSXBQNXJIM3UxSEg0M2NMenVNWk0vK0hpSVhD?=
 =?utf-8?B?THlWcXMrWlpSUm1LaUFiaTRyVmJKd3FEVHRvOXppM1pSV0c0QmtGRE4rdjc1?=
 =?utf-8?B?Mm4wSVNPNU14cjRxMUJaYkUvWnEweXU0T01xelhhRndNM1NRWkFyaC91UnJV?=
 =?utf-8?B?QkdLMTI3TFRRR1NvNWsxOWNqeEl2OUtadlRzdi9TYld0ekRwTGg1ZTBkUEZN?=
 =?utf-8?B?Y0xXQ2M0NkxoeFJ4SURYcWNxa0VYTTRIZmNTRjdUYzRWUnVibmZhNTdEeVVR?=
 =?utf-8?B?RU1mbGZIUHNxemtrK3B2TENDOXc3c1hnOTYyQUk0OWMyRjFLTmkxWUlUY2tT?=
 =?utf-8?B?NVplazR0QkczUnRqTkF5c2I0NlJ6Zk9yQzNwRUN6SjE3SVhRaXJVd1lJbUNB?=
 =?utf-8?B?MlRWaitJNkFIblVENXlJeHhqdUc0MHRDMG1sZWZRaU11WWdFd1NoMW5LQTVm?=
 =?utf-8?B?cER6MzViMTg4VDBaODZWVFQ4M0JGS0VTM3VaNk04N29uWlF2ejFYT3RpRzQr?=
 =?utf-8?B?K3ExaHF0RHZGclJvRFA4V0pFZ2VsTjg4UDVVbEppS0F5UnZNUk1XUjBJZ2ZJ?=
 =?utf-8?B?bVM3U2JPcS9zTWczOG0relFxd2FVa09tcnZCMERJTEhFQXdQQUVaQVluYjVL?=
 =?utf-8?B?ck9qMUhReGNwbWEweTFIc1lrNE1aSTdZMk03WTJzK29zb2hreW1xSlFSc3VT?=
 =?utf-8?B?WkJaY1pxQ280bFhsTnVkbUk5RFpUMmJQbnhTOUhPTXpqam9XcjcwRDJtVVll?=
 =?utf-8?B?dnpBWlAyVld2RDRLd3ZDTjEyaUg2Mzd5YzIrQy9razhBRFFIWHdNTkk4b2pM?=
 =?utf-8?B?K3JvVjU0NlBqK1IrY2lac1M3UFkvY1p3aC9yNTZtS0djN2h5NlRiYWtLQ20z?=
 =?utf-8?B?Z3ZUV3FOL2NUdTNRcCtwZk9xN2Q5R3EwK2VXRW8yOWN1R0tTZEVvVk91b05D?=
 =?utf-8?B?TWc0WWhWNzV1dHA1ZFVlVURpVElFbWRGMkJSN2duV0E0NzYvSzR3UzJza3Zy?=
 =?utf-8?B?ejljS3lOR3kvMDRwMHorTUxBOFpRb3BqSjUrWnB4T3dGNUdaWEhaMUMyVHo5?=
 =?utf-8?B?bjNnOTE3SXlxWklmenJCcTlBSlVJemIvNFJwVTlaY3NsT29VMWQ3UHhRK1Y5?=
 =?utf-8?B?TFFyVnU4U2l4QnZ6Z3VESWxuRU1YOW1rMS9iU2dWTnBTZTdoMlh5ZEx5RHpZ?=
 =?utf-8?B?MUwwMlJLTEtseE0vek1CTnlBY2E0WTFkN1hLNDhuOTI4VThaL0NKMWdzUUFK?=
 =?utf-8?B?M05qRUtpcExYYlFKM2ltby9MdVlDQk50VEtMSmQzSGJETXkrbUNBY2FRamhr?=
 =?utf-8?B?T2hlYVFWQ2ZrTlVROW5PRzAxT1dabjNJRHMwcTBsUWZkNCt2cDJXT2hTMkRn?=
 =?utf-8?B?b3JXQnRtS0NQcmlReHRLY2c4cFdqYlByRkNNVWZuZzlsUXJiM2ZybUVuMEI1?=
 =?utf-8?B?SHZpalpzdGlzMHJWK1F6T29BWWR5aGtLamE4bFhtdm4rRldoNEVEOFo2YmY4?=
 =?utf-8?B?cGhwc1g5RGhNd1NkOHBKTG5LZ3N4SmtGN2JrZFlvSGtNNXBLSENuWnVxc05o?=
 =?utf-8?B?c1ZDQzNKV2Z2Ym5TQndROVNoSTZuUEgrUFJwU2ljeGxJTHZXbVowdVMwZHY5?=
 =?utf-8?B?c0U0WkxxR21BaVh6QXdzRkRzK1hVcHZKU0lCeTZCZm9NdUpXcnBDd2dpVVRj?=
 =?utf-8?B?UU1ScGN0SExCRU0vWWszb3BmT0xnRFI5aWw3Q1BDbGYxOVVSS2MySlpwaTEz?=
 =?utf-8?B?VWVRa2wvWFZuRk5COHpDc2NCaTRNYThPd0d6ZWcyOHU2UmVES1phTHI1RGQw?=
 =?utf-8?B?NmZ2cnNXaklERFZyejVleHR5d056VUxNVXdqR1QrZVdiVkVIcjRWbEdBMG03?=
 =?utf-8?B?OVNwS2twOGVqenVEbThVd2d6QkNCakF2UjVINUdMWlhRb0hUSU9VdmxITzhh?=
 =?utf-8?B?Nnc9PQ==?=
X-MS-Exchange-CrossTenant-Network-Message-Id: 6b9332f7-2b5f-4872-be68-08dbc06ff67c
X-MS-Exchange-CrossTenant-AuthSource: CO1PR11MB4914.namprd11.prod.outlook.com
X-MS-Exchange-CrossTenant-AuthAs: Internal
X-MS-Exchange-CrossTenant-OriginalArrivalTime: 28 Sep 2023 22:12:11.3617
 (UTC)
X-MS-Exchange-CrossTenant-FromEntityHeader: Hosted
X-MS-Exchange-CrossTenant-Id: 46c98d88-e344-4ed4-8496-4ed7712e255d
X-MS-Exchange-CrossTenant-MailboxType: HOSTED
X-MS-Exchange-CrossTenant-UserPrincipalName: /3aVYzVp9Fca+L4vRZXl+VdJnvu4mT2rTmtJYix3NINMiLOP02puUBxdwnOusFUHG+3YzsLfM6nKBibpgfgYpUVxbcZPeI1LV2ic1T1idx8=
X-MS-Exchange-Transport-CrossTenantHeadersStamped: MN6PR11MB8170
X-OriginatorOrg: intel.com
X-Spam-Status: No, score=-3.6 required=5.0 tests=BAYES_00,DKIMWL_WL_HIGH,
        DKIM_SIGNED,DKIM_VALID,DKIM_VALID_AU,DKIM_VALID_EF,NICE_REPLY_A,
        RCVD_IN_DNSWL_BLOCKED,RCVD_IN_MSPIKE_H3,RCVD_IN_MSPIKE_WL,
        SPF_HELO_NONE,SPF_NONE autolearn=ham autolearn_force=no version=3.4.6
X-Spam-Checker-Version: SpamAssassin 3.4.6 (2021-04-09) on
        lindbergh.monkeyblade.net
Precedence: bulk
List-ID: <linux-kernel.vger.kernel.org>
X-Mailing-List: linux-kernel@vger.kernel.org
X-Greylist: Sender passed SPF test, not delayed by milter-greylist-4.6.4 (howler.vger.email [0.0.0.0]); Thu, 28 Sep 2023 15:12:25 -0700 (PDT)

On 9/27/2023 1:31 AM, Ivan Vecera wrote:
> The series makes some house-keeping tasks on i40e driver:
> 
> Patch 1: Removes unnecessary back pointer from i40e_hw
> Patch 2: Moves I40E_MASK macro to i40e_register.h where is used
> Patch 3: Refactors I40E_MDIO_CLAUSE* to use the common macro
> Patch 4: Add header dependencies to <linux/avf/virtchnl.h>
> Patch 5: Simplifies memory alloction functions
> Patch 6: Moves mem alloc structures to i40e_alloc.h
> Patch 7: Splits i40e_osdep.h to i40e_debug.h and i40e_io.h
> Patch 8: Removes circular header deps, fixes and cleans headers
> Patch 9: Moves DDP specific macros and structs to i40e_ddp.c
> 
> Changes:
> v2 - Fixed kdoc comment for i40e_hw_to_pf()
>     - Reordered patches 5 and 7-9 to make them simplier

spelling: simpler

Thanks for this cleanup series, the changes all seem sane.

Reviewed-by: Jesse Brandeburg <jesse.brandeburg@intel.com>