<html>

<head>
<META HTTP-EQUIV="Content-Type" CONTENT="text/html; charset=us-ascii">
<meta name=Generator content="Microsoft Word 11 (filtered)">
<style>
<!--
 /* Font Definitions */
 @font-face
        {font-family:"MS Mincho";
        panose-1:2 2 6 9 4 2 5 8 3 4;}
@font-face
        {font-family:Tahoma;
        panose-1:2 11 6 4 3 5 4 4 2 4;}
@font-face
        {font-family:"\@MS Mincho";
        panose-1:2 2 6 9 4 2 5 8 3 4;}
 /* Style Definitions */
 p.MsoNormal, li.MsoNormal, div.MsoNormal
        {margin:0in;
        margin-bottom:.0001pt;
        font-size:12.0pt;
        font-family:"Times New Roman";}
a:link, span.MsoHyperlink
        {color:blue;
        text-decoration:underline;}
a:visited, span.MsoHyperlinkFollowed
        {color:blue;
        text-decoration:underline;}
pre
        {margin:0in;
        margin-bottom:.0001pt;
        font-size:10.0pt;
        font-family:"Courier New";}
span.EmailStyle18
        {font-family:Arial;
        color:navy;}
@page Section1
        {size:8.5in 11.0in;
        margin:1.0in 1.25in 1.0in 1.25in;}
div.Section1
        {page:Section1;}
-->
</style>

</head>

<body lang=EN-US link=blue vlink=blue>

<div class=Section1>

<p class=MsoNormal><font size=2 color=navy face=Arial><span style='font-size:
10.0pt;font-family:Arial;color:navy'>Ben,</span></font></p>

<p class=MsoNormal><font size=2 color=navy face=Arial><span style='font-size:
10.0pt;font-family:Arial;color:navy'>&nbsp;</span></font></p>

<p class=MsoNormal><font size=2 color=navy face=Arial><span style='font-size:
10.0pt;font-family:Arial;color:navy'>I work with Nishanth and he has already
presented what he added. &nbsp;I&#8217;ve no comment beyond that (other than it works well
for us).</span></font></p>

<p class=MsoNormal><font size=2 color=navy face=Arial><span style='font-size:
10.0pt;font-family:Arial;color:navy'>&nbsp;</span></font></p>

<p class=MsoNormal><font size=2 color=navy face=Arial><span style='font-size:
10.0pt;font-family:Arial;color:navy'>Regards,</span></font></p>

<p class=MsoNormal><font size=2 color=navy face=Arial><span style='font-size:
10.0pt;font-family:Arial;color:navy'>Richard W.</span></font></p>

<p class=MsoNormal><font size=2 color=navy face=Arial><span style='font-size:
10.0pt;font-family:Arial;color:navy'>&nbsp;</span></font></p>

<div style='border:none;border-left:solid blue 1.5pt;padding:0in 0in 0in 4.0pt'>

<div>

<div class=MsoNormal align=center style='text-align:center'><font size=3
face="Times New Roman"><span style='font-size:12.0pt'>

<hr size=2 width="100%" align=center tabindex=-1>

</span></font></div>

<p class=MsoNormal><b><font size=2 face=Tahoma><span style='font-size:10.0pt;
font-family:Tahoma;font-weight:bold'>From:</span></font></b><font size=2
face=Tahoma><span style='font-size:10.0pt;font-family:Tahoma'>
u-boot-users-admin@lists.sourceforge.net
[mailto:u-boot-users-admin@lists.sourceforge.net] <b><span style='font-weight:
bold'>On Behalf Of </span></b>Ben Warren<br>
<b><span style='font-weight:bold'>Sent:</span></b> Tuesday, May 16, 2006 8:50
AM<br>
<b><span style='font-weight:bold'>To:</span></b> Wolfgang Denk<br>
<b><span style='font-weight:bold'>Cc:</span></b> Kumar Gala;
u-boot-users@lists.sourceforge.net<br>
<b><span style='font-weight:bold'>Subject:</span></b> Re: [U-Boot-Users] Using
a second I2C interface</span></font></p>

</div>

<p class=MsoNormal><font size=3 face="Times New Roman"><span style='font-size:
12.0pt'>&nbsp;</span></font></p>

<p class=MsoNormal><font size=3 face="Times New Roman"><span style='font-size:
12.0pt'>Wolfgang,<br>
<br>
This sounds like a sensible approach.&nbsp; This should clean the code up
nicely (as you'll notice, the patch that I submitted yesterday adds quite a bit
of noisy code).&nbsp; Let's look at what Richard Woodruff submits, and we can
take it from there.<br>
<br>
Too bad I didn't get your input until this morning, even though your message
went out on Friday...<br>
<br>
regards,<br>
Ben<br>
<br>
On Fri, 2006-05-12 at 21:01 +0200, Wolfgang Denk wrote: </span></font></p>

