On Tuesday 26 March 2013 04:11 AM, Laurent Pinchart wrote:
Hi Archit,

On Monday 25 March 2013 11:44:35 Archit Taneja wrote:
Hi Laurent,

On Tuesday 19 March 2013 08:25 PM, Laurent Pinchart wrote:
Extend the -P option to allow specifying the plane x and y offsets. The
position is optional, if not specified the plane will be positioned at
the center of the screen as before.

Thanks for this series. I tested the patches with a Panda ES board.

I was facing issues with the plane position though, when I execute this
on the command line:

./modetest -s 12:1440x900 -P 6:(0,0)/300x200

I get a syntax error by bash saying it doesn't expect "(". I guess there
are ways around to get over this,

I use

./modetest -s 12:1440x900 -P '6:(0,0)/300x200'

but I was wondering if we could get rid of the braces all together to keep
it simple? The "/" character could be used to figure out whether the user
has also mentioned position or not.

It makes parsing the option a bit more complex, but I can do that if you think
it's better.

I think it's fine. After googling a bit on the syntax error issue, I thought that putting the command in a script file was the only option, but your method above is convenient enough.

Thanks,
Archit

_______________________________________________
dri-devel mailing list
dri-devel@lists.freedesktop.org
http://lists.freedesktop.org/mailman/listinfo/dri-devel

Reply via email to