On Wed Aug 10 07:48:35 2022 +0000, Zhiyi Zhang wrote:
This call to find_display_mode() is new. What's the purpose of this call here? I assume you did it to verify the display mode from the registry is valid. In that case, you should call it only for registry modes. The copying of dmDriverExtra also seems redundant now that you always call get_full_mode() in the driver backends.
I think you're trying to find full mode here so that there is less work in the driver, right? If that's the case, you didn't do the copying.