On Thu, 27 Dec 2012 01:45:59 +0000 "Sun, Yi" <yi.sun at intel.com> wrote: > > > Thanks > --Sun, Yi Ah, misunderstood the commit message. It looks good to me. Do you have push access? > > > -----Original Message----- > > From: Ben Widawsky [mailto:ben at bwidawsk.net] > > Sent: Thursday, December 27, 2012 2:56 AM > > To: Sun, Yi > > Cc: intel-gfx at lists.freedesktop.org > > Subject: Re: [PATCH v2] tests/testdisplay: Specify the > > display. > > > > On Wed, 26 Dec 2012 17:22:57 +0800 > > Yi Sun <yi.sun at intel.com> wrote: > > > > > It can specify the display when use option '-o'. > > > E.g. command "./testdiplay -o 1,4" means setting number 4 mode > > > line on the > > number 1 display. > > > > > > v2. Specify the display by id which can be got from option '-i' > > > > where? > [Sun, Yi] Run ./testdisplay -i , we can got following: > Connectors: > id encoder status type size (mm) modes > 7 0 disconnected VGA 0x0 0 > 10 0 disconnected HDMI-A 0x0 0 > 13 0 disconnected DisplayPort 0x0 0 > 16 0 disconnected HDMI-A 0x0 0 > 18 17 connected HDMI-A 480x270 31 > modes: > name refresh (Hz) hdisp hss hse htot vdisp vss vse vtot flags type > clock ...... > > I meant we can specify a display by the connector id which list as > the first column. > > Thanks > --Sun, Yi > > > > > > > > > Signed-off-by: Yi Sun <yi.sun at intel.com> > > > > > > diff --git a/tests/testdisplay.c b/tests/testdisplay.c index > > > 7287487..7aef940 100644 > > > --- a/tests/testdisplay.c > > > +++ b/tests/testdisplay.c > > > @@ -72,7 +72,7 @@ int dump_info = 0, test_all_modes =0, > > > test_preferred_mode = 0, force_mode = 0, int sleep_between_modes > > > = 5; uint32_t depth = 24, stride, bpp; int qr_code = 0; -int > > > only_one_mode = 0, specified_mode_num = 0; > > > +int only_one_mode = 0, specified_mode_num = 0, specified_disp_id > > > = 0; > > > > > > drmModeModeInfo force_timing; > > > > > > @@ -598,6 +598,9 @@ int update_display(void) > > > /* Find any connected displays */ > > > for (c = 0; c < resources->count_connectors; > > > c++) { connectors[c].id = resources->connectors[c]; > > > + if ( connectors[c].id != > > > specified_disp_id ) > > > + continue; > > > + > > > set_mode(&connectors[c]); > > > } > > > } > > > @@ -618,7 +621,7 @@ static void __attribute__((noreturn)) > > > usage(char > > *name) > > > fprintf(stderr, "\t-m\ttest the preferred mode\n"); > > > fprintf(stderr, "\t-t\tuse a tiled framebuffer\n"); > > > fprintf(stderr, "\t-r\tprint a QR code on the screen > > > whose content is > > \"pass\" for the automatic test\n"); > > > - fprintf(stderr, "\t-o\t<number of the mode>\tonly test > > > specified mode\n"); > > > + fprintf(stderr, "\t-o\t<id of the display>,<number of the > > > +mode>\tonly test specified mode on the specified display\n"); > > > fprintf(stderr, "\t-f\t<clock > > MHz>,<hdisp>,<hsync-start>,<hsync-end>,<htotal>,\n"); > > > fprintf(stderr, > > > "\t\t<vdisp>,<vsync-start>,<vsync-end>,<vtotal>\n"); > > > fprintf(stderr, "\t\ttest force mode\n"); @@ -715,7 +718,7 @@ int > > > main(int argc, char **argv) break; > > > case 'o': > > > only_one_mode = 1; > > > - specified_mode_num = atoi(optarg); > > > + sscanf(optarg, "%d,%d", > > > &specified_disp_id, > > &specified_mode_num); > > > break; > > > default: > > > fprintf(stderr, "unknown option %c\n", > > > c); > > > > > > > > -- > > Ben Widawsky, Intel Open Source Technology Center
