RE: [PATCH][v4] power/fsl: add MDIO dt binding for FMan

2015-01-29 Thread Shaohui Xie
> -Original Message- > From: Wood Scott-B07421 > Sent: Friday, January 30, 2015 10:44 AM > To: Xie Shaohui-B21989 > Cc: linuxppc-dev@lists.ozlabs.org; devicet...@vger.kernel.org; Medve > Emilian-EMMEDVE1 > Subject: Re: [PATCH][v4] power/fsl: add MDIO dt binding for F

RE: [PATCH][v4] power/fsl: add MDIO dt binding for FMan

2015-01-29 Thread Shaohui Xie
> -Original Message- > From: Wood Scott-B07421 > Sent: Friday, January 30, 2015 8:54 AM > To: shh@gmail.com > Cc: linuxppc-dev@lists.ozlabs.org; devicet...@vger.kernel.org; Medve > Emilian-EMMEDVE1; Xie Shaohui-B21989 > Subject: Re: [PATCH][v4] power/fsl: add MDI

Re: [PATCH][v4] power/fsl: add MDIO dt binding for FMan

2015-01-29 Thread Scott Wood
; > Emilian-EMMEDVE1; Xie Shaohui-B21989 > > Subject: Re: [PATCH][v4] power/fsl: add MDIO dt binding for FMan > > > > On Wed, 2015-01-28 at 19:54 +0800, shh@gmail.com wrote: > > > +- interrupts > > > + Usage: required > > > + Val

Re: [PATCH][v4] power/fsl: add MDIO dt binding for FMan

2015-01-29 Thread Scott Wood
On Wed, 2015-01-28 at 19:54 +0800, shh@gmail.com wrote: > +- interrupts > + Usage: required > + Value type: > + Definition: Event interrupt of external MDIO controller. What if this MDIO controller is not "external"? Should Usage say "required for external