Hi, On Mon, Mar 29, 2021 at 6:34 PM John Stultz <john.stu...@linaro.org> wrote: > > Commit 7bf168c8fe8c ("drm/msm: Fix speed-bin support not to > access outside valid memory"), reworked the nvmem reading of > "speed_bin", but in doing so dropped handling of the -ENOENT > case which was previously documented as "fine". > > That change resulted in the db845c board display to fail to > start, with the following error: > > adreno 5000000.gpu: [drm:a6xx_gpu_init] *ERROR* failed to read speed-bin > (-2). Some OPPs may not be supported by hardware > > Thus, this patch simply re-adds the ENOENT handling so the lack > of the speed_bin entry isn't fatal for display, and gets things > working on db845c. > > Cc: Rob Clark <robdcl...@gmail.com> > Cc: Sean Paul <s...@poorly.run> > Cc: Jordan Crouse <jcro...@codeaurora.org> > Cc: Eric Anholt <e...@anholt.net> > Cc: Douglas Anderson <diand...@chromium.org> > Cc: linux-arm-...@vger.kernel.org > Cc: freedr...@lists.freedesktop.org > Cc: Bjorn Andersson <bjorn.anders...@linaro.org> > Cc: YongQin Liu <yongqin....@linaro.org> > Reported-by: YongQin Liu <yongqin....@linaro.org> > Fixes: 7bf168c8fe8c ("drm/msm: Fix speed-bin support not to access outside > valid memory") > Signed-off-by: John Stultz <john.stu...@linaro.org> > --- > drivers/gpu/drm/msm/adreno/a6xx_gpu.c | 8 +++++++- > 1 file changed, 7 insertions(+), 1 deletion(-)
Reviewed-by: Douglas Anderson <diand...@chromium.org>