On Tue, 06 Jun 2023 11:33:03 +0300, Dan Carpenter wrote: > The "vsync_hz" variable is unsigned int so it can't be less > than zero. The dpu_kms_get_clk_rate() function used to return a u64 > but I previously changed it to return an unsigned long and zero on > error so it matches clk_get_rate(). > > Change the "vsync_hz" type to unsigned long as well and change the > error checking to check for zero instead of negatives. This change > does not affect runtime at all. It's just a clean up. > > [...] Applied, thanks! [1/1] drm/msm/dpu: tidy up some error checking https://gitlab.freedesktop.org/lumag/msm/-/commit/e7a2cf8e058e Best regards, -- Dmitry Baryshkov <dmitry.baryshkov@xxxxxxxxxx>