Hi Laurent, > > > > It is news to me that the atomic ops are the way to go - but then I have > > been off-line for a while so no suprise or maybe I just missed it > > before. > > They're not mandatory as such, but they give us access to the atomic > state, which is sometimes required. Overall I think it would be nice to > move to the atomic operations and drop the legacy ones, to avoid > maintaining two sets of operations. It will take time :-) Yeah, but if we can get more people working on the job.. > > > It would be good if the comments in drm_bridge.h could point out what is > > deprecated, so we know what to avoid in new and updated bridge drivers. > > But this is all un-related to this series. > > It's a good point. Would you like to submit a patch, or should I do so ? Please do as I would have to dig around to do it right as I have fogotten most of the drm internals the last couple of months. Just something simple like: "This is deprecated, do not use!" would do the trick for me. Then I would know what to look for if I was reviewing a new bridge driver or patching an existing one or just trying to gentle push someone in the right direction. For drm_drv.h this really helped me to understand what should not be used. Sam
