diff mbox

[RFC,v2,3/4] ACPI: DBG2: add 16550 UART with 32-bit access

Message ID 20161206175830.6989-4-aleksey.makarov@linaro.org (mailing list archive)
State RFC, archived
Headers show

Commit Message

Aleksey Makarov Dec. 6, 2016, 5:58 p.m. UTC
It was suggested to add a new Microsoft Debug Port Table 2
(DBG2) (the table used to enumerate the various subtypes of serial
port covered by the SPCR) 16550 UART subtype that may be needed for
some additional platforms, such as those based upon AppliedMicro
X-Gene ARMv8 SoCs.  This new subtype would be 16550-compatible
with 32-bit access.  There already exists 32-bit variant
ACPI_DBG2_ARM_SBSA_32BIT of SBSA console ACPI_DBG2_ARM_SBSA_GENERIC.

This patch introduces it to Linux kernel.

Signed-off-by: Aleksey Makarov <aleksey.makarov@linaro.org>
---
 include/acpi/actbl2.h | 1 +
 1 file changed, 1 insertion(+)

Comments

Rafael J. Wysocki Dec. 6, 2016, 9 p.m. UTC | #1
On Tue, Dec 6, 2016 at 6:58 PM, Aleksey Makarov
<aleksey.makarov@linaro.org> wrote:
> It was suggested to add a new Microsoft Debug Port Table 2
> (DBG2) (the table used to enumerate the various subtypes of serial
> port covered by the SPCR) 16550 UART subtype that may be needed for
> some additional platforms, such as those based upon AppliedMicro
> X-Gene ARMv8 SoCs.  This new subtype would be 16550-compatible
> with 32-bit access.  There already exists 32-bit variant
> ACPI_DBG2_ARM_SBSA_32BIT of SBSA console ACPI_DBG2_ARM_SBSA_GENERIC.
>
> This patch introduces it to Linux kernel.
>
> Signed-off-by: Aleksey Makarov <aleksey.makarov@linaro.org>
> ---
>  include/acpi/actbl2.h | 1 +
>  1 file changed, 1 insertion(+)
>
> diff --git a/include/acpi/actbl2.h b/include/acpi/actbl2.h
> index c93dbad..f219b04 100644
> --- a/include/acpi/actbl2.h
> +++ b/include/acpi/actbl2.h
> @@ -371,6 +371,7 @@ struct acpi_dbg2_device {
>
>  #define ACPI_DBG2_16550_COMPATIBLE  0x0000
>  #define ACPI_DBG2_16550_SUBSET      0x0001
> +#define ACPI_DBG2_16550_32BIT       0x0002
>  #define ACPI_DBG2_ARM_PL011         0x0003
>  #define ACPI_DBG2_ARM_SBSA_32BIT    0x000D
>  #define ACPI_DBG2_ARM_SBSA_GENERIC  0x000E
> --

This is an ACPICA change and it first needs to be made to the upstream
ACPICA code base.

Thanks,
Rafael
--
To unsubscribe from this list: send the line "unsubscribe linux-acpi" in
the body of a message to majordomo@vger.kernel.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html
Jon Masters Dec. 6, 2016, 9:15 p.m. UTC | #2
Quick note - specifically, Rafael, this is not currently part of the SPCR specification and should not be considered for inclusion in Linux until that is decided. Once it is determined what the fix for SPCR is, that should then be driven into the component architecture and proposed here.
Rafael J. Wysocki Dec. 6, 2016, 9:19 p.m. UTC | #3
On Tue, Dec 6, 2016 at 10:15 PM, Jon Masters <jcm@redhat.com> wrote:
> Quick note - specifically, Rafael, this is not currently part of the SPCR specification and
> should not be considered for inclusion in Linux until that is decided. Once it is determined
> what the fix for SPCR is, that should then be driven into the component architecture and
> proposed here.

OK, good to know, thanks!


> --
> Computer Architect | Sent from my 64-bit #ARM Powered phone
>
>> On Dec 6, 2016, at 16:00, Rafael J. Wysocki <rafael@kernel.org> wrote:
>>
>> On Tue, Dec 6, 2016 at 6:58 PM, Aleksey Makarov
>> <aleksey.makarov@linaro.org> wrote:
>>> It was suggested to add a new Microsoft Debug Port Table 2
>>> (DBG2) (the table used to enumerate the various subtypes of serial
>>> port covered by the SPCR) 16550 UART subtype that may be needed for
>>> some additional platforms, such as those based upon AppliedMicro
>>> X-Gene ARMv8 SoCs.  This new subtype would be 16550-compatible
>>> with 32-bit access.  There already exists 32-bit variant
>>> ACPI_DBG2_ARM_SBSA_32BIT of SBSA console ACPI_DBG2_ARM_SBSA_GENERIC.
>>>
>>> This patch introduces it to Linux kernel.
>>>
>>> Signed-off-by: Aleksey Makarov <aleksey.makarov@linaro.org>
>>> ---
>>> include/acpi/actbl2.h | 1 +
>>> 1 file changed, 1 insertion(+)
>>>
>>> diff --git a/include/acpi/actbl2.h b/include/acpi/actbl2.h
>>> index c93dbad..f219b04 100644
>>> --- a/include/acpi/actbl2.h
>>> +++ b/include/acpi/actbl2.h
>>> @@ -371,6 +371,7 @@ struct acpi_dbg2_device {
>>>
>>> #define ACPI_DBG2_16550_COMPATIBLE  0x0000
>>> #define ACPI_DBG2_16550_SUBSET      0x0001
>>> +#define ACPI_DBG2_16550_32BIT       0x0002
>>> #define ACPI_DBG2_ARM_PL011         0x0003
>>> #define ACPI_DBG2_ARM_SBSA_32BIT    0x000D
>>> #define ACPI_DBG2_ARM_SBSA_GENERIC  0x000E
>>> --
>>
>> This is an ACPICA change and it first needs to be made to the upstream
>> ACPICA code base.
>>
>> Thanks,
>> Rafael
--
To unsubscribe from this list: send the line "unsubscribe linux-acpi" in
the body of a message to majordomo@vger.kernel.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html
diff mbox

Patch

diff --git a/include/acpi/actbl2.h b/include/acpi/actbl2.h
index c93dbad..f219b04 100644
--- a/include/acpi/actbl2.h
+++ b/include/acpi/actbl2.h
@@ -371,6 +371,7 @@  struct acpi_dbg2_device {
 
 #define ACPI_DBG2_16550_COMPATIBLE  0x0000
 #define ACPI_DBG2_16550_SUBSET      0x0001
+#define ACPI_DBG2_16550_32BIT       0x0002
 #define ACPI_DBG2_ARM_PL011         0x0003
 #define ACPI_DBG2_ARM_SBSA_32BIT    0x000D
 #define ACPI_DBG2_ARM_SBSA_GENERIC  0x000E