diff options
| author | Felix Held <felix-coreboot@felixheld.de> | 2018-07-06 19:38:24 +0200 |
|---|---|---|
| committer | Martin Roth <martinroth@google.com> | 2018-07-17 18:06:19 +0000 |
| commit | 745e58a5ee50373276924349524a2594599e8bb5 (patch) | |
| tree | d94136e670d162e703c8ee720924ef1462ad3e20 /src/device/pnp_device.c | |
| parent | f17a0df112ec3351a9ea344408a39dc895bf1788 (diff) | |
pnp_device: improve readability
Add comments on the ops handling in pnp_enable_devices function and the
pnp_info struct.
Also remove the negation in the check if an LDN-specific override is used.
This patch doesn't change the logic though.
Change-Id: I3e80dbce1f29ee3e95e3b1d71c9b8479561d5c1a
Signed-off-by: Felix Held <felix-coreboot@felixheld.de>
Reviewed-on: https://review.coreboot.org/27384
Tested-by: build bot (Jenkins) <no-reply@coreboot.org>
Reviewed-by: Paul Menzel <paulepanter@users.sourceforge.net>
Reviewed-by: Patrick Rudolph <siro@das-labor.org>
Diffstat (limited to 'src/device/pnp_device.c')
| -rw-r--r-- | src/device/pnp_device.c | 9 |
1 files changed, 6 insertions, 3 deletions
diff --git a/src/device/pnp_device.c b/src/device/pnp_device.c index ba05c82d5e..5600231207 100644 --- a/src/device/pnp_device.c +++ b/src/device/pnp_device.c @@ -386,10 +386,13 @@ void pnp_enable_devices(struct device *base_dev, struct device_operations *ops, if (dev->ops) continue; - if (info[i].ops == 0) - dev->ops = ops; - else + /* use LDN-specific ops override from corresponding pnp_info + entry if not NULL */ + if (info[i].ops) dev->ops = info[i].ops; + /* else use device ops */ + else + dev->ops = ops; get_resources(dev, &info[i]); } |