linux-arm-kernel.lists.infradead.org archive mirror
 help / color / mirror / Atom feed
* [PATCH v1 1/2] dt-bindings: hwmon: Add AST2700 compatible
@ 2025-11-04  5:51 Billy Tsai
  2025-11-04  5:51 ` [PATCH v1 2/2] hwmon: (aspeed-g6-pwm-tach): Add AST2700 compatible string Billy Tsai
  2025-11-04 17:23 ` [PATCH v1 1/2] dt-bindings: hwmon: Add AST2700 compatible Conor Dooley
  0 siblings, 2 replies; 3+ messages in thread
From: Billy Tsai @ 2025-11-04  5:51 UTC (permalink / raw)
  To: linux, robh, krzk+dt, conor+dt, joel, andrew, billy_tsai,
	linux-hwmon, devicetree, linux-arm-kernel, linux-aspeed,
	linux-kernel

Adds support for the AST2700 PWM/Tach controller by extending the
compatible string enumeration in the device tree binding.

The AST2700 PWM/Tach hardware is compatible with the existing binding
schema and requires no additional properties or modifications beyond
the new compatible string.

Signed-off-by: Billy Tsai <billy_tsai@aspeedtech.com>
---
 Documentation/devicetree/bindings/hwmon/aspeed,g6-pwm-tach.yaml | 1 +
 1 file changed, 1 insertion(+)

diff --git a/Documentation/devicetree/bindings/hwmon/aspeed,g6-pwm-tach.yaml b/Documentation/devicetree/bindings/hwmon/aspeed,g6-pwm-tach.yaml
index 9e5ed901ae54..d6ff4182085e 100644
--- a/Documentation/devicetree/bindings/hwmon/aspeed,g6-pwm-tach.yaml
+++ b/Documentation/devicetree/bindings/hwmon/aspeed,g6-pwm-tach.yaml
@@ -20,6 +20,7 @@ properties:
   compatible:
     enum:
       - aspeed,ast2600-pwm-tach
+      - aspeed,ast2700-pwm-tach
 
   reg:
     maxItems: 1
-- 
2.25.1



^ permalink raw reply related	[flat|nested] 3+ messages in thread

* [PATCH v1 2/2] hwmon: (aspeed-g6-pwm-tach): Add AST2700 compatible string
  2025-11-04  5:51 [PATCH v1 1/2] dt-bindings: hwmon: Add AST2700 compatible Billy Tsai
@ 2025-11-04  5:51 ` Billy Tsai
  2025-11-04 17:23 ` [PATCH v1 1/2] dt-bindings: hwmon: Add AST2700 compatible Conor Dooley
  1 sibling, 0 replies; 3+ messages in thread
From: Billy Tsai @ 2025-11-04  5:51 UTC (permalink / raw)
  To: linux, robh, krzk+dt, conor+dt, joel, andrew, billy_tsai,
	linux-hwmon, devicetree, linux-arm-kernel, linux-aspeed,
	linux-kernel

Extends device tree support to include the AST2700 chip variant by
adding its compatible string to the device match table.

The AST2700 PWM/TACH hardware is compatible with the existing driver
implementation used for AST2600.

Signed-off-by: Billy Tsai <billy_tsai@aspeedtech.com>
---
 drivers/hwmon/aspeed-g6-pwm-tach.c | 3 +++
 1 file changed, 3 insertions(+)

diff --git a/drivers/hwmon/aspeed-g6-pwm-tach.c b/drivers/hwmon/aspeed-g6-pwm-tach.c
index 4174b129d1fc..44e1ecba205d 100644
--- a/drivers/hwmon/aspeed-g6-pwm-tach.c
+++ b/drivers/hwmon/aspeed-g6-pwm-tach.c
@@ -528,6 +528,9 @@ static const struct of_device_id aspeed_pwm_tach_match[] = {
 	{
 		.compatible = "aspeed,ast2600-pwm-tach",
 	},
+	{
+		.compatible = "aspeed,ast2700-pwm-tach",
+	},
 	{},
 };
 MODULE_DEVICE_TABLE(of, aspeed_pwm_tach_match);
-- 
2.25.1



^ permalink raw reply related	[flat|nested] 3+ messages in thread

* Re: [PATCH v1 1/2] dt-bindings: hwmon: Add AST2700 compatible
  2025-11-04  5:51 [PATCH v1 1/2] dt-bindings: hwmon: Add AST2700 compatible Billy Tsai
  2025-11-04  5:51 ` [PATCH v1 2/2] hwmon: (aspeed-g6-pwm-tach): Add AST2700 compatible string Billy Tsai
@ 2025-11-04 17:23 ` Conor Dooley
  1 sibling, 0 replies; 3+ messages in thread
From: Conor Dooley @ 2025-11-04 17:23 UTC (permalink / raw)
  To: Billy Tsai
  Cc: linux, robh, krzk+dt, conor+dt, joel, andrew, linux-hwmon,
	devicetree, linux-arm-kernel, linux-aspeed, linux-kernel

[-- Attachment #1: Type: text/plain, Size: 1178 bytes --]

On Tue, Nov 04, 2025 at 01:51:11PM +0800, Billy Tsai wrote:
> Adds support for the AST2700 PWM/Tach controller by extending the
> compatible string enumeration in the device tree binding.
> 
> The AST2700 PWM/Tach hardware is compatible with the existing binding
> schema and requires no additional properties or modifications beyond
> the new compatible string.

With this commit message, I expected there to be a fallback compatible.
Why isn't one used?

> 
> Signed-off-by: Billy Tsai <billy_tsai@aspeedtech.com>
> ---
>  Documentation/devicetree/bindings/hwmon/aspeed,g6-pwm-tach.yaml | 1 +
>  1 file changed, 1 insertion(+)
> 
> diff --git a/Documentation/devicetree/bindings/hwmon/aspeed,g6-pwm-tach.yaml b/Documentation/devicetree/bindings/hwmon/aspeed,g6-pwm-tach.yaml
> index 9e5ed901ae54..d6ff4182085e 100644
> --- a/Documentation/devicetree/bindings/hwmon/aspeed,g6-pwm-tach.yaml
> +++ b/Documentation/devicetree/bindings/hwmon/aspeed,g6-pwm-tach.yaml
> @@ -20,6 +20,7 @@ properties:
>    compatible:
>      enum:
>        - aspeed,ast2600-pwm-tach
> +      - aspeed,ast2700-pwm-tach
>  
>    reg:
>      maxItems: 1
> -- 
> 2.25.1
> 

[-- Attachment #2: signature.asc --]
[-- Type: application/pgp-signature, Size: 228 bytes --]

^ permalink raw reply	[flat|nested] 3+ messages in thread

end of thread, other threads:[~2025-11-04 17:23 UTC | newest]

Thread overview: 3+ messages (download: mbox.gz follow: Atom feed
-- links below jump to the message on this page --
2025-11-04  5:51 [PATCH v1 1/2] dt-bindings: hwmon: Add AST2700 compatible Billy Tsai
2025-11-04  5:51 ` [PATCH v1 2/2] hwmon: (aspeed-g6-pwm-tach): Add AST2700 compatible string Billy Tsai
2025-11-04 17:23 ` [PATCH v1 1/2] dt-bindings: hwmon: Add AST2700 compatible Conor Dooley

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).