Re: [PATCH v3 1/1] staging: Driver for Altera PCI Express ChainingDMA reference design"

[Greg KH]

On Mon, Dec 01, 2008 at 08:45:57AM -0500, leonw@xxxxxxxxxxx wrote:
> Altera PCI Express Chaining DMA driver
> 
> A reference driver that exercises the Chaining DMA logic reference
> design generated along the Altera FPGA PCI Express soft or hard core,
> only if instantiated using the MegaWizard, not the SOPC builder, of
> Quartus 8.1.
> 
> This driver can be used to test the logic instantiation and PCI
> Express layers and acts as a starting point for driving custom logic
> connected to the PCI Express End Point Chaining DMA engines.
> 
> PATCH v3 has some checkpatch.sh cleanups and adds the TODO file.

I got a build error with this version, and had to remove the #include
<byteorder.h> line.

After that, I get the following build warnings:
  CC [M]  drivers/staging/altpciechdma/altpciechdma.o
drivers/staging/altpciechdma/altpciechdma.c: In function âdma_testâ:
drivers/staging/altpciechdma/altpciechdma.c:541: warning: cast from pointer to integer of different size
drivers/staging/altpciechdma/altpciechdma.c:548: warning: assignment makes integer from pointer without a cast
drivers/staging/altpciechdma/altpciechdma.c: In function âprobeâ:
drivers/staging/altpciechdma/altpciechdma.c:803: warning: cast from pointer to integer of different size
drivers/staging/altpciechdma/altpciechdma.c:931: warning: label âerr_revâ defined but not used
drivers/staging/altpciechdma/altpciechdma.c:914: warning: label âerr_cdevâ defined but not used
drivers/staging/altpciechdma/altpciechdma.c: At top level:
drivers/staging/altpciechdma/altpciechdma.c:416: warning: âape_sg_to_chdma_tableâ defined but not used

Also, sparse spits out a lot of warnings as well.

Care to send me a follow-on patch to fix these up?

thanks,

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