lkml.org 
[lkml]   [2021]   [Apr]   [29]   [last100]   RSS Feed
Views: [wrap][no wrap]   [headers]  [forward] 
 
Messages in this thread
    /
    Date
    From
    SubjectRe: [PATCH v5 5/6] PCI: fu740: Add SiFive FU740 PCIe host controller driver
    On Thu, Apr 29, 2021 at 03:59:54PM +0100, Lorenzo Pieralisi wrote:
    > On Wed, Apr 28, 2021 at 02:47:13PM -0500, Bjorn Helgaas wrote:
    > > On Tue, Apr 06, 2021 at 05:26:33PM +0800, Greentime Hu wrote:
    > > > From: Paul Walmsley <paul.walmsley@sifive.com>
    > > >
    > > > Add driver for the SiFive FU740 PCIe host controller.
    > > > This controller is based on the DesignWare PCIe core.
    > > >
    > > > Signed-off-by: Paul Walmsley <paul.walmsley@sifive.com>
    > > > Co-developed-by: Henry Styles <hes@sifive.com>
    > > > Signed-off-by: Henry Styles <hes@sifive.com>
    > > > Co-developed-by: Erik Danie <erik.danie@sifive.com>
    > > > Signed-off-by: Erik Danie <erik.danie@sifive.com>
    > > > Co-developed-by: Greentime Hu <greentime.hu@sifive.com>
    > > > Signed-off-by: Greentime Hu <greentime.hu@sifive.com>
    > >
    > > Tripped over these errors while build testing with the .config below.
    >
    > Sorry about that - the kbot did not managed to test it. Is it a
    > randconfig ? I think we should ask the kbuild bot guys to add it,
    > I think it can be done on a per-repo basis.

    I don't know enough about the bot. The lkp@intel.com reports I get
    include allyesconfig for x86_64; not sure why that wouldn't catch
    this.

    > Waiting for a fix asap - I can move/rebase some commits if the fix
    > takes time.

    If it's feasible you could just move the FU740 stuff to a different
    branch so we can be sure to include the other dwc stuff. Same for
    brcmstb.

    Oh, and Colin just posted a NULL pointer check that could be squashed
    into the new mediatek-gen3 driver.

    Bjorn

    \
     
     \ /
      Last update: 2021-04-29 17:18    [W:2.375 / U:0.692 seconds]
    ©2003-2020 Jasper Spaans|hosted at Digital Ocean and TransIP|Read the blog|Advertise on this site