Date: Sat, 3 Dec 2011 08:36:08 +0800 From: Adrian Chadd <adrian@freebsd.org> To: Stefan Bethke <stb@lassitu.de> Cc: Aleksandr Rybalko <ray@dlink.ua>, freebsd-embedded@freebsd.org Subject: Re: TL-WR1043: switch Message-ID: <CAJ-Vmo=YMTuk6gOsjp15QWpRuBPkdDvM3JvamE6E38MhCsL7iw@mail.gmail.com> In-Reply-To: <E29B1787-FF42-4462-81C0-8185F70C45B6@lassitu.de> References: <68ABED76-CB1F-405A-8036-EC254F7511FA@lassitu.de> <3B3DB17D-BF87-40EE-B1C1-445F178E8844@lassitu.de> <86030CEE-6839-4B96-ACDC-2BA9AC1E4AE4@lassitu.de> <2D625CC9-A0E3-47AA-A504-CE8FB2F90245@lassitu.de> <203BF1C8-D528-40C9-8611-9C7AC7E43BAB@lassitu.de> <3C0E9CA3-E130-4E9A-ABCC-1782E28999D1@lassitu.de> <2B8826C7-00C7-4117-B424-4A86F1346DFF@bsdimp.com> <20111130231311.4a154bc5.ray@ddteam.net> <CAJ-Vmon8-yo-UQ%2B81feLT-Yr%2BJimMsEbLHWfd9kZP_s4804%2BtA@mail.gmail.com> <20111202164539.fff3ea91.ray@dlink.ua> <20111202191122.GK25601@funkthat.com> <E29B1787-FF42-4462-81C0-8185F70C45B6@lassitu.de>
next in thread | previous in thread | raw e-mail | index | archive | help
On 3 December 2011 08:02, Stefan Bethke <stb@lassitu.de> wrote: > I see two ways to make iicbus play with the Realtek "System Management In= terface" protocol: either make the patch I suggested optional via a hint, o= r let a slave device somehow indicate that it wants the relaxed semantics. = =A0The hint is likely the option that is easier to implement. =A0Writing a = new go-between bus driver is likely too complicated for me to accomplish ri= ght now. I think exposing it via a quirk somehow, so the iicbus code knows that for this particular child device, the relaxed semantics are required. That way an i2c eeprom will be fine using the existing code and semantics; the switch PHY child will use your relaxed semantics. As there's a few more realtek switch PHYs to write code for, I can only see this as being needed. :) Think you could code that up for review? Adrian
Want to link to this message? Use this URL: <https://mail-archive.FreeBSD.org/cgi/mid.cgi?CAJ-Vmo=YMTuk6gOsjp15QWpRuBPkdDvM3JvamE6E38MhCsL7iw>