<meta http-equiv="Content-Type" content="text/html; charset=utf-8"><div dir="ltr"><div class="gmail_quote"><div dir="ltr" class="gmail_attr">On Sat, Mar 4, 2023 at 3:21 AM Karel Gardas <karel@functional.vision> 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"><br>
One remark as an random observer here.<br>
"branch 6-freebsd-12" has caught my eyes. Let me ask shouldn't <br>
development patches go into the master branch from which they may be <br>
later cherry-picked if needed and pushed into 6-freebsd-12 branch?<br>
<br>
Just few weeks ago guys were having a discussion how to sync branches <br>
since they diverged due to patches pushed into both directions without a <br>
proper syncing...<br></blockquote><div><br></div><div>Mea culpa,<br></div><div>In general, you're absolutely correct.</div><div><br></div><div>The CGEM driver in particular is one of the places where the current state between 6-freebsd-12 does not match. When I got the CGEM driver working initially for ZynqMP, it was on 6-freebsd-12 and at the time there was a lot of complicated work in progress (some of the other things that still do not match) targeted at master from others in the community that I didn't want to disturb by causing more churn. I was waiting for those to resolve before I moved forward on master, but that never actually happened. Looking back, any patches I had to master would have been unlikely to cause them any real headache, but that's hindsight for you.</div><div><br></div><div>Another complication is that the FreeBSD 13 CGEM driver has a lot of the functionality that has been backported to 6-freebsd-12 or changed from it, so some patches become near-irrelevant, some need a complete rework, and some apply cleanly. This may be one of the cases where it applies relatively cleanly, but there would be no way to test it without bringing up the driver to full functionality and I don't have a good idea of the level of effort that would take.</div><div><br></div><div>All that said, getting CGEM on master up to a functioning level is on my todo list so that patches can be applied to both and tested properly.</div><div><br></div><div>Kinsey<br></div></div></div>