Re: [PATCH v8 0/5] iio: mcp9600: Features and improvements

[Jonathan Cameron <jic23@kernel.org>]

On Fri, 22 Aug 2025 09:23:49 -0400
Ben Collins <bcollins@watter.com> wrote:

Series applied with that unnecessary include that David pointed out dropped.

Applied to the togreg branch of iio.git but initially pushed out as testing
to give 0-day a few days to poke at it.

Thanks,

Jonathan


> ChangeLog:
> v7 -> v8:
>   - Style changes in dt-bindings example
>   - Simplify some return value checks
>   - Move assignment to where it's checked
>   - Speeling
> 
> v6 -> v7:
>   - Separate out the mcp9600 IIR series into its own series as there is
>     a lot of conversation around implementation (removed related
>     comments from this changelog).
> 
> v5 -> v6:
>   - Fix accidental typo added in dt-bindings: IRQ_TYPE_EDGE_RISIN
>   - Correct some constraints in dt-bindings
>   - Reverse if/then for mcp9601 vs mcp9600 constraints in dt-bindings
>   - Updates to changelog for patch 2/6 (dt-bindings mcp9600)
>   - Cleanup tabs that were converted to spaces
>   - Split thermocouple-type default to separate patch
> 
> v4 -> v5:
>   - None
> 
> v3 -> v4:
>   - Based on feedback from David Lechner <dlechner@baylibre.com>
>     * Allow fallback compatible in dt-bindings for mcp9601.
>   - Based on feedback from Jonathan Cameron <jic23@kernel.org>
>     * Be explicit in patch description for fixed width changes.
>     * Check chip_info for NULL to quiet warnings from kernel-test-robot
>     * Remove "and similar" for long description of MCP9600.
>   - Set default 3 for thermocouple in dt-binding
>   - Rework open/short circuit in dt-bindings
> 
> v2 -> v3:
>   - Improve changelogs in each patch
>   - Based on feedback from Andy Shevchenko <andy.shevchenko@gmail.com>
>     * Set register offsets to fixed width
>     * Fix typos
>     * Future-proof Kconfig changes
>     * Convert to using chip_info paradigm
>     * Verbiage: dt -> firmware description
>     * Use proper specifiers and drop castings
>     * Fix register offset to be fixed-width
>     * u8 for cfg var
>     * Fix % type for u32 to be %u
>     * Make blank lines consistent between case statements
> 
> v1 -> v2:
>   - Break into individual patches
> 
> v1:
>   - Initial patch to enable IIR and thermocouple-type
>   - Recognize mcp9601
> 
> Signed-off-by: Ben Collins <bcollins@watter.com>
> ---
> - Link to v7: https://lore.kernel.org/r/20250819-upstream-changes-v7-0-88a33aa78f6a@watter.com
> 
> ---
> Ben Collins (5):
>       dt-bindings: iio: mcp9600: Set default 3 for thermocouple-type
>       dt-bindings: iio: mcp9600: Add microchip,mcp9601 and add constraints
>       iio: mcp9600: White space and fixed width cleanup
>       iio: mcp9600: Recognize chip id for mcp9601
>       iio: mcp9600: Add support for thermocouple-type
> 
>  .../iio/temperature/microchip,mcp9600.yaml         |  57 +++++++-
>  drivers/iio/temperature/Kconfig                    |   8 +-
>  drivers/iio/temperature/mcp9600.c                  | 151 +++++++++++++++++----
>  3 files changed, 187 insertions(+), 29 deletions(-)
> ---
> base-commit: c17b750b3ad9f45f2b6f7e6f7f4679844244f0b9
> change-id: 20250819-upstream-changes-c89af86743fa
> 
> Best regards,