linux-gpio.vger.kernel.org archive mirror
 help / color / mirror / Atom feed
From: Thomas Richard <thomas.richard@bootlin.com>
To: Lee Jones <lee@kernel.org>,
	Linus Walleij <linus.walleij@linaro.org>,
	 Bartosz Golaszewski <brgl@bgdev.pl>,
	Andi Shyti <andi.shyti@kernel.org>,
	 Wim Van Sebroeck <wim@linux-watchdog.org>,
	 Guenter Roeck <linux@roeck-us.net>
Cc: linux-kernel@vger.kernel.org, linux-gpio@vger.kernel.org,
	 linux-i2c@vger.kernel.org, linux-watchdog@vger.kernel.org,
	 thomas.petazzoni@bootlin.com, blake.vermeer@keysight.com,
	 Thomas Richard <thomas.richard@bootlin.com>
Subject: [PATCH v2 0/5] Congatec Board Controller drivers
Date: Tue, 17 Sep 2024 19:00:46 +0200	[thread overview]
Message-ID: <20240503-congatec-board-controller-v2-0-681511a01c8f@bootlin.com> (raw)

This is a new iteration of the Congatec Board Controller series.

For the MFD driver, the main change is the use of
platform_device_register_simple() which makes the code smaller.

For the GPIO driver the main change is the use of guard() and scope_gard().

For the I2C bus driver, the I2C_FUNC_SMBUS_QUICK flag was unset as the
controller doesn't support smbus quick accesses.

For the Watchdog driver, min_timeout and max_timeout are configured now,
and watchdog_init_timeout() is used to initialize the timeout. 

More details in the changelog.

Best Regards,

Thomas

Signed-off-by: Thomas Richard <thomas.richard@bootlin.com>
---
Changes in v2:
- mfd: use platform_device_register_simple(), so struct cgbc_platform_data
  and cgbc_create_platform_device() were removed.
- mfd: rename cgbc_detect_device() to cgbc_wait_device().
- mfd: remove the useless abstracted function cgbc_command().
- mfd: set the release session message as warning instead of error.
- mfd: minor fixes (sort includes, add comments, fix some alignments and
  nit ...).
- gpio: use scoped_guard() and guard().
- gpio: use devm_mutex_init().
- i2c: unset the I2C_FUNC_SMBUS_QUICK flag as smbus quick accesses are not
  supported by the controller.
- i2c: rephrase comment for read_maxtime_us.
- i2c: set the invalid frequency message as an info (previously a warning).
- i2c: other minor fixes (sort includes, fix name of i2c_state enum).
- watchdog: add missing includes.
- watchdog: remove warning for the pretimeout.
- watchdog: remove timeout_action and pretimeout_action (ACTION_RESET and
  ACTION_SMI are directly used in cgbc_wdt_start()).
- watchdog: set max_timeout and min_timeout.
- watchdog: use watchdog_init_timeout() to set the timeout.
- Link to v1: https://lore.kernel.org/r/20240503-congatec-board-controller-v1-0-fec5236270e7@bootlin.com

---
Thomas Richard (5):
      mfd: add Congatec Board Controller mfd driver
      gpio: Congatec Board Controller gpio driver
      i2c: Congatec Board Controller i2c bus driver
      watchdog: Congatec Board Controller watchdog timer driver
      MAINTAINERS: Add entry for Congatec Board Controller

 MAINTAINERS                   |   9 +
 drivers/gpio/Kconfig          |  10 +
 drivers/gpio/Makefile         |   1 +
 drivers/gpio/gpio-cgbc.c      | 196 ++++++++++++++++++++
 drivers/i2c/busses/Kconfig    |  10 +
 drivers/i2c/busses/Makefile   |   1 +
 drivers/i2c/busses/i2c-cgbc.c | 406 +++++++++++++++++++++++++++++++++++++++++
 drivers/mfd/Kconfig           |  12 ++
 drivers/mfd/Makefile          |   1 +
 drivers/mfd/cgbc-core.c       | 411 ++++++++++++++++++++++++++++++++++++++++++
 drivers/watchdog/Kconfig      |  10 +
 drivers/watchdog/Makefile     |   1 +
 drivers/watchdog/cgbc_wdt.c   | 211 ++++++++++++++++++++++
 include/linux/mfd/cgbc.h      |  44 +++++
 14 files changed, 1323 insertions(+)
---
base-commit: 83d650136610d87d8ed0626c178ed18879b69564
change-id: 20240503-congatec-board-controller-82c6b84cd4ea

Best regards,
-- 
Thomas Richard <thomas.richard@bootlin.com>


             reply	other threads:[~2024-09-17 17:01 UTC|newest]

Thread overview: 11+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2024-09-17 17:00 Thomas Richard [this message]
2024-09-17 17:00 ` [PATCH v2 1/5] mfd: add Congatec Board Controller mfd driver Thomas Richard
2024-09-17 17:00 ` [PATCH v2 2/5] gpio: Congatec Board Controller gpio driver Thomas Richard
2024-09-17 19:25   ` Linus Walleij
2024-09-17 17:00 ` [PATCH v2 3/5] i2c: Congatec Board Controller i2c bus driver Thomas Richard
2024-10-01 10:00   ` Andi Shyti
2024-11-19 22:41   ` Andi Shyti
2024-11-20  7:02     ` Andi Shyti
2024-09-17 17:00 ` [PATCH v2 4/5] watchdog: Congatec Board Controller watchdog timer driver Thomas Richard
2024-09-20 14:55   ` Guenter Roeck
2024-09-17 17:00 ` [PATCH v2 5/5] MAINTAINERS: Add entry for Congatec Board Controller Thomas Richard

Reply instructions:

You may reply publicly to this message via plain-text email
using any one of the following methods:

* Save the following mbox file, import it into your mail client,
  and reply-to-all from there: mbox

  Avoid top-posting and favor interleaved quoting:
  https://en.wikipedia.org/wiki/Posting_style#Interleaved_style

* Reply using the --to, --cc, and --in-reply-to
  switches of git-send-email(1):

  git send-email \
    --in-reply-to=20240503-congatec-board-controller-v2-0-681511a01c8f@bootlin.com \
    --to=thomas.richard@bootlin.com \
    --cc=andi.shyti@kernel.org \
    --cc=blake.vermeer@keysight.com \
    --cc=brgl@bgdev.pl \
    --cc=lee@kernel.org \
    --cc=linus.walleij@linaro.org \
    --cc=linux-gpio@vger.kernel.org \
    --cc=linux-i2c@vger.kernel.org \
    --cc=linux-kernel@vger.kernel.org \
    --cc=linux-watchdog@vger.kernel.org \
    --cc=linux@roeck-us.net \
    --cc=thomas.petazzoni@bootlin.com \
    --cc=wim@linux-watchdog.org \
    /path/to/YOUR_REPLY

  https://kernel.org/pub/software/scm/git/docs/git-send-email.html

* If your mail client supports setting the In-Reply-To header
  via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line before the message body.
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for NNTP newsgroup(s).