Received-SPF: pass (google.com: domain of linux-kernel-owner@vger.kernel.org designates 2620:137:e000::1:20 as permitted sender) client-ip=2620:137:e000::1:20;
Date:   Thu, 13 Jul 2023 16:54:44 +0200
From:   Henning Schild <henning.schild@siemens.com>
To:     Hans de Goede <hdegoede@redhat.com>,
        Mark Gross <markgross@kernel.org>,
        <platform-driver-x86@vger.kernel.org>,
        <linux-kernel@vger.kernel.org>
Cc:     Tobias Schaffner <tobias.schaffner@siemens.com>,
        Gerd Haeussler <gerd.haeussler.ext@siemens.com>
Subject: Re: [PATCH v2 0/3] platform/x86: simatic-ipc: add another model and
 hwmon module loading
Message-ID: <20230713165444.1ee67557@md1za8fc.ad001.siemens.net>
In-Reply-To: <20230713144832.26473-1-henning.schild@siemens.com>
References: <20230713144832.26473-1-henning.schild@siemens.com>
Content-Type: text/plain; charset=US-ASCII
Content-Transfer-Encoding: 7bit
MIME-Version: 1.0
X-MS-Exchange-AntiSpam-MessageData-ChunkCount: 1
X-MS-Exchange-AntiSpam-MessageData-0: =?us-ascii?Q?Stm/eDk9XuGf1cALQRPpkg2aQBQsIeIj59p3IaKgJApQbWYY+/fRoU049Ebm?=
 =?us-ascii?Q?ryOBhGRE0gRiWMkGEZle5SJuF27TmeltGDGBQd4naSWUxnqUB+YYEX5tWxeL?=
 =?us-ascii?Q?8cMv4zCUlVTnKJg9lAzl0Neb2GYx5irhXtd0VEI2/U6Af/mDT+fz0X62qh+w?=
 =?us-ascii?Q?dmQxQjeeadnfywTStWQMAv4972n4LFg2BeZkTbCftBDHhpxzejvCEqDsC9jf?=
 =?us-ascii?Q?EWGjDNBhBRRxmNAIc8hXht+CBud0w9Ypj1l2htkmz4PR0iE8HJ+AR7uhKkBK?=
 =?us-ascii?Q?XI1FH15np+hUxzglWzkiirRmGj94vOPMS09i4yE/Xqss3R/FUIfYTRSjfUAl?=
 =?us-ascii?Q?eTZ2RTnuFCZEoHhbC/GZbgNYaruAPC/8dR8wJyYpLg6QEyz/Mk9Gn6zE7oEr?=
 =?us-ascii?Q?mP8s8cbD/50cE8zSTP6JCJrBlCgGaIm4sBCL9b7I7Ofv6IXemkfM4Utf+wsO?=
 =?us-ascii?Q?zfxZhYt1i2hF5AvGCG7j0CNmjL+QUj5LqGXspfBJeJoNxXWhUt4iZEbMw3M2?=
 =?us-ascii?Q?zQwMEkm0qGJzolUtn22j4b1pql4IiJo/JeBCzF1Zpn84+joxkjiVdVWVE2wa?=
 =?us-ascii?Q?HDMKItN5CHkFz4bn/yQhMgCGZZF8vmu+Mm0P18tXIuoTdAeEKpJeXAEF8uNZ?=
 =?us-ascii?Q?mP2Lulf1aIRedTB03jrTXh6RARaCokeYir4GNnfup5tG53v2OUqTeuMevAtN?=
 =?us-ascii?Q?tIM7JsFLCM2FUsTyw41/eR1hcjJwM3cS5E0utpsqc4Z+JX4novvAuH7KbsHk?=
 =?us-ascii?Q?ILNC185k8t90GwU+GNu93eYuWu6vBlS1Pj4KunCYkJXR/zbjrE+dGAhVZXdk?=
 =?us-ascii?Q?dkS5XF/CHF4KSpHMbI40HyNciZlsvz3odk4z7RO42bQPE/IpOtRua6XEFnYd?=
 =?us-ascii?Q?LzcFfXH0dfyQyDwp7TSTi+Rb1lvjrb7kBRypoBLLs1bI+KR2xgMANfJb2OIr?=
 =?us-ascii?Q?TnZa5J1x0A2uYsSUywOL97+eychk+6cQORHgUA4/hjsDZs5+09xzN/y6uKNE?=
 =?us-ascii?Q?ynrX4KbvvuOV0uxF9vx8MlOwHFKThcfcUphth/+RFOWIicWQcGckKdUYLJ6Y?=
 =?us-ascii?Q?dHnGMMJk8C8zvRcBYHg4Sg5zXsERK1PkC3AkLsaYMX/sj86K0A3uHySSa8+P?=
 =?us-ascii?Q?adGF6puHvIWrx8n/Ghc88KuIyNHF4jytrqQ7PQEc1ML7oY6tBd76IHGraQ9c?=
 =?us-ascii?Q?Ba6+rOoi1Ne2UdvMDIIz2Liu48Wx52jH+mdLKMJ0Gx0eHk6waQoziUnD1WQq?=
 =?us-ascii?Q?8LTg0x3wIwz1Ozhn1YGhtTDsn9XzCi1P/elTVrJjlU5oyD7y/ByxgmqfFan4?=
 =?us-ascii?Q?j1z7FSQ5SI77yGv82RsJuL+gcCMnBCXAodBHl/ch4ZId55rXNdY9a5RaEzeS?=
 =?us-ascii?Q?fW6wqs2wxbKv4y/GbMwkpKI77L/AqQRGWcQoJ5zZBstdqI+3NoauHKchcgJM?=
 =?us-ascii?Q?nH9lURl5C2W0SpOehSuRt0x8VZK5YloSZZqiOniSwmxX47bbfwwk52hkXN8/?=
 =?us-ascii?Q?A+C+wqub3g6dR155FLS8t5N/8Sot8t4g/FJUIGf1KWIjZO+HMQzYsF5dnGfO?=
 =?us-ascii?Q?9Gzmmwj2fpOBLlcP3lhj79YiMWot5VcvfXlOFxPjSL0v+vcsTkwyyCC+v+Qx?=
 =?us-ascii?Q?AAkn0ofDWx9lzBIrxVFW8H0sWqV7KLaV0bz1dOceAxNtlUHiUNOzb/wP3gOY?=
 =?us-ascii?Q?uYXSXA=3D=3D?=
