[PATCH 1/3] drm/panel: Add display_timing support

[Thierry Reding]

On Tue, Mar 24, 2015 at 12:52:44PM +0100, Philipp Zabel wrote:
> Am Dienstag, den 24.03.2015, 12:34 +0100 schrieb Thierry Reding:
> [...]
> > > > Would you prefer
> > > > 	struct display_timing *(*get_timing)(struct drm_panel *panel);
> > > > ?
> > > 
> > > I'd like to resend this. Please let me know if you want me to change
> > > this function prototype.
> > 
> > I have no objections to keeping the current prototype. It's something we
> > can always fixup if we want to. Also keeping the symmetry with min/max
> > values for hactive and vactive is okay in my opinion.
> > 
> > Were there any other remaining points? If not I'll just apply this as
> > is.
> 
> No, I'm happy if you apply this as is.

Done. I touched up a couple of things in the commit messages and made a
minor change to the loop which copies the timings in the simple-panel
driver (extract the min(num_timings, p->desc->num_timings) out of the
for statement). Nothing major, but it might still be good if you could
test, just to make sure I didn't make a mess.

This should be in tomorrow's linux-next, but if you want to take a peek
before that, you can grab it from here:

	git://anongit.freedesktop.org/tegra/linux.git#drm/panel/for-next

Thierry
-------------- next part --------------
A non-text attachment was scrubbed...
Name: not available
Type: application/pgp-signature
Size: 819 bytes
Desc: not available
URL: <http://lists.freedesktop.org/archives/dri-devel/attachments/20150324/dbe133b7/attachment.sig>