# Re: \[PATCH v10 6/6\] drm\_modes: add of\_videomode helpers

- List: linux-fbdev
- 2 messages, 2 authors
- Page: https://ratatoskr.run/linux-fbdev/2012/11/8065492/t

## Re: \[PATCH v10 6/6\] drm\_modes: add of\_videomode helpers

- From: Thierry Reding \<hidden\>
- Date: 2012-11-15 09:58:49 
- Message: https://ratatoskr.run/linux-fbdev/2012/11/8065492.md

```text
On Thu, Nov 15, 2012 at 10:23:57AM +0100, Steffen Trumtrar wrote:
[...]
> +int of_get_drm_display_mode(struct device_node *np,
> +			    struct drm_display_mode *dmode, unsigned int index)
> +{
> +	struct videomode vm;
> +	int ret;
> +
> +	ret = of_get_videomode(np, &vm, index);
> +	if (ret)
> +		return ret;
> +
> +	display_mode_from_videomode(&vm, dmode);

This function is now called drm_display_mode_from_videomode().

Thierry
```

## Re: \[PATCH v10 6/6\] drm\_modes: add of\_videomode helpers

- From: Steffen Trumtrar \<hidden\>
- Date: 2012-11-15 10:01:49 
- Message: https://ratatoskr.run/linux-fbdev/2012/11/8065493.md

```text
On Thu, Nov 15, 2012 at 10:58:49AM +0100, Thierry Reding wrote:
> On Thu, Nov 15, 2012 at 10:23:57AM +0100, Steffen Trumtrar wrote:
> [...]
> > +int of_get_drm_display_mode(struct device_node *np,
> > +			    struct drm_display_mode *dmode, unsigned int index)
> > +{
> > +	struct videomode vm;
> > +	int ret;
> > +
> > +	ret = of_get_videomode(np, &vm, index);
> > +	if (ret)
> > +		return ret;
> > +
> > +	display_mode_from_videomode(&vm, dmode);
> 
> This function is now called drm_display_mode_from_videomode().
> 
Well, can't argue with that. You are right.

Regards,
Steffen

-- 
Pengutronix e.K.                           |                             |
Industrial Linux Solutions                 | http://www.pengutronix.de/  |
Peiner Str. 6-8, 31137 Hildesheim, Germany | Phone: +49-5121-206917-0    |
Amtsgericht Hildesheim, HRA 2686           | Fax:   +49-5121-206917-5555 |
```