Return-Path: Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S933790Ab2JWWjJ (ORCPT ); Tue, 23 Oct 2012 18:39:09 -0400 Received: from us-mx3.synaptics.com ([12.239.217.85]:22861 "EHLO us-mx3.synaptics.com" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S933621Ab2JWWjE (ORCPT ); Tue, 23 Oct 2012 18:39:04 -0400 Message-ID: <50871C84.9090803@synaptics.com> Date: Tue, 23 Oct 2012 15:39:00 -0700 From: Christopher Heiny Organization: Synaptics, Inc User-Agent: Mozilla/5.0 (X11; Linux x86_64; rv:15.0) Gecko/20120827 Thunderbird/15.0 MIME-Version: 1.0 To: Dmitry Torokhov CC: Jean Delvare , Linux Kernel , Linux Input , Allie Xiong , Vivian Ly , Daniel Rosenberg , Alexandra Chen , Joerie de Gram , Wolfram Sang , Mathieu Poirier , Linus Walleij , Naveen Kumar Gaddipati Subject: Re: [RFC PATCH 01/06] input/rmi4: Public header and documentation References: <1349496603-20775-1-git-send-email-cheiny@synaptics.com> <1349496603-20775-2-git-send-email-cheiny@synaptics.com> <20121011082051.GB32175@core.coreip.homeip.net> In-Reply-To: <20121011082051.GB32175@core.coreip.homeip.net> Content-Type: text/plain; charset="ISO-8859-1"; format=flowed Content-Transfer-Encoding: 7bit X-Brightmail-Tracker: AAAAAA== Sender: linux-kernel-owner@vger.kernel.org List-ID: X-Mailing-List: linux-kernel@vger.kernel.org Content-Length: 1870 Lines: 49 On 10/11/2012 01:20 AM, Dmitry Torokhov wrote: > On Fri, Oct 05, 2012 at 09:09:58PM -0700, Christopher Heiny wrote: >> + >> + int (*write_block)(struct rmi_phys_device *phys, u16 addr, u8 *buf, >> + int len); >> + int (*read_block)(struct rmi_phys_device *phys, u16 addr, u8 *buf, >> + int len); >> + > > If you declare your buffer as [const] void * instead of u8 * I think you > will be able to get rid of most of your unions. Good point. We'll explore that. If you see it in the next patch, you'll know it worked out. >> + * Helper fn to convert a byte array representing a 16 bit value in the RMI >> + * endian-ness to a 16-bit value in the native processor's specific endianness. >> + * We don't use ntohs/htons here because, well, we're not dealing with >> + * a pair of 16 bit values. Casting dest to u16* wouldn't work, because >> + * that would imply knowing the byte order of u16 in the first place. The >> + * same applies for using shifts and masks. >> + */ >> +static inline u16 batohs(u8 *src) >> +{ >> + return src[1] * 0x100 + src[0]; >> +} >> + >> +/** >> + * Helper function to convert a 16 bit value (in host processor endianess) to >> + * a byte array in the RMI endianess for u16s. See above comment for >> + * why we dont us htons or something like that. >> + */ >> +static inline void hstoba(u8 *dest, u16 src) >> +{ >> + dest[0] = src % 0x100; >> + dest[1] = src / 0x100; >> +} > > These are not used anymore, right? There are function drivers that we chose not to include that depend on these. We'll drop these from rmi.h until we're ready to submit those other function drivers. -- To unsubscribe from this list: send the line "unsubscribe linux-kernel" in the body of a message to majordomo@vger.kernel.org More majordomo info at http://vger.kernel.org/majordomo-info.html Please read the FAQ at http://www.tux.org/lkml/