linux-kernel - Re: [PATCH v3 1/1] staging: Driver for Altera PCI Express Chaining DMA reference design"

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 linux-cve-announce PHC
Open Source and information security mailing list archives

Hash Suite: Windows password security audit tool. GUI, reports in PDF.

[<prev] [next>] [<thread-prev] [day] [month] [year] [list]

Message-ID: <20081204195340.GA30442@kroah.com>
Date:	Thu, 4 Dec 2008 11:53:40 -0800
From:	Greg KH <greg@...ah.com>
To:	leonw@...lcan.com
Cc:	Leon Woestenberg <leon.woestenberg@...n.tv>,
	linux-kernel@...r.kernel.org
Subject: Re: [PATCH v3 1/1] staging: Driver for Altera PCI Express Chaining
	DMA reference design"

On Mon, Dec 01, 2008 at 08:45:57AM -0500, leonw@...lcan.com 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@...r.kernel.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html
Please read the FAQ at  http://www.tux.org/lkml/