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 PHC | |
Open Source and information security mailing list archives
| ||
|
Date: Tue, 17 Sep 2019 12:56:09 +0200 From: Lars Poeschel <poeschel@...onage.de> To: David Miller <davem@...emloft.net> Cc: gregkh@...uxfoundation.org, tglx@...utronix.de, kstewart@...uxfoundation.org, swinslow@...il.com, allison@...utok.net, linux-kernel@...r.kernel.org, netdev@...r.kernel.org, johan@...nel.org, Claudiu.Beznea@...rochip.com Subject: Re: [PATCH v7 5/7] nfc: pn533: add UART phy driver On Wed, Sep 11, 2019 at 10:17:32AM +0200, David Miller wrote: > From: Lars Poeschel <poeschel@...onage.de> > Date: Tue, 10 Sep 2019 11:33:50 +0200 > > > +static int pn532_uart_send_ack(struct pn533 *dev, gfp_t flags) > > +{ > > + struct pn532_uart_phy *pn532 = dev->phy; > > + /* spec 7.1.1.3: Preamble, SoPC (2), ACK Code (2), Postamble */ > > + static const u8 ack[PN533_STD_FRAME_ACK_SIZE] = { > > + 0x00, 0x00, 0xff, 0x00, 0xff, 0x00}; > > + int err; > > Reverse christmas tree ordering for the local variables please. See below. > > +static int pn532_uart_rx_is_frame(struct sk_buff *skb) > > +{ > > + int i; > > + u16 frame_len; > > + struct pn533_std_frame *std; > > + struct pn533_ext_frame *ext; > > Likewise. Ok, I will do a v8 soon with these changes.
Powered by blists - more mailing lists