RE: [PATCH libdrm v4] modetest: Add support for setting mode having floating vertical refresh rate

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 




> -----Original Message-----
> From: Ville Syrjälä <ville.syrjala@xxxxxxxxxxxxxxx>
> Sent: 25 November 2019 06:30
> To: Devarsh Thakkar <DEVARSHT@xxxxxxxxxx>
> Cc: dri-devel@xxxxxxxxxxxxxxxxxxxxx; Hyun Kwon <hyunk@xxxxxxxxxx>; vcu-
> team <vcu-team@xxxxxxxxxx>; Ranganathan Sk <rsk@xxxxxxxxxx>; Dhaval
> Rajeshbhai Shah <dshah@xxxxxxxxxx>; Satish Kumar Nagireddy
> <SATISHNA@xxxxxxxxxx>; Varunkumar Allagadapa <VARUNKUM@xxxxxxxxxx>
> Subject: Re: [PATCH libdrm v4] modetest: Add support for setting mode
> having floating vertical refresh rate
> 
> EXTERNAL EMAIL
> 
> On Mon, Nov 25, 2019 at 07:34:37AM +0000, Devarsh Thakkar wrote:
> > Ping. Just wanted to confirm if the patch is applied or anything still pending
> from my side ?
> 
> Seems to work -> pushed.
> 
> One thing I noticed is that we still print the modes with an integer vrefresh so
> it's a bit hard to know what value to pass in. Might be nice to make it dumo the
> modes with %.2f vrefresh as well.
>

Agreed, thanks for pointing out,  I am updating it and sending separate patch shortly.
 
