On Wed, Apr 24, 2019 at 10:13 AM Sam Ravnborg <sam@xxxxxxxxxxxx> wrote: > > Hi John. > > On Tue, Apr 23, 2019 at 04:20:31PM -0700, John Stultz wrote: > > This patchset contains one fix (in the front, so its easier to > > eventually backport), and a series of changes from YiPing to > > refactor the kirin drm driver so that it can be used on both > > kirin620 based devices (like the original HiKey board) as well > > as kirin960 based devices (like the HiKey960 board). > > > > The full kirin960 drm support is still being refactored, but as > > this base kirin rework was getting to be substantial, I wanted > > to send out the first chunk for some initial review, so that the > > review burden wasn't overwhelming. > I have been through all 25 patches and it triggered a few minor > comments here and there. > I know nothing about the hisilicon driver but based on the fact > that I know nothing about the driver you can give them all > a (smallish) > Reviewed-by: Sam Ravnborg <sam@xxxxxxxxxxxx> > > I trust your judgement to decide the feedback you want to address and > what to ignore. Thanks again for your time and thoughts here! I really appreciate it! > One request: > Could you please in a follow-up patch kill the use of drmP.h. > We want to get rid of it one day and three fewer users are one > small step towards this goal. I'll take a swing at this. Thanks! -john _______________________________________________ dri-devel mailing list dri-devel@xxxxxxxxxxxxxxxxxxxxx https://lists.freedesktop.org/mailman/listinfo/dri-devel