<div dir="ltr"><div class="gmail_quote"><div dir="ltr" class="gmail_attr">On Thu, Jan 31, 2019 at 10:36 AM Eric Anholt <<a href="mailto:eric@anholt.net">eric@anholt.net</a>> wrote:<br></div><blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left:1px solid rgb(204,204,204);padding-left:1ex">Alyssa Rosenzweig <<a href="mailto:alyssa@rosenzweig.io" target="_blank">alyssa@rosenzweig.io</a>> writes:<br>
<br>
>> There's u_pipe_screen_get_param_defaults() that you really want to be<br>
>> using to avoid regressions when people add new pipe caps.  You can get<br>
>> rid of a lot of your switch statement that way, too.<br>
><br>
> Oh, good to know. Is that newer? I'm pretty sure that whole block was<br>
> copied from vc4 at some point ;)<br>
><br>
>> We should probably be sticking the kmsro entrypoints in a shared group,<br>
>> since there's nothing specific for the KMS .  Looks like we haven't been<br>
>> doing that, though.<br>
><br>
> Yeah, I had the same thought, but I didn't want to disrupt other drivers<br>
> yet (since that delays merge due to pinging more people).<br>
><br>
>> These are both things we can change later.  I'm mostly excited to see<br>
>> you finally in-tree!<br></blockquote><div><br></div><div>Hear, Hear!  Whatever merge/patch you do is</div><div><br></div><div>Acked-by: Jason Ekstrand <<a href="mailto:jason@jlekstrand.net">jason@jlekstrand.net</a>></div><div><br></div><div>It's about time!<br></div><div> </div><blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left:1px solid rgb(204,204,204);padding-left:1ex">
> ^_^<br>
><br>
> Should either of the above issues be resolved in a v2 now, or just a<br>
> follow-up patch after merging?<br>
<br>
Follow-up, imo.  New drivers have a long awkward period where they<br>
wouldn't pass review the way we would for incremental development, but<br>
continuing to work out-of-tree means rebase pain and losing git history.<br>
<br>
(If you have git history out of tree that you'd like to retain when<br>
merging in, I'm happy to try helping with that.  I've done fun git<br>
surgery before)<br></blockquote><div><br></div><div>Doing merge commits for new drivers is not unheard of and preserves history (if the history is something you want to preserve).<br></div><div><br></div><div>--Jason<br></div></div></div>