From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: X-Spam-Checker-Version: SpamAssassin 3.4.0 (2014-02-07) on aws-us-west-2-korg-lkml-1.web.codeaurora.org X-Spam-Level: X-Spam-Status: No, score=-5.3 required=3.0 tests=BAYES_00, HEADER_FROM_DIFFERENT_DOMAINS,MAILING_LIST_MULTI,SPF_HELO_NONE,SPF_PASS, USER_AGENT_SANE_2 autolearn=no autolearn_force=no version=3.4.0 Received: from mail.kernel.org (mail.kernel.org [198.145.29.99]) by smtp.lore.kernel.org (Postfix) with ESMTP id 4B36CC433ED for ; Tue, 6 Apr 2021 14:57:08 +0000 (UTC) Received: from vger.kernel.org (vger.kernel.org [23.128.96.18]) by mail.kernel.org (Postfix) with ESMTP id 0E56A613C6 for ; Tue, 6 Apr 2021 14:57:08 +0000 (UTC) Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S233301AbhDFO5P (ORCPT ); Tue, 6 Apr 2021 10:57:15 -0400 Received: from lizzard.sbs.de ([194.138.37.39]:58764 "EHLO lizzard.sbs.de" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S233155AbhDFO5O (ORCPT ); Tue, 6 Apr 2021 10:57:14 -0400 Received: from mail2.sbs.de (mail2.sbs.de [192.129.41.66]) by lizzard.sbs.de (8.15.2/8.15.2) with ESMTPS id 136EuRp2022676 (version=TLSv1.2 cipher=DHE-RSA-AES256-GCM-SHA384 bits=256 verify=OK); Tue, 6 Apr 2021 16:56:27 +0200 Received: from md1za8fc.ad001.siemens.net ([167.87.42.66]) by mail2.sbs.de (8.15.2/8.15.2) with ESMTP id 136EqmdF003812; Tue, 6 Apr 2021 16:52:48 +0200 Date: Tue, 6 Apr 2021 16:52:47 +0200 From: Henning Schild To: "Enrico Weigelt, metux IT consult" Cc: , , , , Srikanth Krishnakar , Jan Kiszka , Gerd Haeussler , Guenter Roeck , Wim Van Sebroeck , Mark Gross , Hans de Goede , Pavel Machek , Andy Shevchenko Subject: Re: [PATCH v3 3/4] watchdog: simatic-ipc-wdt: add new driver for Siemens Industrial PCs Message-ID: <20210406165247.78791bf7@md1za8fc.ad001.siemens.net> In-Reply-To: References: <20210329174928.18816-1-henning.schild@siemens.com> <20210329174928.18816-4-henning.schild@siemens.com> X-Mailer: Claws Mail 3.17.8 (GTK+ 2.24.32; x86_64-pc-linux-gnu) MIME-Version: 1.0 Content-Type: text/plain; charset=US-ASCII Content-Transfer-Encoding: 7bit Precedence: bulk List-ID: X-Mailing-List: linux-leds@vger.kernel.org Am Thu, 1 Apr 2021 18:15:41 +0200 schrieb "Enrico Weigelt, metux IT consult" : > On 29.03.21 19:49, Henning Schild wrote: > > Hi, > > > This driver adds initial support for several devices from Siemens. > > It is based on a platform driver introduced in an earlier commit. > > Where does the wdt actually come from ? > > Is it in the SoC ? (which SoC exactly). SoC-builtin wdt is a pretty > usual case. > > Or some external chip ? I guess external chip, but again we are talking about multiple machines. And the manuals i read so far do not go into that sort of detail. In fact on some of the machines you will have two watchdogs, one from the SoC and that "special" one. That has several reasons, probably not too important here. The HW guys are adding another wd not just for fun, and it would be nice to have a driver. > The code smells a bit like two entirely different wdt's that just have > some similarities. If that's the case, I'd rather split it into two > separate drivers and let the parent driver (board file) instantiate > the correct one. Yes, it is two. Just like for the LEDs. One version PIO-based another version gpio/p2sb/mmio based. In fact the latter should very likely be based on that gpio pinctl, whether it really needs to be a separate driver will have to be seen. There are probably pros and cons for both options. regards, Henning > > --mtx >