X-OriginatorOrg: siemens.com
X-MS-Exchange-CrossTenant-Network-Message-Id: cef8043f-9657-4443-0f6b-08db83b11e67
X-MS-Exchange-CrossTenant-AuthSource: PA4PR10MB5780.EURPRD10.PROD.OUTLOOK.COM
X-MS-Exchange-CrossTenant-AuthAs: Internal
X-MS-Exchange-CrossTenant-OriginalArrivalTime: 13 Jul 2023 14:54:54.8090
 (UTC)
X-MS-Exchange-CrossTenant-FromEntityHeader: Hosted
X-MS-Exchange-CrossTenant-Id: 38ae3bcd-9579-4fd4-adda-b42e1495d55a
X-MS-Exchange-CrossTenant-MailboxType: HOSTED
X-MS-Exchange-CrossTenant-UserPrincipalName: wtyQm0beObV4QwXB7CHhwbrr+ryDN7To4yOlNsF32UFnISZosm3ZelPsXV6+YCz60lIurMiLrkjr9C5p4Id32a3ChKduvlJ8xcbpcaC6MEs=
X-MS-Exchange-Transport-CrossTenantHeadersStamped: PAWPR10MB7125
X-Spam-Status: No, score=-1.1 required=5.0 tests=BAYES_00,DKIMWL_WL_MED,
        DKIM_SIGNED,DKIM_VALID,DKIM_VALID_AU,DKIM_VALID_EF,FORGED_SPF_HELO,
        RCVD_IN_DNSWL_NONE,RCVD_IN_MSPIKE_H2,SPF_HELO_PASS,SPF_NONE,
        T_SCC_BODY_TEXT_LINE autolearn=no autolearn_force=no version=3.4.6
X-Spam-Checker-Version: SpamAssassin 3.4.6 (2021-04-09) on
        lindbergh.monkeyblade.net
Precedence: bulk
List-ID: <linux-kernel.vger.kernel.org>
X-Mailing-List: linux-kernel@vger.kernel.org

Am Thu, 13 Jul 2023 16:48:29 +0200
schrieb Henning Schild <henning.schild@siemens.com>:

> cahnged since v1:
>  - switch to using a list of modules per device
>  - add p3 and make the watchdog module load code use what p2 created
> 
> The first patch just adds a device that is pretty similar to another
> one we already had here.
> 
> The second patch loads modules for hwmon support, should they be
> available. That will save users the need to detect and manually load
> those modules after a machine has been clearly identified by its
> Siemens Simatic IPC station id.
> 
> And finally p3 changes another request_module call to use the
> mechanism introduced in p2.

Note that this is based on

 [PATCH 0/2] platform/x86: add CMOS battery monitoring for simatic IPCs

which is based on

 [PATCH v2 1/2] platform/x86: simatic-ipc: add another model BX-21A

Henning

> Henning Schild (3):
>   platform/x86: simatic-ipc: add another model
>   platform/x86: simatic-ipc: add auto-loading of hwmon modules
>   platform/x86: simatic-ipc: use extra module loading for watchdog
> 
>  drivers/platform/x86/simatic-ipc.c            | 74
> ++++++++++++++----- include/linux/platform_data/x86/simatic-ipc.h |
> 1 + 2 files changed, 57 insertions(+), 18 deletions(-)
>