2022-12-22 11:43:36

by Henning Schild

Subject: [PATCH 0/2] simatic-ipc: enable one more board

This series is rather trivial. It renames a few variables to use the
product names as known by end users and enables the current driver
to pick up on a board they also work for.

Henning Schild (2):
platform/x86: simatic-ipc: correct name of a model
platform/x86: simatic-ipc: add another model

drivers/platform/x86/simatic-ipc.c | 3 ++-
include/linux/platform_data/x86/simatic-ipc.h | 3 ++-
2 files changed, 4 insertions(+), 2 deletions(-)

--
2.38.2

2022-12-22 12:18:25

by Henning Schild

Subject: [PATCH 2/2] platform/x86: simatic-ipc: add another model

Add IPC PX-39A support.

Signed-off-by: Henning Schild <[email protected]>
---
drivers/platform/x86/simatic-ipc.c | 1 +
include/linux/platform_data/x86/simatic-ipc.h | 1 +
2 files changed, 2 insertions(+)

diff --git a/drivers/platform/x86/simatic-ipc.c b/drivers/platform/x86/simatic-ipc.c
index 2ab1f8da32b0..b3622419cd1a 100644
--- a/drivers/platform/x86/simatic-ipc.c
+++ b/drivers/platform/x86/simatic-ipc.c
@@ -47,6 +47,7 @@ static struct {
{SIMATIC_IPC_IPC427E, SIMATIC_IPC_DEVICE_427E, SIMATIC_IPC_DEVICE_427E},
{SIMATIC_IPC_IPC477E, SIMATIC_IPC_DEVICE_NONE, SIMATIC_IPC_DEVICE_427E},
{SIMATIC_IPC_IPCBX_39A, SIMATIC_IPC_DEVICE_227G, SIMATIC_IPC_DEVICE_227G},
+ {SIMATIC_IPC_IPCPX_39A, SIMATIC_IPC_DEVICE_NONE, SIMATIC_IPC_DEVICE_227G},
};

static int register_platform_devices(u32 station_id)
diff --git a/include/linux/platform_data/x86/simatic-ipc.h b/include/linux/platform_data/x86/simatic-ipc.h
index a4a6cba412cb..a48bb5240977 100644
--- a/include/linux/platform_data/x86/simatic-ipc.h
+++ b/include/linux/platform_data/x86/simatic-ipc.h
@@ -33,6 +33,7 @@ enum simatic_ipc_station_ids {
SIMATIC_IPC_IPC127E = 0x00000D01,
SIMATIC_IPC_IPC227G = 0x00000F01,
SIMATIC_IPC_IPCBX_39A = 0x00001001,
+ SIMATIC_IPC_IPCPX_39A = 0x00001002,
};

static inline u32 simatic_ipc_get_station_id(u8 *data, int max_len)
--
2.38.2

2023-01-12 19:28:37

by Hans de Goede

Subject: Re: [PATCH 0/2] simatic-ipc: enable one more board

Hi,

On 12/22/22 11:37, Henning Schild wrote:
> This series is rather trivial. It renames a few variables to use the
> product names as known by end users and enables the current driver
> to pick up on a board they also work for.

Thank you for your patch series, I've applied this series
to my fixes branch:
https://git.kernel.org/pub/scm/linux/kernel/git/pdx86/platform-drivers-x86.git/log/?h=fixes

Note it will show up in my fixes branch once I've pushed my
local branch there, which might take a while.

I will include this series in my next fixes pull-req to Linus
for the current kernel development cycle.

Regards,

Hans


>
> Henning Schild (2):
> platform/x86: simatic-ipc: correct name of a model
> platform/x86: simatic-ipc: add another model
>
> drivers/platform/x86/simatic-ipc.c | 3 ++-
> include/linux/platform_data/x86/simatic-ipc.h | 3 ++-
> 2 files changed, 4 insertions(+), 2 deletions(-)
>