search for: nvkm_i2c_aux_release

...ags & I2C_M_RD) cmd = 1; @@ -51,10 +50,19 @@ nvkm_i2c_aux_i2c_xfer(struct i2c_adapter *adap, struct i2c_msg *msgs, int num) if (mcnt || remaining > 16) cmd |= 4; /* MOT */ - ret = aux->func->xfer(aux, true, cmd, msg->addr, ptr, &cnt); - if (ret < 0) { - nvkm_i2c_aux_release(aux); - return ret; + for (retries = 0, cnt = 0; + retries < 32 && !cnt; + retries++) { + cnt = min_t(u8, remaining, 16); + ret = aux->func->xfer(aux, true, cmd, + msg->addr, ptr, &cnt); + if (ret < 0) + goto out; + } + if (!...

This is another attempt at fixing an issue with yes | sensors-detect Causing some machines with nouveau loaded to hang if certain kinds of displays are attached. I've also included one minor fix that I found along the way of troubleshooting this issue. Lyude Paul (2): drm/nouveau: Fix missing elses in g94_i2c_aux_xfer drm/nouveau: Don't retry infinitely when receiving no data on i2c

...406f0 [ 28.616438] Call Trace: [ 28.616463] ? nv50_mstm_register_connector+0x2c/0x50 [nouveau] [ 28.616477] ? drm_dp_add_port+0x32d/0x460 [drm_kms_helper] [ 28.616501] ? g94_i2c_aux_fini.isra.0+0x27/0x40 [nouveau] [ 28.616523] ? g94_i2c_aux_xfer+0x6a2/0x7d0 [nouveau] [ 28.616544] ? nvkm_i2c_aux_release+0x42/0x50 [nouveau] [ 28.616569] ? nouveau_connector_aux_xfer+0x7f/0xc0 [nouveau] [ 28.616582] ? drm_dp_dpcd_access+0xee/0x120 [drm_kms_helper] [ 28.616595] ? drm_dp_send_link_address+0x16b/0x1f0 [drm_kms_helper] [ 28.616610] ? drm_dp_check_and_send_link_address+0xab/0xb0 [drm_kms_helpe...

This fixes quite a number of runtime PM bugs I found that have been causing some pretty nasty issues such as: - Deadlocking on boot - Connector probing potentially not working while the GPU is in runtime suspend - i2c char dev not working while the GPU is in runtime suspend - aux char dev not working while the GPU is in runtime suspend There's definitely more parts of nouveau that need