netdev - Query: Regarding 8211E Phy module support in drivers/net/phy

lists.openwall.net		lists / announce owl-users owl-dev john-users john-dev passwdqc-users yescrypt popa3d-users / oss-security kernel-hardening musl sabotage tlsify passwords / crypt-dev xvendor / Bugtraq Full-Disclosure linux-kernel linux-netdev linux-ext4 linux-hardening linux-cve-announce PHC
Open Source and information security mailing list archives

Hash Suite: Windows password security audit tool. GUI, reports in PDF.

[<prev] [next>] [thread-next>] [day] [month] [year] [list]

Message-ID: <A1A6EA40F8503D48BB002B42BD65974E0A0E6C8C@039-SN2MPN1-012.039d.mgd.msft.net>
Date:	Fri, 2 Aug 2013 18:50:42 +0000
From:	Sharma Bhupesh-B45370 <B45370@...escale.com>
To:	"'peppe.cavallaro@...com'" <peppe.cavallaro@...com>
CC:	"'netdev@...r.kernel.org'" <netdev@...r.kernel.org>
Subject: Query: Regarding 8211E Phy module support in drivers/net/phy

Hi Peppe,

This regarding the patch prepared by you for adding RTL8211E support in
drivers/net/phy/realtek.c

commit ef3d90491a15f0d5cf1ec39a38a45dac6968fb2a
Author: Giuseppe CAVALLARO <peppe.cavallaro@...com>
Date:   Wed Jan 23 00:30:03 2013 +0000

    net: phy: realtek: add rtl8211e driver
    
    This patch adds the minimal driver to manage the
    Realtek RTL8211E 10/100/1000 Transceivers.

I see a macro specific to 8211E:
+#define        RTL8211E_INER_LINK_STAT 0x10

I was having a look at the RTL8211E(G) datasheet and could see
the following bit positions for INER register (Address 0x12):

[7:1] as Reserved
[0] as Jabber Interrupt

If I am not wrong a value 0x10 written to the INER register tries to set bit [4] which seems
Reserved.

Or am I missing something here.
Please let me know your views on the same.

Thanks,
Bhupesh 

--
To unsubscribe from this list: send the line "unsubscribe netdev" in
the body of a message to majordomo@...r.kernel.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html