From mboxrd@z Thu Jan 1 00:00:00 1970 From: Sam Ravnborg Subject: Re: [PATCH v5 2/2] drm/panel: Add Feiyang FY07024DI26A30-D MIPI-DSI LCD panel Date: Tue, 29 Jan 2019 16:19:16 +0100 Message-ID: <20190129151916.GA12635@ravnborg.org> References: <20190125192233.10019-1-jagan@amarulasolutions.com> <20190125192233.10019-2-jagan@amarulasolutions.com> <20190125195205.GA31938@ravnborg.org> Mime-Version: 1.0 Content-Type: text/plain; charset=us-ascii Return-path: Content-Disposition: inline In-Reply-To: Sender: linux-kernel-owner@vger.kernel.org To: Jagan Teki Cc: Thierry Reding , David Airlie , Daniel Vetter , Michael Trimarchi , dri-devel , linux-kernel , linux-amarula@amarulasolutions.com List-Id: dri-devel@lists.freedesktop.org Hi Jagan. > > > > I see DRM_MODE_ARG as mode argument, that print all mode timings but > > here we need only 3 timings out of it. do we really need? if yes > > please suggest an example. > > fyi: sent v6 for this except this change. Let me know if you have any > comments on this. Drivers looks fine, the above was just a quick suggestion to use some exising plumbing. You have done a nice job following up on all the feedback. Sam