Re: [PATCH] drm/msm/dpu: Fix usage of ERR_PTR()

2020-06-22 Thread Zenghui Yu
ping for this obvious fix... On 2020/5/28 21:08, Zenghui Yu wrote: ERR_PTR() is used in the kernel to encode an usual *negative* errno code into a pointer. Passing a positive value (ENOMEM) to it will break the following IS_ERR() check. Though memory allocation is unlikely to fail, it's still

Re: [PATCH] drm/msm/dpu: Fix usage of ERR_PTR()

2020-06-20 Thread Rob Clark
On Sat, Jun 20, 2020 at 3:26 AM Zenghui Yu wrote: > > ping for this obvious fix... Hi, thanks, but there is already a similar fix in msm-next: commit aa472721c8dbe1713cf510f56ffbc56ae9e14247 Refs: v5.7-rc5-33-gaa472721c8db Author: Chen Tao AuthorDate: Mon Jun 8 09:48:59 2020 +0800 Commit:

[PATCH] drm/msm/dpu: Fix usage of ERR_PTR()

2020-05-28 Thread Zenghui Yu
ERR_PTR() is used in the kernel to encode an usual *negative* errno code into a pointer. Passing a positive value (ENOMEM) to it will break the following IS_ERR() check. Though memory allocation is unlikely to fail, it's still worth fixing. And grepping shows that this is the only misuse of ERR_P