<div dir="ltr"><br><div class="gmail_extra"><br><div class="gmail_quote">On Wed, Apr 20, 2016 at 6:47 PM, Chris Johns <span dir="ltr"><<a href="mailto:chrisj@rtems.org" target="_blank">chrisj@rtems.org</a>></span> wrote:<br><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex"><span class="">On 21/04/2016 8:57 AM, Joel Sherrill wrote:<br>
><br>
>     +/*<br>
>     + *  IO Register Access Routines<br>
>     + */<br>
>     +static uint8_t pci_ns16550_io_get_register(uint32_t addr, uint8_t i)<br>
>     +{<br>
>     +  uint8_t val = rtems_inb(addr + i);<br>
>     +  if (UART_PCI_IO)<br>
>     +    printk( "RD(%p -> 0x%02x) ", addr + i, val );<br>
>     +  return val;<br>
>     +}<br>
>     +<br>
>     +static void pci_ns16550_io_set_register(uint32_t addr, uint8_t i,<br>
>     uint8_t val)<br>
>     +{<br>
>     +  if (UART_PCI_IO)<br>
>     +    printk( "WR(%p <- 0x%02x) ", addr + i, val );<br>
>     +  rtems_outb(addr + i, val);<br>
>     +}<br>
>     +<br>
><br>
><br>
> Is it possible to use the same com access ports that are used for com1-com4?<br>
> Or at least move to shared ones?<br>
<br>
</span>It could be done if want.<br>
<span class=""><br></span></blockquote><div><br></div><div>Just pointing out common code possibilities.</div><div> </div><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex"><span class="">
><br>
> They are in conscfg.c. It would be good if we could eliminate some code<br>
> during this. :)<br>
><br>
<br>
</span>I have not looked at the differences between the old driver and the new<br>
one Sebastian created. We need to move to that driver for SMP support.<br>
Maybe this all gets cleaned up then?<br>
<span class="HOEnZb"><font color="#888888"><br></font></span></blockquote><div>Sure. There is definitely SMP work on the x86. Add this to the list</div><div>along with TLS, etc.</div><div> </div><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex"><span class="HOEnZb"><font color="#888888">
Chris<br>
</font></span></blockquote></div><br></div></div>