Groups | Search | Server Info | Keyboard shortcuts | Login | Register [http] [https] [nntp] [nntps]

Groups > linux.kernel > #1299798

Re: [PATCH] net: emac: emac gigabit ethernet controller driver

Show all headers | View raw

On Tue, Dec 15, 2015 at 4:49 PM, Gilad Avidov <gavidov@codeaurora.org> wrote:
> On Mon, 14 Dec 2015 17:39:09 -0800
> Florian Fainelli <f.fainelli@gmail.com> wrote:
>
>> On 14/12/15 16:19, Gilad Avidov wrote:
>>
>> [snip]
>>
>> > +                   "sgmii_irq";
>> > +           qcom,emac-gpio-mdc = <&msmgpio 123 0>;
>> > +           qcom,emac-gpio-mdio = <&msmgpio 124 0>;
>> > +           qcom,emac-tstamp-en;
>> > +           qcom,emac-ptp-frac-ns-adj = <125000000 1>;
>> > +           phy-addr = <0>;
>>
>> Please use the standard Ethernet PHY and MDIO device tree bindings to
>> describe your MAC to PHY connection here, that includes using a
>> phy-connection-type property to describe the (x)MII lanes.
>>
>
>
> Hi Florian,
>
> Thank you for the review.
>
> Unfortunately this Ethernet controller's PHY is non standard and fits
> poorly into the standard MDIO framework layer. Rather than read/writs
> over MDIO only, this hw have some of the PHY registers internal and
> accessed by memory mapped IO, while others are accessed over the MDIO.
> Some standard functions requires using both. Additionally a number
> of different functions are controlled from different fields of the
> same register.

Even so, the bindings should follow the standard binding for MDIO bus
whether you can use the common kernel infrastructure or not.

Having internal phy connected to external phy is pretty common for
10G. Not sure if that is what you mean here or not.

Rob
--
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majordomo@vger.kernel.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html
Please read the FAQ at  http://www.tux.org/lkml/

Back to linux.kernel | Previous | Next — Previous in thread | Find similar | Unroll thread

Thread

Re: [PATCH] net: emac: emac gigabit ethernet controller driver Florian Fainelli <f.fainelli@gmail.com> - 2015-12-15 02:40 +0100
  Re: [PATCH] net: emac: emac gigabit ethernet controller driver Christopher Covington <cov@codeaurora.org> - 2015-12-15 15:40 +0100
    Re: [PATCH] net: emac: emac gigabit ethernet controller driver Arnd Bergmann <arnd@arndb.de> - 2015-12-15 16:00 +0100
      Re: [PATCH] net: emac: emac gigabit ethernet controller driver Timur Tabi <timur@codeaurora.org> - 2015-12-15 16:20 +0100
        Re: [PATCH] net: emac: emac gigabit ethernet controller driver Arnd Bergmann <arnd@arndb.de> - 2015-12-15 16:50 +0100
          Re: [PATCH] net: emac: emac gigabit ethernet controller driver Timur Tabi <timur@codeaurora.org> - 2015-12-15 22:10 +0100
            Re: [PATCH] net: emac: emac gigabit ethernet controller driver Arnd Bergmann <arnd@arndb.de> - 2015-12-15 23:00 +0100
  Re: [PATCH] net: emac: emac gigabit ethernet controller driver Gilad Avidov <gavidov@codeaurora.org> - 2015-12-15 23:50 +0100
    Re: [PATCH] net: emac: emac gigabit ethernet controller driver Rob Herring <robh@kernel.org> - 2016-01-01 00:10 +0100

csiph-web