From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Date: Mon, 17 Jul 2023 12:33:47 -0700 Message-ID: <87sf9mib8k.wl-ashutosh.dixit@intel.com> From: "Dixit, Ashutosh" To: Rodrigo Vivi In-Reply-To: References: <20230717165318.831766-1-tvrtko.ursulin@linux.intel.com> <20230717171219.832728-1-tvrtko.ursulin@linux.intel.com> MIME-Version: 1.0 (generated by SEMI-EPG 1.14.7 - "Harue") Content-Type: text/plain; charset=US-ASCII Subject: Re: [igt-dev] [PATCH i-g-t v2] tests/i915_pm_rps: Fix test after silent conflict List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , Cc: igt-dev@lists.freedesktop.org, Intel-gfx@lists.freedesktop.org, Tvrtko Ursulin Errors-To: igt-dev-bounces@lists.freedesktop.org Sender: "igt-dev" List-ID: On Mon, 17 Jul 2023 11:53:27 -0700, Rodrigo Vivi wrote: > > On Mon, Jul 17, 2023 at 06:12:19PM +0100, Tvrtko Ursulin wrote: > > From: Tvrtko Ursulin > > > > A silent conflict sneaked in as I was merging > > d86ca7e17b58 ("tests/i915_pm_rps: Exercise sysfs thresholds") in a way > > that igt_sysfs_set_u32 has became a function returning void. > > > > Assert is now built-in so drop it from the test. > > > > v2: > > * Fix invalid value test. > > * Assert new values after write while at it. > > Reviewed-by: Rodrigo Vivi Merged to fix the IGT build.