On 03/14/18 07:21, Ian Lepore wrote:
On Wed, 2018-03-14 at 09:20 +0000, Wojciech Macek wrote:
Author: wma
Date: Wed Mar 14 09:20:03 2018
New Revision: 330925
URL: https://svnweb.freebsd.org/changeset/base/330925
Log:
PowerNV: Fix I2C to compile if FDT is disabled
Submitted by: Wojciech Macek <w...@semihalf.com>
Obtained from: Semihalf
Sponsored by: IBM, QCM Technologies
This seems to be a nonsensical commit. What's the point of compiling
in a driver which is hard-coded to return failure from the probe and
attach routines? The pre-change code seemed more correct: the driver
can only possibly work if FDT/OFW data is available, so only compile it
in when that option is in effect.
-- Ian
It's even worse than that: the files.powerpc uses of fdt are *also*
wrong, because it depends on OF, not FDT per se, which is
unconditionally available on all AIM-type PowerPC kernels, so there is
no need to test for *anything*.
-Nathan
_______________________________________________
svn-src-head@freebsd.org mailing list
https://lists.freebsd.org/mailman/listinfo/svn-src-head
To unsubscribe, send any mail to "svn-src-head-unsubscr...@freebsd.org"