From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1751303AbbEIQz0 (ORCPT ); Sat, 9 May 2015 12:55:26 -0400 Received: from sender1.zohomail.com ([74.201.84.162]:29254 "EHLO sender1.zohomail.com" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1750886AbbEIQzY (ORCPT ); Sat, 9 May 2015 12:55:24 -0400 Date: Sat, 09 May 2015 11:55:17 -0500 From: Jaime Arrocha To: Dan Carpenter Cc: , , , , , Message-ID: <14d39953976.ac8f06c0202743.2448506682461818605@kerneldev.net> In-Reply-To: <20150509105940.GE14154@mwanda> References: <20150508232924.GA30574@localhost.localdomain> <20150509105940.GE14154@mwanda> Subject: Re: Re: [PATCH] staging: gdm724: adding kernel endianness header MIME-Version: 1.0 Content-Type: text/plain; charset="UTF-8" Content-Transfer-Encoding: 7bit X-Priority: Medium User-Agent: Zoho Mail X-Mailer: Zoho Mail Sender: linux-kernel-owner@vger.kernel.org List-ID: X-Mailing-List: linux-kernel@vger.kernel.org ---- On Sat, 09 May 2015 05:59:40 -0500 Dan Carpenter wrote ---- > On Fri, May 08, 2015 at 06:29:24PM -0500, Jaime Arrocha wrote: > > >From TODO list: remove test for host endian > > Included header to gather information about host endianness. > > Please let me know if the code addition requires corrections > > to meet standards. > > > > No. This isn't really an improvement. We need to get rid of ->host_ed > entirely. Which is actually easy. Basically change gdm_cpu_to_dev16() > and friends to do so something: > > u16 gdm_cpu_to_dev16(struct gdm_endian *ed, u16 x) > { > if (ed->dev_ed == ENDIANNESS_LITTLE) > return cpu_to_le16(x); > else > return cpu_to_be16(x); > } > > regards, > dan carpenter > > You are right, I will dive a little further to find out the use for host_ed and eliminate it. I did read an email from a while ago where one author mentioned that he needed to keep this function. Also, the kernel already has functions to swap numbers for endianness. Will you rather use those? Thanks for your feedback, Jaime