Re: [Intel-gfx] [PATCH v3 i-g-t] test/kms_plane_lowres: Fix display_commit_mode() so it returns the crc

2017-12-12 Thread Arkadiusz Hiler
On Tue, Dec 12, 2017 at 02:10:56PM +0200, Arkadiusz Hiler wrote: > On Tue, Dec 12, 2017 at 11:59:13AM +0100, Maarten Lankhorst wrote: > > From: Arkadiusz Hiler > > > > Compiler complained on crc_lowres and crc_hires2 being uninitialized > > and indeed,

Re: [Intel-gfx] [PATCH v3 i-g-t] test/kms_plane_lowres: Fix display_commit_mode() so it returns the crc

2017-12-12 Thread Arkadiusz Hiler
On Tue, Dec 12, 2017 at 11:59:13AM +0100, Maarten Lankhorst wrote: > From: Arkadiusz Hiler > > Compiler complained on crc_lowres and crc_hires2 being uninitialized > and indeed, display_commit_mode() seems to have intention of retruning > the value through the

Re: [Intel-gfx] [PATCH v3 i-g-t] test/kms_plane_lowres: Fix display_commit_mode() so it returns the crc

2017-12-12 Thread Petri Latvala
On Tue, Dec 12, 2017 at 11:59:13AM +0100, Maarten Lankhorst wrote: > From: Arkadiusz Hiler > > Compiler complained on crc_lowres and crc_hires2 being uninitialized > and indeed, display_commit_mode() seems to have intention of retruning

[Intel-gfx] [PATCH v3 i-g-t] test/kms_plane_lowres: Fix display_commit_mode() so it returns the crc

2017-12-12 Thread Maarten Lankhorst
From: Arkadiusz Hiler Compiler complained on crc_lowres and crc_hires2 being uninitialized and indeed, display_commit_mode() seems to have intention of retruning the value through the parameter that is only a single pointer. This couses only the local copy of the