Hi Doug, > Seems reasonable to me. I would probably put an error message in this > case, though? I don't think regmap_read() necessarily prints an error > so it would just be a mysterious failure for why things didn't probe, > right? OK, can do that. > This also only solves the problems for one of the 4 devices in this > file. I think the GPIO device, PWM device, and DP-AUX device will > still confusingly stick around. It's probably better to add a bogus > regmap read in ti_sn65dsi86_probe(). Could be argued. I just wonder about all the other error cases in ti_sn_attach_host() plus ti_sn_bridge_probe(). They have the same problem with the other devices dangling? Happy hacking, Wolfram
Attachment:
signature.asc
Description: PGP signature