[[PATCH v2]] drm/ast: fixed reading monitor EDID not stable issue

Dave Airlie airlied at gmail.com
Thu Nov 22 02:48:14 UTC 2018


On Thu, 22 Nov 2018 at 11:42, 陳雅正 <yacheng600221 at gmail.com> wrote:
>
> Hi Dave,
> Thanks for your feedback. No issue found actually if I remove "volatile" on my platform. In my experience, if the value is volatile, adding "volatile" will be safer and no harm, that is why I add it by default. If you think it is not necessary, it's ok for me to remove it.
>

Don't add it unless you've got proof you need it, we don't want it
confusing people later.

Dave.


More information about the dri-devel mailing list