Re: [PATCH v2] drm/meson: Use 1000ULL when operating with mode->clock

2025-05-10 Thread I Hsin Cheng
On Fri, May 09, 2025 at 08:56:01PM +0200, Neil Armstrong wrote: > Hi, > > On Tue, 06 May 2025 02:43:38 +0800, I Hsin Cheng wrote: > > Coverity scan reported the usage of "mode->clock * 1000" may lead to > > integer overflow. Use "1000ULL" instead of "1000" > > when utilizing it to avoid potential

Re: [PATCH v2] drm/meson: Use 1000ULL when operating with mode->clock

2025-05-09 Thread Neil Armstrong
Hi, On Tue, 06 May 2025 02:43:38 +0800, I Hsin Cheng wrote: > Coverity scan reported the usage of "mode->clock * 1000" may lead to > integer overflow. Use "1000ULL" instead of "1000" > when utilizing it to avoid potential integer overflow issue. > > Thanks, Applied to https://gitlab.freedesktop

Re: [PATCH v2] drm/meson: Use 1000ULL when operating with mode->clock

2025-05-09 Thread Neil Armstrong
On 09/05/2025 20:27, Martin Blumenstingl wrote: On Fri, May 9, 2025 at 5:35 PM wrote: On 05/05/2025 20:43, I Hsin Cheng wrote: Coverity scan reported the usage of "mode->clock * 1000" may lead to integer overflow. Use "1000ULL" instead of "1000" when utilizing it to avoid potential integer ov

Re: [PATCH v2] drm/meson: Use 1000ULL when operating with mode->clock

2025-05-09 Thread Martin Blumenstingl
On Fri, May 9, 2025 at 5:35 PM wrote: > > On 05/05/2025 20:43, I Hsin Cheng wrote: > > Coverity scan reported the usage of "mode->clock * 1000" may lead to > > integer overflow. Use "1000ULL" instead of "1000" > > when utilizing it to avoid potential integer overflow issue. > > > > Link: > > http

Re: [PATCH v2] drm/meson: Use 1000ULL when operating with mode->clock

2025-05-09 Thread neil . armstrong
On 05/05/2025 20:43, I Hsin Cheng wrote: Coverity scan reported the usage of "mode->clock * 1000" may lead to integer overflow. Use "1000ULL" instead of "1000" when utilizing it to avoid potential integer overflow issue. Link: https://scan5.scan.coverity.com/#/project-view/10074/10063?selectedI

Re: [PATCH v2] drm/meson: Use 1000ULL when operating with mode->clock

2025-05-05 Thread Martin Blumenstingl
On Mon, May 5, 2025 at 8:43 PM I Hsin Cheng wrote: > > Coverity scan reported the usage of "mode->clock * 1000" may lead to > integer overflow. Use "1000ULL" instead of "1000" > when utilizing it to avoid potential integer overflow issue. > > Link: > https://scan5.scan.coverity.com/#/project-view

[PATCH v2] drm/meson: Use 1000ULL when operating with mode->clock

2025-05-05 Thread I Hsin Cheng
Coverity scan reported the usage of "mode->clock * 1000" may lead to integer overflow. Use "1000ULL" instead of "1000" when utilizing it to avoid potential integer overflow issue. Link: https://scan5.scan.coverity.com/#/project-view/10074/10063?selectedIssue=1646759 Signed-off-by: I Hsin Cheng -