Date: Tue, 17 Dec 2024 10:47:58 -0700 From: Warner Losh <imp@bsdimp.com> To: Ruslan Bukin <br@freebsd.org> Cc: Jessica Clarke <jrtc27@freebsd.org>, src-committers <src-committers@freebsd.org>, "<dev-commits-src-all@freebsd.org>" <dev-commits-src-all@freebsd.org>, "<dev-commits-src-main@freebsd.org>" <dev-commits-src-main@freebsd.org> Subject: Re: git: 56816e687557 - main - riscv: Eswin hwreset support added. Message-ID: <CANCZdfqky7C4%2BJTXYVdd2jnEL7RuYWOy64yMM52RPGwdLEwTdQ@mail.gmail.com> In-Reply-To: <Z2G2fA1iv3tVDcSE@bsdpad.com> References: <202412171147.4BHBlAs7010565@gitrepo.freebsd.org> <F35C5308-535A-460F-9076-A1357F022C8F@freebsd.org> <Z2G2fA1iv3tVDcSE@bsdpad.com>
next in thread | previous in thread | raw e-mail | index | archive | help
--00000000000089dbb506297ae67b Content-Type: text/plain; charset="UTF-8" Content-Transfer-Encoding: quoted-printable On Tue, Dec 17, 2024, 10:36=E2=80=AFAM Ruslan Bukin <br@freebsd.org> wrote: > On Tue, Dec 17, 2024 at 02:39:00PM +0000, Jessica Clarke wrote: > > On 17 Dec 2024, at 11:47, Ruslan Bukin <br@FreeBSD.org> wrote: > > > > > > The branch main has been updated by br: > > > > > > URL: > https://cgit.FreeBSD.org/src/commit/?id=3D56816e68755725371ac89dc5aa882fd= ce46605a5 > > > > > > commit 56816e68755725371ac89dc5aa882fdce46605a5 > > > Author: Ruslan Bukin <br@FreeBSD.org> > > > AuthorDate: 2024-12-17 11:35:08 +0000 > > > Commit: Ruslan Bukin <br@FreeBSD.org> > > > CommitDate: 2024-12-17 11:46:10 +0000 > > > > > > riscv: Eswin hwreset support added. > > > > > > Add reset controller driver for Eswin EIC7700. > > > > > > This one has two reset cells in FDT, which is unusual, so provide = a > > > custom hwreset_map method. > > > > > > Tested on SiFive Premier P550. > > > > > > Reviewed by: mhorne, jrtc27 > > > > I gave one review comment, I didn=E2=80=99t get to approving it. Are yo= u > > writing these manually rather than using (git-)arc?... > > > > Jess > > > > I apologize. I'm using git arc. The comment you left resulted to a > positive impact on the code so I decided to indicate you as well, > despite you did not press Accept. > Discussed-with: I think is the trailer to use for this situation where input was given without final approval. Warner > --00000000000089dbb506297ae67b Content-Type: text/html; charset="UTF-8" Content-Transfer-Encoding: quoted-printable <div dir=3D"auto"><div><br><br><div class=3D"gmail_quote gmail_quote_contai= ner"><div dir=3D"ltr" class=3D"gmail_attr">On Tue, Dec 17, 2024, 10:36=E2= =80=AFAM Ruslan Bukin <<a href=3D"mailto:br@freebsd.org">br@freebsd.org<= /a>> wrote:<br></div><blockquote class=3D"gmail_quote" style=3D"margin:0= 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">On Tue, Dec 17, 2024= at 02:39:00PM +0000, Jessica Clarke wrote:<br> > On 17 Dec 2024, at 11:47, Ruslan Bukin <br@FreeBSD.org> wrote:<b= r> > > <br> > > The branch main has been updated by br:<br> > > <br> > > URL: <a href=3D"https://cgit.FreeBSD.org/src/commit/?id=3D56816e6= 8755725371ac89dc5aa882fdce46605a5" rel=3D"noreferrer noreferrer" target=3D"= _blank">https://cgit.FreeBSD.org/src/commit/?id=3D56816e68755725371ac89dc5a= a882fdce46605a5</a><br> > > <br> > > commit 56816e68755725371ac89dc5aa882fdce46605a5<br> > > Author:=C2=A0 =C2=A0 =C2=A0Ruslan Bukin <br@FreeBSD.org><br= > > > AuthorDate: 2024-12-17 11:35:08 +0000<br> > > Commit:=C2=A0 =C2=A0 =C2=A0Ruslan Bukin <br@FreeBSD.org><br= > > > CommitDate: 2024-12-17 11:46:10 +0000<br> > > <br> > >=C2=A0 =C2=A0 riscv: Eswin hwreset support added.<br> > > <br> > >=C2=A0 =C2=A0 Add reset controller driver for Eswin EIC7700.<br> > > <br> > >=C2=A0 =C2=A0 This one has two reset cells in FDT, which is unusua= l, so provide a<br> > >=C2=A0 =C2=A0 custom hwreset_map method.<br> > > <br> > >=C2=A0 =C2=A0 Tested on SiFive Premier P550.<br> > > <br> > >=C2=A0 =C2=A0 Reviewed by: mhorne, jrtc27<br> > <br> > I gave one review comment, I didn=E2=80=99t get to approving it. Are y= ou<br> > writing these manually rather than using (git-)arc?...<br> > <br> > Jess<br> > <br> <br> I apologize. I'm using git arc. The comment you left resulted to a<br> positive impact on the code so I decided to indicate you as well,<br> despite you did not press Accept.<br></blockquote></div></div><div dir=3D"a= uto"><br></div><div dir=3D"auto">Discussed-with: I think is the trailer to = use for this situation where input was given without final approval.=C2=A0<= /div><div dir=3D"auto"><br></div><div dir=3D"auto">Warner=C2=A0</div><div d= ir=3D"auto"><div class=3D"gmail_quote gmail_quote_container"><blockquote cl= ass=3D"gmail_quote" style=3D"margin:0 0 0 .8ex;border-left:1px #ccc solid;p= adding-left:1ex"> </blockquote></div></div></div> --00000000000089dbb506297ae67b--
Want to link to this message? Use this URL: <https://mail-archive.FreeBSD.org/cgi/mid.cgi?CANCZdfqky7C4%2BJTXYVdd2jnEL7RuYWOy64yMM52RPGwdLEwTdQ>