[U-Boot] [PATCH] net : null check after pointer dereference
Joe Hershberger
joe.hershberger at gmail.com
Wed Jan 27 21:05:28 CET 2016
On Tue, Jan 26, 2016 at 1:26 AM, Vishwas Srivastava
<vishu.kernel at gmail.com> wrote:
> Author: Vishwas Srivastava <vishu.kernel at gmail.com>
> Date: Tue Jan 26 12:46:42 2016 +0530
Please fix the subject of this patch. No space in the "net :".
>
> null check after pointer dereference
All of your commit logs should not have spaces in them (at the start of lines).
>
> NULL check is made after the pointer dereference is davinci-
> emac driver. This patch fixes this issue.
>
> CC: Sergey Kubushyn <ksi at koi8.net>;Joe Hershberger <
> joe.hershberger at ni.com>
Fix this format. One "Cc:" per line.
> Signed-off-by: Vishwas Srivastava <vishu.kernel at gmail.com>
This belongs first... before the Cc:
>
> diff --git a/drivers/net/davinci_emac.c b/drivers/net/davinci_emac.c
> index 3f54a3f..1966eb9 100644
> --- a/drivers/net/davinci_emac.c
> +++ b/drivers/net/davinci_emac.c
> @@ -692,8 +692,10 @@ static int davinci_eth_rcv_packet (struct eth_device
> *dev)
> davinci_invalidate_rx_descs();
>
> rx_curr_desc = emac_rx_active_head;
You need to run your patches through scripts/checkpatch.pl.
> + if (!rx_curr_desc)
> + return 0;
> status = rx_curr_desc->pkt_flag_len;
> - if ((rx_curr_desc) && ((status & EMAC_CPPI_OWNERSHIP_BIT) == 0)) {
> + if ((status & EMAC_CPPI_OWNERSHIP_BIT) == 0) {
> if (status & EMAC_CPPI_RX_ERROR_FRAME) {
> /* Error in packet - discard it and requeue desc */
> printf ("WARN: emac_rcv_pkt: Error in packet\n");
Consider using tools/patman.
Thanks,
-Joe
More information about the U-Boot
mailing list