Re: [PATCH 06/10] serial: 8250_dw: proper support for UARTs without busy functionality

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 



On Wed, 2015-09-16 at 14:30 +0300, Heikki Krogerus wrote:
> On Tue, Sep 15, 2015 at 04:29:45PM +0300, Andy Shevchenko wrote:
> > > +		id = acpi_match_device(p->dev->driver
> > > ->acpi_match_table, p->dev);
> > > +		if ((id && strcmp(id->id, "AMD0020") &&
> > > +		     strcmp(id->id, "APMC0D08")) || !id) {
> > 
> > Shouldn't we ask the actual owners of such devices if they have the
> > Busy Functionality enabled?
> 
> Yes. Now this check will simply ensure that the behaviour on those
> platforms is the same as before. But in any case, like you said, let's
> get the confirmation.
> 
> Ken, Feng! Could you tell us if the DW_apb_uart on your platforms has
> the Busy Functionality or not (UART_16550_COMPATIBLE)?

Thanks for your reminders.
Currently, "AMD0020" also does not support "UART_IIR_BUSY".


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



[Index of Archives]     [Kernel Newbies]     [Security]     [Netfilter]     [Bugtraq]     [Linux PPP]     [Linux FS]     [Yosemite News]     [MIPS Linux]     [ARM Linux]     [Linux Security]     [Linux RAID]     [Samba]     [Video 4 Linux]     [Linmodem]     [Device Mapper]     [Linux Kernel for ARM]

  Powered by Linux