lkml.org 
[lkml]   [2020]   [Sep]   [21]   [last100]   RSS Feed
Views: [wrap][no wrap]   [headers]  [forward] 
 
Messages in this thread
    Patch in this message
    /
    From
    Subject[PATCH 2/2] x86: Support APU5 in PCEngines platform driver
    Date
    PCEngines make a number of SBC. APU5 has 5 mpcie slots + MSATA
    It also has support for 3x LTE modems with 6x SIM slots (pairs with a
    SIM switch device). Each mpcie slot for modems has a reset GPIO

    To ensure that the naming is sane between APU2/3/4/5 I have renamed the
    reset GPIOs to be modem1-reset, modem2-reset, etc. This is significant
    because the slots that can be reset change between APU2 and APU3/4

    GPIO for simswap is moved to the end of the list as it could be dropped
    for APU2 boards (but causes no harm to leave it in, hardware could be
    added to a future rev of the board).

    Structure of the GPIOs for APU5 is extremely similar to APU2-4, but
    many lines are moved around and there are simply more
    modems/resets/sim-swap lines to breakout.

    Signed-off-by: Ed Wildgoose <lists@wildgooses.com>
    ---
    drivers/platform/x86/pcengines-apuv2.c | 175 +++++++++++++++++++------
    1 file changed, 136 insertions(+), 39 deletions(-)

    diff --git a/drivers/platform/x86/pcengines-apuv2.c b/drivers/platform/x86/pcengines-apuv2.c
    index 41e3005cd..4dfc9ff9f 100644
    --- a/drivers/platform/x86/pcengines-apuv2.c
    +++ b/drivers/platform/x86/pcengines-apuv2.c
    @@ -1,7 +1,7 @@
    // SPDX-License-Identifier: GPL-2.0+

    /*
    - * PC-Engines APUv2/APUv3 board platform driver
    + * PC-Engines APUv2-5 board platform driver
    * for GPIO buttons and LEDs
    *
    * Copyright (C) 2018 metux IT consult
    @@ -22,38 +22,73 @@
    #include <linux/platform_data/gpio/gpio-amd-fch.h>

    /*
    - * NOTE: this driver only supports APUv2/3 - not APUv1, as this one
    + * NOTE: this driver only supports APUv2-5 - not APUv1, as this one
    * has completely different register layouts.
    */

    +/*
    + * There are a number of APU2 variants, with differing features
    + * APU2 has SIM slots 1/2 mapping to mPCIe sockets 1/2
    + * APU3/4 moved SIM slot 1 to mPCIe socket 3, ie logically reversed
    + * However, most APU3/4 have a SIM switch which we default on to reverse
    + * the order and keep physical SIM order matching physical modem order
    + *
    + * APU5 has 3x SIM sockets, all with a SIM switch
    + * several GPIOs are shuffled (see schematic), including MODESW
    + */
    +
    +/* Match types for dmi */
    +#define DMI_MATCH_APU2 2
    +#define DMI_MATCH_APU5 5
    +
    /* Register mappings */
    #define APU2_GPIO_REG_LED1 AMD_FCH_GPIO_REG_GPIO57
    #define APU2_GPIO_REG_LED2 AMD_FCH_GPIO_REG_GPIO58
    #define APU2_GPIO_REG_LED3 AMD_FCH_GPIO_REG_GPIO59_DEVSLP1
    #define APU2_GPIO_REG_MODESW AMD_FCH_GPIO_REG_GPIO32_GE1
    #define APU2_GPIO_REG_SIMSWAP AMD_FCH_GPIO_REG_GPIO33_GE2
    -#define APU2_GPIO_REG_MPCIE2 AMD_FCH_GPIO_REG_GPIO55_DEVSLP0
    -#define APU2_GPIO_REG_MPCIE3 AMD_FCH_GPIO_REG_GPIO51
    +#define APU2_GPIO_REG_RESETM1 AMD_FCH_GPIO_REG_GPIO51
    +#define APU2_GPIO_REG_RESETM2 AMD_FCH_GPIO_REG_GPIO55_DEVSLP0
    +
    +#define APU5_GPIO_REG_MODESW AMT_FCH_GPIO_REG_GEVT22
    +#define APU5_GPIO_REG_SIMSWAP1 AMD_FCH_GPIO_REG_GPIO68
    +#define APU5_GPIO_REG_SIMSWAP2 AMD_FCH_GPIO_REG_GPIO32_GE1
    +#define APU5_GPIO_REG_SIMSWAP3 AMD_FCH_GPIO_REG_GPIO33_GE2
    +#define APU5_GPIO_REG_RESETM1 AMD_FCH_GPIO_REG_GPIO51
    +#define APU5_GPIO_REG_RESETM2 AMD_FCH_GPIO_REG_GPIO55_DEVSLP0
    +#define APU5_GPIO_REG_RESETM3 AMD_FCH_GPIO_REG_GPIO64

    /* Order in which the GPIO lines are defined in the register list */
    #define APU2_GPIO_LINE_LED1 0
    #define APU2_GPIO_LINE_LED2 1
    #define APU2_GPIO_LINE_LED3 2
    #define APU2_GPIO_LINE_MODESW 3
    -#define APU2_GPIO_LINE_SIMSWAP 4
    -#define APU2_GPIO_LINE_MPCIE2 5
    -#define APU2_GPIO_LINE_MPCIE3 6
    +#define APU2_GPIO_LINE_RESETM1 4
    +#define APU2_GPIO_LINE_RESETM2 5
    +#define APU2_GPIO_LINE_SIMSWAP 6

    -/* GPIO device */
    +#define APU5_GPIO_LINE_LED1 0
    +#define APU5_GPIO_LINE_LED2 1
    +#define APU5_GPIO_LINE_LED3 2
    +#define APU5_GPIO_LINE_MODESW 3
    +#define APU5_GPIO_LINE_RESETM1 4
    +#define APU5_GPIO_LINE_RESETM2 5
    +#define APU5_GPIO_LINE_RESETM3 6
    +#define APU5_GPIO_LINE_SIMSWAP1 7
    +#define APU5_GPIO_LINE_SIMSWAP2 8
    +#define APU5_GPIO_LINE_SIMSWAP3 9
    +
    +
    +/* GPIO device - APU2/3/4 */

    static int apu2_gpio_regs[] = {
    [APU2_GPIO_LINE_LED1] = APU2_GPIO_REG_LED1,
    [APU2_GPIO_LINE_LED2] = APU2_GPIO_REG_LED2,
    [APU2_GPIO_LINE_LED3] = APU2_GPIO_REG_LED3,
    [APU2_GPIO_LINE_MODESW] = APU2_GPIO_REG_MODESW,
    + [APU2_GPIO_LINE_RESETM1] = APU2_GPIO_REG_RESETM1,
    + [APU2_GPIO_LINE_RESETM2] = APU2_GPIO_REG_RESETM2,
    [APU2_GPIO_LINE_SIMSWAP] = APU2_GPIO_REG_SIMSWAP,
    - [APU2_GPIO_LINE_MPCIE2] = APU2_GPIO_REG_MPCIE2,
    - [APU2_GPIO_LINE_MPCIE3] = APU2_GPIO_REG_MPCIE3,
    };

    static const char * const apu2_gpio_names[] = {
    @@ -61,9 +96,9 @@ static const char * const apu2_gpio_names[] = {
    [APU2_GPIO_LINE_LED2] = "front-led2",
    [APU2_GPIO_LINE_LED3] = "front-led3",
    [APU2_GPIO_LINE_MODESW] = "front-button",
    + [APU2_GPIO_LINE_RESETM1] = "modem1-reset",
    + [APU2_GPIO_LINE_RESETM2] = "modem2-reset",
    [APU2_GPIO_LINE_SIMSWAP] = "simswap",
    - [APU2_GPIO_LINE_MPCIE2] = "mpcie2_reset",
    - [APU2_GPIO_LINE_MPCIE3] = "mpcie3_reset",
    };

    static const struct amd_fch_gpio_pdata board_apu2 = {
    @@ -72,8 +107,55 @@ static const struct amd_fch_gpio_pdata board_apu2 = {
    .gpio_names = apu2_gpio_names,
    };

    +/* GPIO device - APU5 */
    +
    +static int apu5_gpio_regs[] = {
    + [APU5_GPIO_LINE_LED1] = APU2_GPIO_REG_LED1,
    + [APU5_GPIO_LINE_LED2] = APU2_GPIO_REG_LED2,
    + [APU5_GPIO_LINE_LED3] = APU2_GPIO_REG_LED3,
    + [APU5_GPIO_LINE_MODESW] = APU5_GPIO_REG_MODESW,
    + [APU5_GPIO_LINE_RESETM1] = APU5_GPIO_REG_RESETM1,
    + [APU5_GPIO_LINE_RESETM2] = APU5_GPIO_REG_RESETM2,
    + [APU5_GPIO_LINE_RESETM3] = APU5_GPIO_REG_RESETM3,
    + [APU5_GPIO_LINE_SIMSWAP1] = APU5_GPIO_REG_SIMSWAP1,
    + [APU5_GPIO_LINE_SIMSWAP2] = APU5_GPIO_REG_SIMSWAP2,
    + [APU5_GPIO_LINE_SIMSWAP3] = APU5_GPIO_REG_SIMSWAP3,
    +};
    +
    +static const char * const apu5_gpio_names[] = {
    + [APU5_GPIO_LINE_LED1] = "front-led1",
    + [APU5_GPIO_LINE_LED2] = "front-led2",
    + [APU5_GPIO_LINE_LED3] = "front-led3",
    + [APU5_GPIO_LINE_MODESW] = "front-button",
    + [APU5_GPIO_LINE_RESETM1] = "modem1-reset",
    + [APU5_GPIO_LINE_RESETM2] = "modem2-reset",
    + [APU5_GPIO_LINE_RESETM3] = "modem3-reset",
    + [APU5_GPIO_LINE_SIMSWAP1] = "simswap1",
    + [APU5_GPIO_LINE_SIMSWAP2] = "simswap2",
    + [APU5_GPIO_LINE_SIMSWAP3] = "simswap3",
    +};
    +
    +static const struct amd_fch_gpio_pdata board_apu5 = {
    + .gpio_num = ARRAY_SIZE(apu5_gpio_regs),
    + .gpio_reg = apu5_gpio_regs,
    + .gpio_names = apu5_gpio_names,
    +};
    +
    +
    /* Board setup */

    +struct apu_driver_data {
    + const struct amd_fch_gpio_pdata *board_data;
    +};
    +
    +static struct apu_driver_data apu2_driver_data = {
    + .board_data = &board_apu2,
    +};
    +
    +static struct apu_driver_data apu5_driver_data = {
    + .board_data = &board_apu5,
    +};
    +
    /* Note: matching works on string prefix, so "apu2" must come before "apu" */
    static const struct dmi_system_id apu_gpio_dmi_table[] __initconst = {

    @@ -84,25 +166,25 @@ static const struct dmi_system_id apu_gpio_dmi_table[] __initconst = {
    DMI_MATCH(DMI_SYS_VENDOR, "PC Engines"),
    DMI_MATCH(DMI_BOARD_NAME, "APU2")
    },
    - .driver_data = (void *)&board_apu2,
    + .driver_data = (void *)&apu2_driver_data,
    },
    - /* APU2 w/ legacy BIOS >= 4.0.8 */
    + /* APU2 w/ mainline BIOS */
    {
    .ident = "apu2",
    .matches = {
    DMI_MATCH(DMI_SYS_VENDOR, "PC Engines"),
    DMI_MATCH(DMI_BOARD_NAME, "apu2")
    },
    - .driver_data = (void *)&board_apu2,
    + .driver_data = (void *)&apu2_driver_data,
    },
    - /* APU2 w/ mainline BIOS */
    + /* APU2 w/ legacy BIOS >= 4.0.8 */
    {
    .ident = "apu2",
    .matches = {
    DMI_MATCH(DMI_SYS_VENDOR, "PC Engines"),
    DMI_MATCH(DMI_BOARD_NAME, "PC Engines apu2")
    },
    - .driver_data = (void *)&board_apu2,
    + .driver_data = (void *)&apu2_driver_data,
    },

    /* APU3 w/ legacy BIOS < 4.0.8 */
    @@ -112,52 +194,61 @@ static const struct dmi_system_id apu_gpio_dmi_table[] __initconst = {
    DMI_MATCH(DMI_SYS_VENDOR, "PC Engines"),
    DMI_MATCH(DMI_BOARD_NAME, "APU3")
    },
    - .driver_data = (void *)&board_apu2,
    + .driver_data = (void *)&apu2_driver_data,
    },
    - /* APU3 w/ legacy BIOS >= 4.0.8 */
    + /* APU3 w/ mainline BIOS */
    {
    - .ident = "apu3",
    - .matches = {
    + .ident = "apu3",
    + .matches = {
    DMI_MATCH(DMI_SYS_VENDOR, "PC Engines"),
    DMI_MATCH(DMI_BOARD_NAME, "apu3")
    },
    - .driver_data = (void *)&board_apu2,
    + .driver_data = (void *)&apu2_driver_data,
    },
    - /* APU3 w/ mainline BIOS */
    + /* APU3 w/ legacy BIOS >= 4.0.8 */
    {
    - .ident = "apu3",
    - .matches = {
    + .ident = "apu3",
    + .matches = {
    DMI_MATCH(DMI_SYS_VENDOR, "PC Engines"),
    DMI_MATCH(DMI_BOARD_NAME, "PC Engines apu3")
    },
    - .driver_data = (void *)&board_apu2,
    + .driver_data = (void *)&apu2_driver_data,
    },
    /* APU4 w/ legacy BIOS < 4.0.8 */
    {
    - .ident = "apu4",
    - .matches = {
    + .ident = "apu4",
    + .matches = {
    DMI_MATCH(DMI_SYS_VENDOR, "PC Engines"),
    DMI_MATCH(DMI_BOARD_NAME, "APU4")
    },
    - .driver_data = (void *)&board_apu2,
    + .driver_data = (void *)&apu2_driver_data,
    },
    - /* APU4 w/ legacy BIOS >= 4.0.8 */
    + /* APU4 w/ mainline BIOS */
    {
    - .ident = "apu4",
    - .matches = {
    + .ident = "apu4",
    + .matches = {
    DMI_MATCH(DMI_SYS_VENDOR, "PC Engines"),
    DMI_MATCH(DMI_BOARD_NAME, "apu4")
    },
    - .driver_data = (void *)&board_apu2,
    + .driver_data = (void *)&apu2_driver_data,
    },
    - /* APU4 w/ mainline BIOS */
    + /* APU4 w/ legacy BIOS >= 4.0.8 */
    {
    - .ident = "apu4",
    - .matches = {
    + .ident = "apu4",
    + .matches = {
    DMI_MATCH(DMI_SYS_VENDOR, "PC Engines"),
    DMI_MATCH(DMI_BOARD_NAME, "PC Engines apu4")
    },
    - .driver_data = (void *)&board_apu2,
    + .driver_data = (void *)&apu2_driver_data,
    + },
    + /* APU5 w/ mainline BIOS */
    + {
    + .ident = "apu5",
    + .matches = {
    + DMI_MATCH(DMI_SYS_VENDOR, "PC Engines"),
    + DMI_MATCH(DMI_BOARD_NAME, "apu5")
    + },
    + .driver_data = (void *)&apu5_driver_data,
    },
    {}
    };
    @@ -188,16 +279,22 @@ static struct platform_device * __init apu_create_pdev(
    static int __init apu_board_init(void)
    {
    const struct dmi_system_id *id;
    + const struct apu_driver_data *driver_data;

    id = dmi_first_match(apu_gpio_dmi_table);
    if (!id) {
    pr_err("No APU board detected via DMI\n");
    return -ENODEV;
    + } else {
    + pr_debug("Detected APU board: %s\n",
    + dmi_get_system_info(DMI_BOARD_NAME));
    }

    + driver_data = id->driver_data;
    +
    apu_gpio_pdev = apu_create_pdev(
    AMD_FCH_GPIO_DRIVER_NAME,
    - id->driver_data,
    + driver_data->board_data,
    sizeof(struct amd_fch_gpio_pdata));

    return 0;
    @@ -212,7 +309,7 @@ module_init(apu_board_init);
    module_exit(apu_board_exit);

    MODULE_AUTHOR("Enrico Weigelt, metux IT consult <info@metux.net>");
    -MODULE_DESCRIPTION("PC Engines APUv2/APUv3 board GPIO/LEDs/keys driver");
    +MODULE_DESCRIPTION("PC Engines APUv2-5 board GPIO/LEDs/keys driver");
    MODULE_LICENSE("GPL");
    MODULE_DEVICE_TABLE(dmi, apu_gpio_dmi_table);
    MODULE_ALIAS("platform:pcengines-apuv2");
    --
    2.26.2
    \
     
     \ /
      Last update: 2020-09-22 00:00    [W:3.341 / U:23.904 seconds]
    ©2003-2020 Jasper Spaans|hosted at Digital Ocean and TransIP|Read the blog|Advertise on this site