On Thu, 22 Nov 2018 at 11:42, 陳雅正 <yacheng600221@xxxxxxxxx> 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. _______________________________________________ dri-devel mailing list dri-devel@xxxxxxxxxxxxxxxxxxxxx https://lists.freedesktop.org/mailman/listinfo/dri-devel
