| Message ID | 20210302163309.25528-1-henning.schild@siemens.com (mailing list archive) |
|---|---|
| Headers | show |
| Series | add device drivers for Siemens Industrial PCs | expand |
On Thu, Mar 4, 2021 at 9:29 AM Henning Schild <henning.schild@siemens.com> wrote: > This series adds support for watchdogs and leds of several x86 devices > from Siemens. > > It is structured with a platform driver that mainly does identification > of the machines. It might trigger loading of the actual device drivers > by attaching devices to the platform bus. > > The identification is vendor specific, parsing a special binary DMI > entry. The implementation of that platform identification is applied on > pmc_atom clock quirks in the final patch. > > It is all structured in a way that we can easily add more devices and > more platform drivers later. Internally we have some more code for > hardware monitoring, more leds, watchdogs etc. This will follow some > day. > > But the idea here is to share early, and hopefully not fail early. I have given a few comments here and there, so please check the entire series and address them in _all_ similar locations. As I have noticed, I have different approach about P2SB code, I have to give the series a dust and see if you can utilize it. I would like to be Cc'ed on the next version.
On Thu, Mar 4, 2021 at 12:19 PM Andy Shevchenko <andy.shevchenko@gmail.com> wrote: > On Thu, Mar 4, 2021 at 9:29 AM Henning Schild > <henning.schild@siemens.com> wrote: > I have given a few comments here and there, so please check the entire > series and address them in _all_ similar locations. As I have noticed, > I have different approach about P2SB code, I have to give the series a > dust and see if you can utilize it. > > I would like to be Cc'ed on the next version. One more thing, is it Apollo Lake based?
Thanks Andy, Am Thu, 4 Mar 2021 12:19:44 +0200 schrieb Andy Shevchenko <andy.shevchenko@gmail.com>: > On Thu, Mar 4, 2021 at 9:29 AM Henning Schild > <henning.schild@siemens.com> wrote: > > > This series adds support for watchdogs and leds of several x86 > > devices from Siemens. > > > > It is structured with a platform driver that mainly does > > identification of the machines. It might trigger loading of the > > actual device drivers by attaching devices to the platform bus. > > > > The identification is vendor specific, parsing a special binary DMI > > entry. The implementation of that platform identification is > > applied on pmc_atom clock quirks in the final patch. > > > > It is all structured in a way that we can easily add more devices > > and more platform drivers later. Internally we have some more code > > for hardware monitoring, more leds, watchdogs etc. This will follow > > some day. > > > > But the idea here is to share early, and hopefully not fail early. > > I have given a few comments here and there, so please check the entire > series and address them in _all_ similar locations. As I have noticed, > I have different approach about P2SB code, I have to give the series a > dust and see if you can utilize it. You did find some things that others found as well. SPDX vs blabla, header ordering, some other style. Some things are already done and will be in v2. Other findings are new, and we will look into them. The only thing that did stick out is P2SB, also was a point in internal pre-review. Let us see what you have, i can include a patch of yours into the q. But i am kind of afraid once it is there, several existing users should be touched with it, and this series would come later. Or this series comes first and you come later and clean up our "mess". Not sure i would want to take over all P2SB unhiders, but with you on board it will work. > I would like to be Cc'ed on the next version. Sure thing. regards, Henning >
Am Thu, 4 Mar 2021 12:20:22 +0200 schrieb Andy Shevchenko <andy.shevchenko@gmail.com>: > On Thu, Mar 4, 2021 at 12:19 PM Andy Shevchenko > <andy.shevchenko@gmail.com> wrote: > > On Thu, Mar 4, 2021 at 9:29 AM Henning Schild > > <henning.schild@siemens.com> wrote: > > > I have given a few comments here and there, so please check the > > entire series and address them in _all_ similar locations. As I > > have noticed, I have different approach about P2SB code, I have to > > give the series a dust and see if you can utilize it. > > > > I would like to be Cc'ed on the next version. > > One more thing, is it Apollo Lake based? Not sure i can answer that, or what you even refer to. The whole series is about a range of devices, some even have sub-models with differing CPUs and Lakes regards, Henning
From: Henning Schild <henning.schild@siemens.com> This series adds support for watchdogs and leds of several x86 devices from Siemens. It is structured with a platform driver that mainly does identification of the machines. It might trigger loading of the actual device drivers by attaching devices to the platform bus. The identification is vendor specific, parsing a special binary DMI entry. The implementation of that platform identification is applied on pmc_atom clock quirks in the final patch. It is all structured in a way that we can easily add more devices and more platform drivers later. Internally we have some more code for hardware monitoring, more leds, watchdogs etc. This will follow some day. But the idea here is to share early, and hopefully not fail early. Henning Schild (4): platform/x86: simatic-ipc: add main driver for Siemens devices leds: simatic-ipc-leds: add new driver for Siemens Industial PCs watchdog: simatic-ipc-wdt: add new driver for Siemens Industrial PCs platform/x86: pmc_atom: improve critclk_systems matching for Siemens PCs drivers/leds/Kconfig | 11 + drivers/leds/Makefile | 1 + drivers/leds/simatic-ipc-leds.c | 224 +++++++++++++ drivers/platform/x86/Kconfig | 9 + drivers/platform/x86/Makefile | 3 + drivers/platform/x86/pmc_atom.c | 39 +-- drivers/platform/x86/simatic-ipc.c | 166 ++++++++++ drivers/watchdog/Kconfig | 11 + drivers/watchdog/Makefile | 1 + drivers/watchdog/simatic-ipc-wdt.c | 305 ++++++++++++++++++ .../platform_data/x86/simatic-ipc-base.h | 33 ++ include/linux/platform_data/x86/simatic-ipc.h | 68 ++++ 12 files changed, 853 insertions(+), 18 deletions(-) create mode 100644 drivers/leds/simatic-ipc-leds.c create mode 100644 drivers/platform/x86/simatic-ipc.c create mode 100644 drivers/watchdog/simatic-ipc-wdt.c create mode 100644 include/linux/platform_data/x86/simatic-ipc-base.h create mode 100644 include/linux/platform_data/x86/simatic-ipc.h