diff options
| author | Ben Skeggs <bskeggs@redhat.com> | 2014-08-21 10:39:01 +1000 |
|---|---|---|
| committer | Ben Skeggs <bskeggs@redhat.com> | 2014-12-02 15:37:18 +1000 |
| commit | f50e0a87a443e61da64a80fbc837fb2f24affc63 (patch) | |
| tree | 5b101f221e71143d7bc40e44ac539d808e827278 /nvkm | |
| parent | a619d4ae8f64828250be876a7c555dbf7b3dcfa2 (diff) | |
| download | nouveau-f50e0a87a443e61da64a80fbc837fb2f24affc63.tar.gz | |
bios: log if auxch accesses fail, also return 0x00 from rd when it does
Logging at trace level, rather than as en error, as it seems conceivable
that failure could be normal under certain circumstances (new bios,
older sink that doesn't support a particular DPCD address)
Signed-off-by: Ben Skeggs <bskeggs@redhat.com>
Diffstat (limited to 'nvkm')
| -rw-r--r-- | nvkm/subdev/bios/init.c | 18 |
1 files changed, 11 insertions, 7 deletions
diff --git a/nvkm/subdev/bios/init.c b/nvkm/subdev/bios/init.c index 626380f9e..2c9f2c36f 100644 --- a/nvkm/subdev/bios/init.c +++ b/nvkm/subdev/bios/init.c @@ -278,7 +278,7 @@ init_wri2cr(struct nvbios_init *init, u8 index, u8 addr, u8 reg, u8 val) return -ENODEV; } -static int +static u8 init_rdauxr(struct nvbios_init *init, u32 addr) { struct nouveau_i2c_port *port = init_i2c(init, -2); @@ -286,20 +286,24 @@ init_rdauxr(struct nvbios_init *init, u32 addr) if (port && init_exec(init)) { int ret = nv_rdaux(port, addr, &data, 1); - if (ret) - return ret; - return data; + if (ret == 0) + return data; + trace("auxch read failed with %d\n", ret); } - return -ENODEV; + return 0x00; } static int init_wrauxr(struct nvbios_init *init, u32 addr, u8 data) { struct nouveau_i2c_port *port = init_i2c(init, -2); - if (port && init_exec(init)) - return nv_wraux(port, addr, &data, 1); + if (port && init_exec(init)) { + int ret = nv_wraux(port, addr, &data, 1); + if (ret) + trace("auxch write failed with %d\n", ret); + return ret; + } return -ENODEV; } |