# RE: \[PATCH\] p1010rdb: gianfar config does not have queues.

- From: Aggrwal Poonam-B10812 \<hidden\>
- Date: 2012-03-17 05:37:16 
- List: linuxppc-dev
- Page: https://ratatoskr.run/linuxppc-dev/2012/03/9468877
- Thread: https://ratatoskr.run/linuxppc-dev/2011/08/9465896/t.md

```text
Hello Pankaj, Rajan

DO you have any comments on the below patch of gianfar?
> > -			fsl,num_rx_queues =3D <0x8>;
> > -			fsl,num_tx_queues =3D <0x8>;
Have been removed from P1010RDB device tree to avoid a kernel panic.

Kumar, as such I see this is the old dts format. Also in the latest sdk tre=
e these properties are defined, not sure it is working on P1010RDB.
I can check on this and get back.


Regards
Poonam

> -----Original Message-----
> From: Kumar Gala [mailto:<redacted>]
> Sent: Friday, March 16, 2012 8:50 PM
> To: Aggrwal Poonam-B10812
> Cc: Robin Holt; U Bhaskar-B22300; PPC list; Eric Dumazet
> Subject: Re: [PATCH] p1010rdb: gianfar config does not have queues.
>=20
>=20
> On Aug 11, 2011, at 9:25 AM, Robin Holt wrote:
>=20
> > If I have the the fsl,num_rx_queues and fsl,num_tx_queues properties
> > defined in the p1010's device tree file, I get a kernel panic very
> > shortly after boot.  The failure indicates we are configuring the
> > gianfar.c driver for a queue depth greater than actual.  Removing the
> > properties got the problem resolved.
> >
> > Signed-off-by: Robin Holt <redacted>
> > To: U Bhaskar-B22300 <redacted>
> > Cc: PPC list <redacted>
> > Cc: Eric Dumazet <redacted>
>=20
> Poonam,
>=20
> Can you comment on this patch, does it look correct?
>=20
> - k
>=20
> >
> > diff --git a/arch/powerpc/boot/dts/p1010si.dtsi
> > b/arch/powerpc/boot/dts/p1010si.dtsi
> > index 7f51104..91566aa 100644
> > --- a/arch/powerpc/boot/dts/p1010si.dtsi
> > +++ b/arch/powerpc/boot/dts/p1010si.dtsi
> > @@ -258,8 +258,6 @@
> > 			device_type =3D "network";
> > 			model =3D "eTSEC";
> > 			compatible =3D "fsl,etsec2";
> > -			fsl,num_rx_queues =3D <0x8>;
> > -			fsl,num_tx_queues =3D <0x8>;
> > 			local-mac-address =3D [ 00 00 00 00 00 00 ];
> > 			interrupt-parent =3D <&mpic>;
> >
> > @@ -280,8 +278,6 @@
> > 			device_type =3D "network";
> > 			model =3D "eTSEC";
> > 			compatible =3D "fsl,etsec2";
> > -			fsl,num_rx_queues =3D <0x8>;
> > -			fsl,num_tx_queues =3D <0x8>;
> > 			local-mac-address =3D [ 00 00 00 00 00 00 ];
> > 			interrupt-parent =3D <&mpic>;
> >
> > @@ -302,8 +298,6 @@
> > 			device_type =3D "network";
> > 			model =3D "eTSEC";
> > 			compatible =3D "fsl,etsec2";
> > -			fsl,num_rx_queues =3D <0x8>;
> > -			fsl,num_tx_queues =3D <0x8>;
> > 			local-mac-address =3D [ 00 00 00 00 00 00 ];
> > 			interrupt-parent =3D <&mpic>;
> >
> > _______________________________________________
> > Linuxppc-dev mailing list
> > Linuxppc-dev@lists.ozlabs.org
> > https://lists.ozlabs.org/listinfo/linuxppc-dev
>=20
```