linux-kernel - Re: [PATCH] GPIO/TCA6424: merge I2C transactions, remove cast

lists.openwall.net		lists / announce owl-users owl-dev john-users john-dev passwdqc-users yescrypt popa3d-users / oss-security kernel-hardening musl sabotage tlsify passwords / crypt-dev xvendor / Bugtraq Full-Disclosure linux-kernel linux-netdev linux-ext4 linux-hardening linux-cve-announce PHC
Open Source and information security mailing list archives

Hash Suite: Windows password security audit tool. GUI, reports in PDF.

[<prev] [next>] [<thread-prev] [day] [month] [year] [list]

Message-Id: <20120518234754.1DD013E07C8@localhost>
Date:	Fri, 18 May 2012 17:47:54 -0600
From:	Grant Likely <grant.likely@...retlab.ca>
To:	Andreas Schallenberg <Andreas.Schallenberg@...itytechnica.com>,
	linus.walleij@...ricsson.com, linux-kernel@...r.kernel.org
Subject: Re: [PATCH] GPIO/TCA6424: merge I2C transactions, remove cast

On Fri, 18 May 2012 17:44:10 -0600, Grant Likely <grant.likely@...retlab.ca> wrote:
> On Fri, 18 May 2012 17:01:36 +0200, Andreas Schallenberg <Andreas.Schallenberg@...itytechnica.com> wrote:
> > From 1a80280e20dfef57c6bc6429e5bdf7e10d10fda9 Mon Sep 17 00:00:00 2001
> > From: Andreas Schallenberg <Andreas.Schallenberg@...itytechnica.com>
> > Date: Fri, 18 May 2012 16:20:31 +0200
> > Subject: [PATCH] GPIO/TCA6424: merge I2C transactions, remove cast
> > 
> > This is a follow-up to "[PATCH v2] Add support for TCA6424":
> > - Use 3 byte transfers instead of two separate transfers (2+1 byte)
> > - An unnecessary cast removed
> > 
> > Please check especially if endianness conversion is done the way it should be.
> 
> It works I suppose.  Personally, I'd explicitly use a byte array, but
> I'll apply this patch as-is anyway.

Patch is damaged and cannot be applied (line wraps and tabs converted
to spaces).  Please check your mailer (please use git send-email) and
repost.

g.

--
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majordomo@...r.kernel.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html
Please read the FAQ at  http://www.tux.org/lkml/