Groups | Search | Server Info | Keyboard shortcuts | Login | Register [http] [https] [nntp] [nntps]

Groups > linux.kernel > #1536713

Re: [PATCH] SPCR: check bit width for the 16550 UART

Show all headers | View raw

On 12/06/2016 01:53 AM, Jon Masters wrote:
> On 12/06/2016 01:34 AM, Jon Masters wrote:
>> On 12/05/2016 10:55 PM, Duc Dang wrote:
>>> On Mon, Dec 5, 2016 at 6:27 PM, Jon Masters <jcm@redhat.com> wrote:

>                         ToUUID ("daffd814-6eba-4d8c-8a91-bc9bbf4aa301") /* Device Properties for _DSD */, 
>                         Package (0x01)
>                         {
>                             Package (0x02)
>                             {
>                                 "clock-frequency", 
>                                 Zero
>                             }
>                         }

> I suspect Mustang is doing similar. So you read the magic frequency
                       ^^^^^^^^^^^^^
                     spoiler, it is :)

> info and pass this in through a DSD in the AML. It's ok, I've made my
> peace with this (but obviously we're trying to kill all DSD hacks),
> but it explains how this "works" after boot, but not for early console.
> 
> To fix this, we're going to need to be able to know that your 8250 is
> both needful of 32-bit accesses *AND* needs a different base UART clock.
> Fortunately our good friends at Microsoft are amenable to adding a
> subtype that covers this and are going to followup tomorrow for me.

For the earlycon setup, you'll probably benefit from figuring out
ahead of time how you'll want to handle this. The "easy" option
is to remove the baud in the case of X-Gene with new subtype
but that'll be potential fragile. You might also decide that
struct earlycon_device wants to grow a "quirks" flag that
you can set for this kind of thing (bound to be more).

Copying Rob as well in case he has suggestions for you. Rob, as
an FYI the AppliedMicro X-Gene 16550 "compatible" requires both
32-bit accesses (which Aleksey tried to address earlier in this
thread) but also has a special non-standard clock (to be fair,
it's not as if we architected a standard UART clock for arm64,
we just assumed it was like x86 but didn't mandate this early
when Applied were doing their design - another life lesson).
Microsoft will add a new DBG2 subtype that allows for both
of these situations. But you'll want to somehow convey this
information (quirks) in one of the earlycon structures.

Jon.

-- 
Computer Architect | Sent from my Fedora powered laptop

Back to linux.kernel | Previous | Next — Previous in thread | Next in thread | Find similar | Unroll thread

Thread

[PATCH] SPCR: check bit width for the 16550 UART Aleksey Makarov <aleksey.makarov@linaro.org> - 2016-12-05 14:10 +0100
  Re: [PATCH] SPCR: check bit width for the 16550 UART Duc Dang <dhdang@apm.com> - 2016-12-05 20:00 +0100
    Re: [PATCH] SPCR: check bit width for the 16550 UART Jon Masters <jcm@redhat.com> - 2016-12-06 00:30 +0100
      Re: [PATCH] SPCR: check bit width for the 16550 UART Duc Dang <dhdang@apm.com> - 2016-12-06 01:00 +0100
        Re: [PATCH] SPCR: check bit width for the 16550 UART Jon Masters <jcm@redhat.com> - 2016-12-06 01:10 +0100
        Re: [PATCH] SPCR: check bit width for the 16550 UART Jon Masters <jcm@redhat.com> - 2016-12-06 01:10 +0100
        Re: [PATCH] SPCR: check bit width for the 16550 UART Duc Dang <dhdang@apm.com> - 2016-12-06 01:40 +0100
          Re: [PATCH] SPCR: check bit width for the 16550 UART Duc Dang <dhdang@apm.com> - 2016-12-06 05:00 +0100
            Re: [PATCH] SPCR: check bit width for the 16550 UART Jon Masters <jcm@redhat.com> - 2016-12-06 07:40 +0100
              Re: [PATCH] SPCR: check bit width for the 16550 UART Jon Masters <jcm@redhat.com> - 2016-12-06 08:00 +0100
                Re: [PATCH] SPCR: check bit width for the 16550 UART Jon Masters <jcm@redhat.com> - 2016-12-06 08:20 +0100
                Re: [PATCH] SPCR: check bit width for the 16550 UART Aleksey Makarov <aleksey.makarov@linaro.org> - 2016-12-06 09:50 +0100
              Re: [PATCH] SPCR: check bit width for the 16550 UART Mark Salter <msalter@redhat.com> - 2016-12-07 16:30 +0100

csiph-web