> > > > Fair enough, but at this point, only lib/igt_chamelium.c is using > > igtrc and all > > the parsing is implemented there. > > > > So one course of action here would be to move igtrc parsing outside > > of > > igt_chamelium (maybe to igt_core?) and use some global variable for > > storing this > > value. What do you think? > > > > That seems like a plan, except you can just call a function that will > do > the parsing for you and return a structure with all the parameters. > If > it is called a second time, just return the cached value. > > What do you think, Lyude? Yep, the reason it's called .igtrc is mainly because we were eventually expecting someone else to start using it, and suspend/resume delay is definitely something that will be useful for the rest of the tests (I've got quite a few machines myself that i had to increase the delay on just because the firmware was too slow when suspending/resuming). So I'm all for it. _______________________________________________ Intel-gfx mailing list Intel-gfx@xxxxxxxxxxxxxxxxxxxxx https://lists.freedesktop.org/mailman/listinfo/intel-gfx