linux-kernel - Re: [patch v2] doc: DMA-mapping.txt has undeclared variables [Bug 10397]

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 for Android: free password hash cracker in your pocket

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

Message-ID: <20080409184532.GG1624@colo.lackof.org>
Date:	Wed, 9 Apr 2008 12:45:32 -0600
From:	Grant Grundler <grundler@...isc-linux.org>
To:	Jesse Barnes <jbarnes@...tuousgeek.org>
Cc:	linux-pci@...ey.karlin.mff.cuni.cz,
	Matti Linnanvuori <mattilinnanvuori@...oo.com>,
	Grant Grundler <grundler@...isc-linux.org>, gregkh@...e.de,
	linux-kernel@...r.kernel.org
Subject: Re: [patch v2] doc: DMA-mapping.txt has undeclared variables [Bug
	10397]

On Mon, Apr 07, 2008 at 09:05:51AM -0700, Jesse Barnes wrote:
> On Monday, April 07, 2008 9:01 am Matti Linnanvuori wrote:
> > From: Matti Linnanvuori <mattilinnanvuori@...oo.com>
> >
> > Fix undeclared variables in code examples.
> > Consistently use pdev as the variable of type struct pci_dev *.
> > Bug 10397.
> 
> I think Grant's comment still stands, "fix undeclared variables" isn't an 
> accurate description of what you're doing here...

Yes, please just drop the first sentence. The second sentence is sufficient.

> That said, the patch itself looks fine to me, if a bit pedantic.  The compiler 
> will catch any mismatched arguments in this case, and the rest of the 
> documentation is pretty clear that these routines take struct pci_dev not 
> struct device, so I'd be inclined to just leave it the way it is to avoid 
> churn.

I think it's a good habit to use "pdev" in device drivers and would
like the documentation to incorporate that.

I'm ok with the churn since it's not conflicting with anything else AFAIK.

thanks,
grant

--
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/