From mboxrd@z Thu Jan 1 00:00:00 1970 From: Mark Salter Subject: Re: [PATCH v8 4/4] drivers: net: Add APM X-Gene SoC ethernet driver support. Date: Wed, 25 Jun 2014 12:40:45 -0400 Message-ID: <1403714445.755.112.camel@deneb.redhat.com> References: <1403306296-26198-1-git-send-email-isubramanian@apm.com> <1403306296-26198-5-git-send-email-isubramanian@apm.com> Mime-Version: 1.0 Content-Type: text/plain; charset="UTF-8" Content-Transfer-Encoding: 7bit Return-path: In-Reply-To: Sender: netdev-owner@vger.kernel.org To: Dann Frazier Cc: Iyappan Subramanian , davem@davemloft.net, netdev@vger.kernel.org, "devicetree@vger.kernel.org" , Greg Kroah-Hartman , "linux-kernel@vger.kernel.org" , linux-arm-kernel@lists.infradead.org, jcm@redhat.com, patches@apm.com, Ravi Patel , Keyur Chudgar List-Id: devicetree@vger.kernel.org On Tue, 2014-06-24 at 22:29 -0600, Dann Frazier wrote: > On Fri, Jun 20, 2014 at 5:18 PM, Iyappan Subramanian > > + ring->desc_addr = dma_zalloc_coherent(dev, size, &ring->dma, > > + GFP_KERNEL); > > Iyappan, > When testing this driver on a 3.16-rc2 base, I'm finding that > desc_addr gets assigned to NULL here, which results in an oops later > on (see below). > > I wasn't seeing this before (3.15 base), so I'm guessing something > changed upstream, or in my config, to change this behavior. But it > does illuminate a place where we could maybe use some better error > handling (also see below). https://lkml.org/lkml/2014/6/23/10