[Bug 1001274] New: Add another KSZ8041 silicon-release PHY entry.

[Bug 1001274] New: Add another KSZ8041 silicon-release PHY entry.

[bugzilla-daemon]

Please do not reply to this email. Use the web interface provided at:
http://bugs.ecos.sourceware.org/show_bug.cgi?id=1001274

           Summary: Add another KSZ8041 silicon-release PHY entry.
           Product: eCos
           Version: CVS
          Platform: Other (please specify)
        OS/Version: All
            Status: UNCONFIRMED
          Severity: enhancement
          Priority: low
         Component: Patches and contributions
        AssignedTo: unassigned@bugs.ecos.sourceware.org
        ReportedBy: ilijak@siva.com.mk
                CC: ecos-patches@ecos.sourceware.org
             Class: Advice Request


Created an attachment (id=1297)
 --> (http://bugs.ecos.sourceware.org/attachment.cgi?id=1297)
KSZ8041.c.diff

A trivial patch to add another KSZ8041 silicon-release entry to PHY table.

-- 
Configure bugmail: http://bugs.ecos.sourceware.org/userprefs.cgi?tab=email
------- You are receiving this mail because: -------
You are on the CC list for the bug.

[Bug 1001274] Add another KSZ8041 silicon-release PHY entry.

[bugzilla-daemon]

Please do not reply to this email. Use the web interface provided at:
http://bugs.ecos.sourceware.org/show_bug.cgi?id=1001274

--- Comment #1 from Sergei Gavrikov <sergei.gavrikov@gmail.com> 2011-07-02 12:49:13 BST ---
Hi Ilija,

Googling around PHY IDs for KSZ8041 I found no PHY_ID2 = 0x1513 ...
Where this value came from? Can you, please, point on your source?
I got for this PHY chip PHY_ID1 = 0x22 and PHY_ID2 = 0x1512 only.

Sergei

-- 
Configure bugmail: http://bugs.ecos.sourceware.org/userprefs.cgi?tab=email
------- You are receiving this mail because: -------
You are on the CC list for the bug.

[Bug 1001274] Add another KSZ8041 silicon-release PHY entry.

[bugzilla-daemon]

Please do not reply to this email. Use the web interface provided at:
http://bugs.ecos.sourceware.org/show_bug.cgi?id=1001274

--- Comment #2 from Ilija Kocho <ilijak@siva.com.mk> 2011-07-02 13:45:21 BST ---
(In reply to comment #1)
> Hi Ilija,
> 
> Googling around PHY IDs for KSZ8041 I found no PHY_ID2 = 0x1513 ...
> Where this value came from? Can you, please, point on your source?
> I got for this PHY chip PHY_ID1 = 0x22 and PHY_ID2 = 0x1512 only.
> 
> Sergei

Hi Sergei, Thanks for prompt response.

I found it out in my Tower dev. kit during a course of making eth driver for
Kinetis. Here is the stat:
-----------------------------
PHY: Micrel KSZ8041 at addr 0
KSZ8041 MIIM Register setings:
r00: 3000 786d
r02: 0022 1513
r04: 01e1 45e1
r06: 0007 2001
r08: 0000 0000
r0a: 0000 0000
r0c: 0000 0000
r0e: 0000 0000
r10: 0000 0000
r12: 0000 0000
r14: 0001 0000
r16: 0600 0000
r18: 0000 0000
r1a: 0000 0029
r1c: 0008 0000
r1e: 0000 9198
----------------------------

I too didn't find this revision explicitly referenced in Micrel docs. There is
(probably older) document that refers to r03 == 2 and this one 
http://www.micrel.com/_PDF/Ethernet/datasheets/ksz8041nl_rnl.pdf
(probably newer) that simply states  "Indicates silicon revision". This change
is reflected also in document's revision history.

Ilija

-- 
Configure bugmail: http://bugs.ecos.sourceware.org/userprefs.cgi?tab=email
------- You are receiving this mail because: -------
You are on the CC list for the bug.

[Bug 1001274] Add another KSZ8041 silicon-release PHY entry.

[bugzilla-daemon]

Please do not reply to this email. Use the web interface provided at:
http://bugs.ecos.sourceware.org/show_bug.cgi?id=1001274

--- Comment #3 from Sergei Gavrikov <sergei.gavrikov@gmail.com> 2011-07-02 14:59:28 BST ---
(In reply to comment #2)
> (In reply to comment #1)
> > Hi Ilija,
> > 
> > Googling around PHY IDs for KSZ8041 I found no PHY_ID2 = 0x1513 ...
> > Where this value came from? Can you, please, point on your source?
> > I got for this PHY chip PHY_ID1 = 0x22 and PHY_ID2 = 0x1512 only.
> > 
> > Sergei
> 
> Hi Sergei, Thanks for prompt response.
> 
> I found it out in my Tower dev. kit during a course of making eth
> driver for Kinetis. Here is the stat:
> -----------------------------
> PHY: Micrel KSZ8041 at addr 0
> KSZ8041 MIIM Register setings:
> r00: 3000 786d
> r02: 0022 1513

Thank you, I see.

[snip]

> I too didn't find this revision explicitly referenced in Micrel docs.
> There is (probably older) document that refers to r03 == 2 and this
> one http://www.micrel.com/_PDF/Ethernet/datasheets/ksz8041nl_rnl.pdf
> (probably newer) that simply states  "Indicates silicon revision".
> This change is reflected also in document's revision history.

Thanks for the source, I found that table. So, it seems for me we may
apply the patch safely. Could you provide and ChangeLog record, please?

Thank you,
Sergei

-- 
Configure bugmail: http://bugs.ecos.sourceware.org/userprefs.cgi?tab=email
------- You are receiving this mail because: -------
You are on the CC list for the bug.

[Bug 1001274] Add another KSZ8041 silicon-release PHY entry.

[bugzilla-daemon]

Please do not reply to this email. Use the web interface provided at:
http://bugs.ecos.sourceware.org/show_bug.cgi?id=1001274

--- Comment #4 from Ilija Kocho <ilijak@siva.com.mk> 2011-07-02 15:44:21 BST ---
Created an attachment (id=1298)
 --> (http://bugs.ecos.sourceware.org/attachment.cgi?id=1298)
ChangeLog

(In reply to comment #3)
> (In reply to comment #2)
> > (In reply to comment #1)
> > > Hi Ilija,
> > > 
> > > Googling around PHY IDs for KSZ8041 I found no PHY_ID2 = 0x1513 ...
> > > Where this value came from? Can you, please, point on your source?
> > > I got for this PHY chip PHY_ID1 = 0x22 and PHY_ID2 = 0x1512 only.
> > > 
> > > Sergei
> > 
> > Hi Sergei, Thanks for prompt response.
> > 
> > I found it out in my Tower dev. kit during a course of making eth
> > driver for Kinetis. Here is the stat:
> > -----------------------------
> > PHY: Micrel KSZ8041 at addr 0
> > KSZ8041 MIIM Register setings:
> > r00: 3000 786d
> > r02: 0022 1513
> 
> Thank you, I see.
> 
> [snip]
> 
> > I too didn't find this revision explicitly referenced in Micrel docs.
> > There is (probably older) document that refers to r03 == 2 and this
> > one http://www.micrel.com/_PDF/Ethernet/datasheets/ksz8041nl_rnl.pdf
> > (probably newer) that simply states  "Indicates silicon revision".
> > This change is reflected also in document's revision history.
> 
> Thanks for the source, I found that table. So, it seems for me we may
> apply the patch safely. Could you provide and ChangeLog record, please?
> 
> Thank you,
> Sergei

Here it is.

Thanks
Ilija

-- 
Configure bugmail: http://bugs.ecos.sourceware.org/userprefs.cgi?tab=email
------- You are receiving this mail because: -------
You are on the CC list for the bug.

[Bug 1001274] Add another KSZ8041 silicon-release PHY entry.

[bugzilla-daemon]

Please do not reply to this email. Use the web interface provided at:
http://bugs.ecos.sourceware.org/show_bug.cgi?id=1001274

--- Comment #5 from Sergei Gavrikov <sergei.gavrikov@gmail.com> 2011-07-02 16:58:19 BST ---
Ilija, I changed your CL entry as we can find more silicon revisions then.

Checked-in.

Thank you for your contribution to eCos.

Sergei

-- 
Configure bugmail: http://bugs.ecos.sourceware.org/userprefs.cgi?tab=email
------- You are receiving this mail because: -------
You are on the CC list for the bug.

[Bug 1001274] Add another KSZ8041 silicon-release PHY entry.

[bugzilla-daemon]

Please do not reply to this email. Use the web interface provided at:
http://bugs.ecos.sourceware.org/show_bug.cgi?id=1001274

--- Comment #6 from Ilija Kocho <ilijak@siva.com.mk> 2011-07-02 21:32:43 BST ---
(In reply to comment #5)
> Ilija, I changed your CL entry as we can find more silicon revisions then.

NP, Thanks.

Ilija

-- 
Configure bugmail: http://bugs.ecos.sourceware.org/userprefs.cgi?tab=email
------- You are receiving this mail because: -------
You are on the CC list for the bug.

[Bug 1001274] Add another KSZ8041 silicon-release PHY entry.

[bugzilla-daemon]

Please do not reply to this email. Use the web interface provided at:
http://bugs.ecos.sourceware.org/show_bug.cgi?id=1001274

Ilija Kocho <ilijak@siva.com.mk> changed:

           What    |Removed                     |Added
----------------------------------------------------------------------------
             Status|UNCONFIRMED                 |RESOLVED
         Resolution|                            |CURRENTRELEASE

--- Comment #7 from Ilija Kocho <ilijak@siva.com.mk> 2011-08-25 09:28:07 BST ---
As checked in CVS.

-- 
Configure bugmail: http://bugs.ecos.sourceware.org/userprefs.cgi?tab=email
------- You are receiving this mail because: -------
You are on the CC list for the bug.