From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1752289AbcBJIdl (ORCPT ); Wed, 10 Feb 2016 03:33:41 -0500 Received: from comal.ext.ti.com ([198.47.26.152]:42029 "EHLO comal.ext.ti.com" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1751137AbcBJIdk (ORCPT ); Wed, 10 Feb 2016 03:33:40 -0500 Subject: Re: [PATCH] net: ti: netcp: restore get/set_pad_info() functionality To: Arnd Bergmann , "Karicheri, Muralidharan" , David Laight , "David S . Miller" , Greg Kroah-Hartman References: <1455026303-17014-1-git-send-email-grygorii.strashko@ti.com> <063D6719AE5E284EB5DD2968C1650D6D1CCDC216@AcuExch.aculab.com> <3E54258959B69E4282D79E01AB1F32B704D12E49@DFLE11.ent.ti.com> <4072312.pCFuaqSbhC@wuerfel> CC: "netdev@vger.kernel.org" , "Cooper Jr., Franklin" , "Nori, Sekhar" , "linux-kernel@vger.kernel.org" , "Kwok, WingMan" , "N, Mugunthan V" From: Grygorii Strashko Message-ID: <56BAF5D4.6060106@ti.com> Date: Wed, 10 Feb 2016 10:33:24 +0200 User-Agent: Mozilla/5.0 (X11; Linux x86_64; rv:38.0) Gecko/20100101 Thunderbird/38.5.1 MIME-Version: 1.0 In-Reply-To: <4072312.pCFuaqSbhC@wuerfel> Content-Type: text/plain; charset="windows-1252" Content-Transfer-Encoding: 7bit Sender: linux-kernel-owner@vger.kernel.org List-ID: X-Mailing-List: linux-kernel@vger.kernel.org Hi All, On 02/09/2016 09:38 PM, Arnd Bergmann wrote: > On Tuesday 09 February 2016 16:55:42 Karicheri, Muralidharan wrote: >> >> The descriptors are usable by different drivers, one driver may use it as >> buf ptr/ len, other for something else. So they should remain as generic >> and it is up to individual drivers to use it in whatever way it requires. >> My suggestion is to rename pad field in struct knav_dma_desc to sw_data >> to avoid confusion. i.e >> >> + __le32 pad[4]; >> >> to >> >> + __le32 sw_data[4]; >> > > If the hardware doesn't access them, they can probably just be u32 > and not do any byte swapping. > Most of propositions, mentioned in this thread (and not only), sounds reasonable, and there are definitely a lot of optimization/cleanups and refactorings which could be done for this driver (of course it's up to NETCP maintainers to decide). But this particular patch is intended to fix regression and restore Keystone 2 networking functionality, *including NFS boot*, when vanilla's config files are used (keystone_defconfig, multi_v7_defconfig). And this is "not" an optimization or cleanup - this is partial revert of buggy commit. So, could we fix regression first, pleeeaaase? Then other things can be done if someone have time and would like to take the initiative (and would be ready to get at minimum "Tested-by" tag from netcp maintainers :)). -- regards, -grygorii