From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1755366AbcGHNkl (ORCPT ); Fri, 8 Jul 2016 09:40:41 -0400 Received: from smtp.codeaurora.org ([198.145.29.96]:56659 "EHLO smtp.codeaurora.org" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1754947AbcGHNkd (ORCPT ); Fri, 8 Jul 2016 09:40:33 -0400 From: Kalle Valo To: Amitkumar Karwar Cc: Prasun Maiti , Nishant Sarmukadam , Linux Wireless , Linux Next , Linux Kernel Subject: Re: [PATCH v4] mwifiex: Reduce endian conversion for REG Host Commands References: <39f33dff10344e53b0fbbc9681f90b33@SC-EXCH04.marvell.com> <1467022402-23990-1-git-send-email-prasunmaiti87@gmail.com> <8e1cc4765b4242a4a48f86abddbe40fc@SC-EXCH04.marvell.com> Date: Fri, 08 Jul 2016 16:40:26 +0300 In-Reply-To: <8e1cc4765b4242a4a48f86abddbe40fc@SC-EXCH04.marvell.com> (Amitkumar Karwar's message of "Fri, 8 Jul 2016 11:02:45 +0000") Message-ID: <877fcw1c51.fsf@kamboji.qca.qualcomm.com> User-Agent: Gnus/5.13 (Gnus v5.13) Emacs/23.3 (gnu/linux) MIME-Version: 1.0 Content-Type: text/plain; charset=us-ascii Sender: linux-kernel-owner@vger.kernel.org List-ID: X-Mailing-List: linux-kernel@vger.kernel.org Amitkumar Karwar writes: >> From: Prasun Maiti [mailto:prasunmaiti87@gmail.com] >> Sent: Monday, June 27, 2016 3:43 PM >> To: Amitkumar Karwar >> Cc: Nishant Sarmukadam; Linux Wireless; Linux Next; Linux Kernel; Kalle >> Valo >> Subject: [PATCH v4] mwifiex: Reduce endian conversion for REG Host >> Commands >> >> For multiple REG Host Commands (e.g HostCmd_CMD_802_11_EEPROM_ACCESS, >> HostCmd_CMD_MAC_REG_ACCESS etc.) "cpu_to_leX"-converted values are saved >> to driver. So, "leX_to_cpu" conversion is required too many times >> afterwards in driver. >> >> This patch reduces the endian: conversion without saving "cpu_to_leX" >> converted values in driver. This will convert endianness in prepare >> command and command response path. >> >> Signed-off-by: Prasun Maiti [deleted almost 300 lines of unnecessary quotes] > Any specific reason for dropping this patch? > I can see the status as "Deferred" on patchwork. It's not dropped, "Deferred" means that the patch was postponed for some reason and I need to look at the patch more carefully. In this case I did it because I was hoping to see an Acked-by from you. -- Kalle Valo