lkml.org 
[lkml]   [2014]   [Nov]   [11]   [last100]   RSS Feed
Views: [wrap][no wrap]   [headers]  [forward] 
 
Messages in this thread
    Patch in this message
    /
    Date
    From
    SubjectRe: [PATCH 19/22] dt/bindings: add micrel,rmii_ref_clk_sel_25_mhz to eth-phy binding
    On Tue, Nov 11, 2014 at 05:57:42PM +0000, Mark Rutland wrote:
    > On Tue, Nov 11, 2014 at 05:37:37PM +0000, Johan Hovold wrote:
    > > Add "micrel,rmii_ref_clk_sel_25_mhz" to Micrel ethernet PHY binding
    > > documentation.
    > >
    > > Cc: devicetree@vger.kernel.org
    > > Signed-off-by: Johan Hovold <johan@kernel.org>
    > > ---
    > > Documentation/devicetree/bindings/net/micrel.txt | 5 +++++
    > > 1 file changed, 5 insertions(+)
    > >
    > > diff --git a/Documentation/devicetree/bindings/net/micrel.txt b/Documentation/devicetree/bindings/net/micrel.txt
    > > index a1bab5eaae02..9b08dd6551dd 100644
    > > --- a/Documentation/devicetree/bindings/net/micrel.txt
    > > +++ b/Documentation/devicetree/bindings/net/micrel.txt
    > > @@ -19,6 +19,11 @@ Optional properties:
    > >
    > > See the respective PHY datasheet for the mode values.
    > >
    > > + - micrel,rmii_ref_clk_sel_25_mhz: rmii_ref_clk_sel bit selects 25 MHz mode
    > > +
    > > + Whether 25 MHz (rather than 50 Mhz) clock mode is selected
    > > + when the rmii_ref_clk_sel bit is set.
    >
    > s/_/-/ in property names please.

    Ouch, copied from variable name, sorry.

    > That said, I don't follow the meaning. Does this cause the kernel to do
    > something different, or is is simply that a 25MHz ref clock is wired up?

    Yes, the driver currently sets this configuration bit based on a common
    clock binding.

    However, it turns out the meaning of the bit is reversed on some PHY
    variants. On most PHYs 50 MHz mode is selected by setting this bit,
    whereas on the PHYs that need this new property, setting it selects 25
    MHz mode instead.

    > Surely that should be described via the common clock bindings? Or if
    > internal through a clock-frequency property?

    The driver currently selects the mode using the common clock bindings,
    but this new property is needed to properly handle those PHY variants on
    which the clock configuration bit has the reverse meaning.
    Thanks,
    Johan


    \
     
     \ /
      Last update: 2014-11-11 19:41    [W:4.182 / U:0.260 seconds]
    ©2003-2020 Jasper Spaans|hosted at Digital Ocean and TransIP|Read the blog|Advertise on this site