From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1753036AbcF2PyD (ORCPT ); Wed, 29 Jun 2016 11:54:03 -0400 Received: from smtp.codeaurora.org ([198.145.29.96]:40090 "EHLO smtp.codeaurora.org" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1752318AbcF2PyA (ORCPT ); Wed, 29 Jun 2016 11:54:00 -0400 Content-Type: text/plain; charset="utf-8" MIME-Version: 1.0 Content-Transfer-Encoding: 7bit Subject: Re: mwifiex: Fixed endianness for event TLV type TLV_BTCOEX_WL_SCANTIME From: Kalle Valo In-Reply-To: <1466050770-21789-1-git-send-email-prasunmaiti87@gmail.com> To: Prasun Maiti Cc: Amitkumar Karwar , Nishant Sarmukadam , Linux Wireless , Linux Next , Linux Kernel Message-Id: <20160629155259.08DB1613A4@smtp.codeaurora.org> Date: Wed, 29 Jun 2016 15:52:59 +0000 (UTC) Sender: linux-kernel-owner@vger.kernel.org List-ID: X-Mailing-List: linux-kernel@vger.kernel.org Prasun Maiti wrote: > The two members min_scan_time and max_scan_time of structure > "mwifiex_ie_types_btcoex_scan_time" are of two bytes each. The values > are assigned directtly from firmware without endian conversion handling. > So, wrong datas will get saved in big-endian systems. > > This patch converts the values into cpu's byte order before assigning them > into the local members. > > Signed-off-by: Prasun Maiti I would like to see Reviewed-by from someone else before I apply this. -- Sent by pwcli https://patchwork.kernel.org/patch/9179811/