<pre><font size=2 face="Courier New"><span style='font-size:10.0pt'>&nbsp;</span></font></pre><pre><font
size=2 color=black face="Courier New"><span style='font-size:10.0pt;color:black'>Hello,</span></font></pre><pre><font
size=2 face="Courier New"><span style='font-size:10.0pt'>&nbsp;</span></font></pre><pre><font
size=2 color=black face="Courier New"><span style='font-size:10.0pt;color:black'>in message &lt;<a
href="mailto:E541A453-EC07-4597-8D60-2EE3919C1A18@kernel.crashing.org">E541A453-EC07-4597-8D60-2EE3919C1A18@kernel.crashing.org</a>&gt; you wrote:</span></font></pre><pre><font
size=2 color=black face="Courier New"><span style='font-size:10.0pt;color:black'>&gt; </span></font></pre><pre><font
size=2 color=black face="Courier New"><span style='font-size:10.0pt;color:black'>&gt; &gt; The CPU I'm using (MPC8349) has two hardware I2C interfaces, and&nbsp; </span></font></pre><pre><font
size=2 color=black face="Courier New"><span style='font-size:10.0pt;color:black'>&gt; &gt; I'd like to access the second one in U-boot.&nbsp; Implementing this&nbsp; </span></font></pre><pre><font
size=2 color=black face="Courier New"><span style='font-size:10.0pt;color:black'>&gt; &gt; looks easy to me, but I don't want to reinvent the wheel if it's&nbsp; </span></font></pre><pre><font
size=2 color=black face="Courier New"><span style='font-size:10.0pt;color:black'>&gt; &gt; already been done elsewhere.&nbsp; Has anyone done this?&nbsp; If not, I'll&nbsp; </span></font></pre><pre><font
size=2 color=black face="Courier New"><span style='font-size:10.0pt;color:black'>&gt; &gt; follow up with a proposal.</span></font></pre><pre><font
size=2 color=black face="Courier New"><span style='font-size:10.0pt;color:black'>&gt; </span></font></pre><pre><font
size=2 color=black face="Courier New"><span style='font-size:10.0pt;color:black'>&gt; Depending on what you want to do, you can cheat and just change the&nbsp; </span></font></pre><pre><font
size=2 color=black face="Courier New"><span style='font-size:10.0pt;color:black'>&gt; pointer to the 2nd interface.&nbsp; If you want something more full&nbsp; </span></font></pre><pre><font
size=2 color=black face="Courier New"><span style='font-size:10.0pt;color:black'>&gt; featured up to the command level then I think some discussion would&nbsp; </span></font></pre><pre><font
size=2 color=black face="Courier New"><span style='font-size:10.0pt;color:black'>&gt; be required on how the commands should work for specifying which&nbsp; </span></font></pre><pre><font
size=2 color=black face="Courier New"><span style='font-size:10.0pt;color:black'>&gt; controller to use.</span></font></pre><pre><font
size=2 face="Courier New"><span style='font-size:10.0pt'>&nbsp;</span></font></pre><pre><font
size=2 color=black face="Courier New"><span style='font-size:10.0pt;color:black'>A clean design should allow to switch I2C&nbsp; busses&nbsp; usign&nbsp; a&nbsp; command;</span></font></pre><pre><font
size=2 color=black face="Courier New"><span style='font-size:10.0pt;color:black'>additionally,&nbsp; I think we should try to stay backward-compatible with</span></font></pre><pre><font
size=2 color=black face="Courier New"><span style='font-size:10.0pt;color:black'>the existing code. Eventually this is not too difficult:&nbsp; similar&nbsp; to</span></font></pre><pre><font
size=2 color=black face="Courier New"><span style='font-size:10.0pt;color:black'>the&nbsp; &quot;ide&quot;&nbsp; or&nbsp; &quot;usb&quot;&nbsp; commands&nbsp; we&nbsp; could&nbsp; create a new command &quot;i2c</span></font></pre><pre><font
size=2 color=black face="Courier New"><span style='font-size:10.0pt;color:black'>device&quot; which takes as argument the number of the I2C bus which&nbsp; shal</span></font></pre><pre><font
size=2 color=black face="Courier New"><span style='font-size:10.0pt;color:black'>become&nbsp; the&nbsp; current&nbsp; (=&nbsp; active)&nbsp; bus. Then all existing I2C related</span></font></pre><pre><font
size=2 color=black face="Courier New"><span style='font-size:10.0pt;color:black'>commands can be left unchanged (and &quot;i2c&nbsp; device&quot;&nbsp; without&nbsp; arguments</span></font></pre><pre><font
size=2 color=black face="Courier New"><span style='font-size:10.0pt;color:black'>should of course print the ID of the current bus).</span></font></pre><pre><font
size=2 face="Courier New"><span style='font-size:10.0pt'>&nbsp;</span></font></pre><pre><font
size=2 color=black face="Courier New"><span style='font-size:10.0pt;color:black'>In parallel, existing I2C commands could be changed to migrate&nbsp; to&nbsp; a</span></font></pre><pre><font
size=2 color=black face="Courier New"><span style='font-size:10.0pt;color:black'>new&nbsp; &quot;i2c&quot;&nbsp; group&nbsp; of&nbsp; commands:&nbsp; &quot;imd&quot;&nbsp; would become &quot;i2c md&quot;, &quot;imw&quot;</span></font></pre><pre><font
size=2 color=black face="Courier New"><span style='font-size:10.0pt;color:black'>becomes &quot;i2c mw&quot;, &quot;iprobe&quot; becomes &quot;i2c probe&quot;, etc. For some&nbsp; (long)</span></font></pre><pre><font
size=2 color=black face="Courier New"><span style='font-size:10.0pt;color:black'>transition&nbsp; period old and new commands could be provided in parallel</span></font></pre><pre><font
size=2 color=black face="Courier New"><span style='font-size:10.0pt;color:black'>(at least optionally). Later, the old syntax could be dropped.</span></font></pre><pre><font
size=2 face="Courier New"><span style='font-size:10.0pt'>&nbsp;</span></font></pre><pre><font
size=2 color=black face="Courier New"><span style='font-size:10.0pt;color:black'>Best regards,</span></font></pre><pre><font
size=2 face="Courier New"><span style='font-size:10.0pt'>&nbsp;</span></font></pre><pre><font
size=2 color=black face="Courier New"><span style='font-size:10.0pt;color:black'>Wolfgang Denk</span></font></pre><pre><font
size=2 face="Courier New"><span style='font-size:10.0pt'>&nbsp;</span></font></pre></div>

</div>

</body>

</html>