From mboxrd@z Thu Jan 1 00:00:00 1970 From: Jagan Teki Subject: Re: [PATCH v3 00/25] drm/sun4i: Allwinner A64 MIPI-DSI support Date: Mon, 29 Oct 2018 18:04:35 +0530 Message-ID: References: <20181026144344.27778-1-jagan@amarulasolutions.com> <20181029091508.gyhn2m5fnbucvb34@flea> Reply-To: jagan-dyjBcgdgk7Pe9wHmmfpqLFaTQe2KTcn/@public.gmane.org Mime-Version: 1.0 Content-Type: text/plain; charset="UTF-8" Return-path: Sender: linux-sunxi-/JYPxA39Uh5TLH3MbocFFw@public.gmane.org In-Reply-To: <20181029091508.gyhn2m5fnbucvb34@flea> List-Post: , List-Help: , List-Archive: , List-Unsubscribe: , To: Maxime Ripard Cc: Chen-Yu Tsai , Icenowy Zheng , Jernej Skrabec , Vasily Khoruzhick , Rob Herring , Mark Rutland , Catalin Marinas , Will Deacon , David Airlie , dri-devel , Michael Turquette , Stephen Boyd , linux-clk , Michael Trimarchi , linux-arm-kernel , devicetree , linux-kernel , linux-sunxi-/JYPxA39Uh5TLH3MbocFFw@public.gmane.org List-Id: dri-devel@lists.freedesktop.org On Mon, Oct 29, 2018 at 2:45 PM Maxime Ripard wrote: > > On Fri, Oct 26, 2018 at 08:13:19PM +0530, Jagan Teki wrote: > > This series fixed the issues related to work DSI on 2-lane panel > > which is reported on previous version[1]. > > > > Few comments from previous version still in discussion, but I just > > send this version just to group all working changes together. > > anyway I will fix in this in next version if any. > > > > PLL_MIPI min rate is still weird, I tried many possible dclock rate > > from panel driver to satisfy manual suggested min rate 500MHz but > > none working so eventually moved 300MHz. any inputs on this area > > are welcome. > > > > All these changes are tested in 2-lane, 4-lane MIPI-DSI panels. > > > > If anyone wants to test, use this repo [2] with WIP-A64-DSI branch. > > A changelog would be nice here, and usually you shouldn't send a new > version while there is ongoing discussion on the previous version. True, I have few patches that fixes 2-lane device, I just grouped all of these working-set together for further communication. mentioned the same thing above.