> >
> > Also I tried to subscribe as per instructions at
> > https://lists.freedesktop.org/mailman/listinfo/dri-devel
> > Sometime back, but still not able to receive messages from mailing list. Is
> there any other way to subscribe ?
> >
> > Regards,
> > Devarsh
> > > -----Original Message-----
> > > From: Devarsh Thakkar <devarsh.thakkar@xxxxxxxxxx>
> > > Sent: 15 November 2019 06:31
> > > To: dri-devel@xxxxxxxxxxxxxxxxxxxxx; ville.syrjala@xxxxxxxxxxxxxxx
> > > Cc: Hyun Kwon <hyunk@xxxxxxxxxx>; vcu-team <vcu-team@xxxxxxxxxx>;
> > > Ranganathan Sk <rsk@xxxxxxxxxx>; Dhaval Rajeshbhai Shah
> > > <dshah@xxxxxxxxxx>; Satish Kumar Nagireddy <SATISHNA@xxxxxxxxxx>;
> > > Varunkumar Allagadapa <VARUNKUM@xxxxxxxxxx>; Devarsh Thakkar
> > > <DEVARSHT@xxxxxxxxxx>
> > > Subject: [PATCH libdrm v4] modetest: Add support for setting mode
> > > having floating vertical refresh rate
> > >
> > > For the scenario where user may require to modeset with a mode
> > > supporting a fractional value for vertical refresh-rate, appropriate
> > > mode can be selected by searching for mode having matching
> > > fractional vertical refresh rate using below equation.
> > >
> > > vrefresh = (1000 * pixel clock) / (htotal * vtotal) Hz.
> > >
> > > We do this way since driver doesn't return float value of vrefresh
> > > as it use int for vrefresh in struct drm_mode_info, but we can
> > > derive the actual value using pixel clock, horizontal total size and vertical
> total size values.
> > >
> > > So for e.g. if user want to select mode having 59.94 Hz as refresh
> > > rate then with this patch it be can done as shown in below command,
> > > given there is an appropriate mode is available :
> > >
> > > modetest -M xlnx -s 39:1920x1080-59.94@BG24 -v
> > >
> > > NOTE: Above command was tested on xilinx DRM driver with DP monitor
> > > which was supporting mode having 59.94 Hz refresh rate.
> > >
> > > V2: Update commit message
> > > V3: Update with below changes as per review comments :
> > >   1) Use epsilon for vrefresh comparison
> > >   2) Use implicit type-casting wherever possible
> > > V4: Keep patch version history on main commit message
> > >
> > > Signed-off-by: Devarsh Thakkar <devarsh.thakkar@xxxxxxxxxx>
> > > Reviewed-by: Ville Syrjälä <ville.syrjala@xxxxxxxxxxxxxxx>
> > > ---
> > >  tests/modetest/modetest.c | 20 ++++++++++++--------
> > >  1 file changed, 12 insertions(+), 8 deletions(-)
> > >
> > > diff --git a/tests/modetest/modetest.c b/tests/modetest/modetest.c
> > > index e66be66..b4edfcb 100644
> > > --- a/tests/modetest/modetest.c
> > > +++ b/tests/modetest/modetest.c
> > > @@ -54,6 +54,7 @@
> > >  #ifdef HAVE_SYS_SELECT_H
> > >  #include <sys/select.h>
> > >  #endif
> > > +#include <math.h>
> > >
> > >  #include "xf86drm.h"
> > >  #include "xf86drmMode.h"
> > > @@ -795,7 +796,7 @@ struct pipe_arg {
> > >     uint32_t crtc_id;
> > >     char mode_str[64];
> > >     char format_str[5];
> > > -   unsigned int vrefresh;
> > > +   float vrefresh;
> > >     unsigned int fourcc;
> > >     drmModeModeInfo *mode;
> > >     struct crtc *crtc;
> > > @@ -822,11 +823,12 @@ struct plane_arg {
> > >
> > >  static drmModeModeInfo *
> > >  connector_find_mode(struct device *dev, uint32_t con_id, const char
> > > *mode_str,
> > > -        const unsigned int vrefresh)
> > > +   const float vrefresh)
> > >  {
> > >     drmModeConnector *connector;
> > >     drmModeModeInfo *mode;
> > >     int i;
> > > +   float mode_vrefresh;
> > >
> > >     connector = get_connector_by_id(dev, con_id);
> > >     if (!connector || !connector->count_modes) @@ -839,9 +841,11 @@
> > > connector_find_mode(struct device *dev, uint32_t con_id, const char
> > > *mode_str,
> > >                      * first mode that match with the name. Else,
> > > return the mode that match
> > >                      * the name and the specified vertical refresh
> > > frequency.
> > >                      */
> > > +                   mode_vrefresh = mode->clock * 1000.00
> > > +                                   / (mode->htotal * mode->vtotal);
> > >                     if (vrefresh == 0)
> > >                             return mode;
> > > -                   else if (mode->vrefresh == vrefresh)
> > > +                   else if (fabs(mode_vrefresh - vrefresh) < 0.005)
> > >                             return mode;
> > >             }
> > >     }
> > > @@ -1393,8 +1397,8 @@ static void atomic_set_mode(struct device
> > > *dev, struct pipe_arg *pipes, unsigned
> > >             if (pipe->mode == NULL)
> > >                     continue;
> > >
> > > -           printf("setting mode %s-%dHz on connectors ",
> > > -                  pipe->mode_str, pipe->mode->vrefresh);
> > > +           printf("setting mode %s-%.2fHz on connectors ",
> > > +                  pipe->mode_str, pipe->vrefresh);
> > >             for (j = 0; j < pipe->num_cons; ++j) {
> > >                     printf("%s, ", pipe->cons[j]);
> > >                     add_property(dev, pipe->con_ids[j], "CRTC_ID",
> > > pipe-
> > > >crtc->crtc->crtc_id); @@ -1476,8 +1480,8 @@ static void
> > > >crtc->crtc->set_mode(struct
> > > device *dev, struct pipe_arg *pipes, unsigned int co
> > >             if (pipe->mode == NULL)
> > >                     continue;
> > >
> > > -           printf("setting mode %s-%dHz@%s on connectors ",
> > > -                  pipe->mode_str, pipe->mode->vrefresh, pipe-
> > > >format_str);
> > > +           printf("setting mode %s-%.2fHz@%s on connectors ",
> > > +                  pipe->mode_str, pipe->vrefresh,
> > > + pipe->format_str);
> > >             for (j = 0; j < pipe->num_cons; ++j)
> > >                     printf("%s, ", pipe->cons[j]);
> > >             printf("crtc %d\n", pipe->crtc->crtc->crtc_id); @@
> > > -1713,7
> > > +1717,7 @@ static int parse_connector(struct pipe_arg *pipe, const
> > > +char
> > > *arg)
> > >     pipe->mode_str[len] = '\0';
> > >
> > >     if (*p == '-') {
> > > -           pipe->vrefresh = strtoul(p + 1, &endp, 10);
> > > +           pipe->vrefresh = strtof(p + 1, &endp);
> > >             p = endp;
> > >     }
> > >
> > > --
> > > 2.7.4
> >
> 
> --
> Ville Syrjälä
> Intel
_______________________________________________
dri-devel mailing list
dri-devel@xxxxxxxxxxxxxxxxxxxxx
https://lists.freedesktop.org/mailman/listinfo/dri-devel




[Index of Archives]     [Linux DRI Users]     [Linux Intel Graphics]     [Linux USB Devel]     [Video for Linux]     [Linux Audio Users]     [Yosemite News]     [Linux Kernel]     [Linux SCSI]     [XFree86]     [Linux USB Devel]     [Video for Linux]     [Linux Audio Users]     [Linux Kernel]     [Linux SCSI]     [XFree86]
  Powered by Linux