Date: Mon, 19 Dec 2022 23:16:28 +0800 From: Ganbold Tsagaankhuu <ganbold@gmail.com> To: Emmanuel Vadot <manu@bidouilliste.com> Cc: Ganbold Tsagaankhuu <ganbold@freebsd.org>, src-committers@freebsd.org, dev-commits-src-all@freebsd.org, dev-commits-src-main@freebsd.org Subject: Re: git: 6692670f58f9 - main - Enable setting the phy id. Message-ID: <CAGtf9xMjzfNf8MHeR7fstpCigSo%2BAHjTYp7aAG%2BNAy3DW=p-yg@mail.gmail.com> In-Reply-To: <20221219151812.0b94c79f6df7f724de5b961b@bidouilliste.com> References: <202212191310.2BJDAErn075666@gitrepo.freebsd.org> <20221219151812.0b94c79f6df7f724de5b961b@bidouilliste.com>
next in thread | previous in thread | raw e-mail | index | archive | help
--000000000000604e3305f02fceff Content-Type: text/plain; charset="UTF-8" Content-Transfer-Encoding: quoted-printable On Mon, Dec 19, 2022 at 10:18 PM Emmanuel Vadot <manu@bidouilliste.com> wrote: > On Mon, 19 Dec 2022 13:10:14 GMT > Ganbold Tsagaankhuu <ganbold@FreeBSD.org> wrote: > > > The branch main has been updated by ganbold: > > > > URL: > https://cgit.FreeBSD.org/src/commit/?id=3D6692670f58f9069e59bc0c958fdaefb= 9bea5f726 > > > > commit 6692670f58f9069e59bc0c958fdaefb9bea5f726 > > Author: S=C3=B8ren Schmidt <sos@FreeBSD.org> > > AuthorDate: 2022-12-19 12:56:51 +0000 > > Commit: Ganbold Tsagaankhuu <ganbold@FreeBSD.org> > > CommitDate: 2022-12-19 13:08:42 +0000 > > > > Enable setting the phy id. > > > > It is needed for the RK356X combo phy. > > How so ? > > The id must be set in the phynode creation with phynode_create so I > don't understand why you need this. > Will revert this commit until we find a proper solution. Ganbold > > > --- > > sys/dev/extres/phy/phy.c | 7 +++++++ > > sys/dev/extres/phy/phy.h | 1 + > > 2 files changed, 8 insertions(+) > > > > diff --git a/sys/dev/extres/phy/phy.c b/sys/dev/extres/phy/phy.c > > index 0ed633ffbe41..e95cfdfbd4b3 100644 > > --- a/sys/dev/extres/phy/phy.c > > +++ b/sys/dev/extres/phy/phy.c > > @@ -210,6 +210,13 @@ intptr_t phynode_get_id(struct phynode *phynode) > > return (phynode->id); > > } > > > > +void > > +phynode_set_id(struct phynode *phynode, intptr_t id) > > +{ > > + > > + phynode->id =3D id; > > +} > > + > > #ifdef FDT > > phandle_t > > phynode_get_ofw_node(struct phynode *phynode) > > diff --git a/sys/dev/extres/phy/phy.h b/sys/dev/extres/phy/phy.h > > index b0e5249e4246..34ba59447257 100644 > > --- a/sys/dev/extres/phy/phy.h > > +++ b/sys/dev/extres/phy/phy.h > > @@ -65,6 +65,7 @@ struct phynode *phynode_register(struct phynode > *phynode); > > void *phynode_get_softc(struct phynode *phynode); > > device_t phynode_get_device(struct phynode *phynode); > > intptr_t phynode_get_id(struct phynode *phynode); > > +void phynode_set_id(struct phynode *phynode, intptr_t id); > > int phynode_enable(struct phynode *phynode); > > int phynode_disable(struct phynode *phynode); > > int phynode_status(struct phynode *phynode, int *status); > > > -- > Emmanuel Vadot <manu@bidouilliste.com> <manu@FreeBSD.org> > --000000000000604e3305f02fceff Content-Type: text/html; charset="UTF-8" Content-Transfer-Encoding: quoted-printable <div dir=3D"ltr"><div dir=3D"ltr"><br></div><br><div class=3D"gmail_quote">= <div dir=3D"ltr" class=3D"gmail_attr">On Mon, Dec 19, 2022 at 10:18 PM Emma= nuel Vadot <<a href=3D"mailto:manu@bidouilliste.com">manu@bidouilliste.c= om</a>> wrote:<br></div><blockquote class=3D"gmail_quote" style=3D"margi= n:0px 0px 0px 0.8ex;border-left:1px solid rgb(204,204,204);padding-left:1ex= ">On Mon, 19 Dec 2022 13:10:14 GMT<br> Ganbold Tsagaankhuu <ganbold@FreeBSD.org> wrote:<br> <br> > The branch main has been updated by ganbold:<br> > <br> > URL: <a href=3D"https://cgit.FreeBSD.org/src/commit/?id=3D6692670f58f9= 069e59bc0c958fdaefb9bea5f726" rel=3D"noreferrer" target=3D"_blank">https://= cgit.FreeBSD.org/src/commit/?id=3D6692670f58f9069e59bc0c958fdaefb9bea5f726<= /a><br> > <br> > commit 6692670f58f9069e59bc0c958fdaefb9bea5f726<br> > Author:=C2=A0 =C2=A0 =C2=A0S=C3=B8ren Schmidt <sos@FreeBSD.org><= br> > AuthorDate: 2022-12-19 12:56:51 +0000<br> > Commit:=C2=A0 =C2=A0 =C2=A0Ganbold Tsagaankhuu <ganbold@FreeBSD.org= ><br> > CommitDate: 2022-12-19 13:08:42 +0000<br> > <br> >=C2=A0 =C2=A0 =C2=A0Enable setting the phy id.<br> >=C2=A0 =C2=A0 =C2=A0<br> >=C2=A0 =C2=A0 =C2=A0It is needed for the RK356X combo phy.<br> <br> =C2=A0How so ?<br> <br> =C2=A0The id must be set in the phynode creation with phynode_create so I<b= r> don't understand why you need this.<br></blockquote><div><br></div><div= >Will revert this commit until we find a proper solution.</div><div><br></d= iv><div>Ganbold</div><div><br></div><div>=C2=A0</div><blockquote class=3D"g= mail_quote" style=3D"margin:0px 0px 0px 0.8ex;border-left:1px solid rgb(204= ,204,204);padding-left:1ex"> <br> > ---<br> >=C2=A0 sys/dev/extres/phy/phy.c | 7 +++++++<br> >=C2=A0 sys/dev/extres/phy/phy.h | 1 +<br> >=C2=A0 2 files changed, 8 insertions(+)<br> > <br> > diff --git a/sys/dev/extres/phy/phy.c b/sys/dev/extres/phy/phy.c<br> > index 0ed633ffbe41..e95cfdfbd4b3 100644<br> > --- a/sys/dev/extres/phy/phy.c<br> > +++ b/sys/dev/extres/phy/phy.c<br> > @@ -210,6 +210,13 @@ intptr_t phynode_get_id(struct phynode *phynode)<= br> >=C2=A0 =C2=A0 =C2=A0 =C2=A0return (phynode->id);<br> >=C2=A0 }<br> >=C2=A0 <br> > +void<br> > +phynode_set_id(struct phynode *phynode, intptr_t id)<br> > +{<br> > +<br> > +=C2=A0 =C2=A0 =C2=A0phynode->id =3D id;<br> > +}<br> > +<br> >=C2=A0 #ifdef FDT<br> >=C2=A0 phandle_t<br> >=C2=A0 phynode_get_ofw_node(struct phynode *phynode)<br> > diff --git a/sys/dev/extres/phy/phy.h b/sys/dev/extres/phy/phy.h<br> > index b0e5249e4246..34ba59447257 100644<br> > --- a/sys/dev/extres/phy/phy.h<br> > +++ b/sys/dev/extres/phy/phy.h<br> > @@ -65,6 +65,7 @@ struct phynode *phynode_register(struct phynode *phy= node);<br> >=C2=A0 void *phynode_get_softc(struct phynode *phynode);<br> >=C2=A0 device_t phynode_get_device(struct phynode *phynode);<br> >=C2=A0 intptr_t phynode_get_id(struct phynode *phynode);<br> > +void phynode_set_id(struct phynode *phynode, intptr_t id);<br> >=C2=A0 int phynode_enable(struct phynode *phynode);<br> >=C2=A0 int phynode_disable(struct phynode *phynode);<br> >=C2=A0 int phynode_status(struct phynode *phynode, int *status);<br> <br> <br> -- <br> Emmanuel Vadot <<a href=3D"mailto:manu@bidouilliste.com" target=3D"_blan= k">manu@bidouilliste.com</a>> <manu@FreeBSD.org><br> </blockquote></div></div> --000000000000604e3305f02fceff--
Want to link to this message? Use this URL: <https://mail-archive.FreeBSD.org/cgi/mid.cgi?CAGtf9xMjzfNf8MHeR7fstpCigSo%2BAHjTYp7aAG%2BNAy3DW=p-yg>