From: Krzysztof Kozlowski <krzk@kernel.org>
To: linux-aspeed@lists.ozlabs.org
Subject: [PATCH v1 06/10] dt-bindings: arm: aspeed: Add aspeed,ast2700-evb compatible string
Date: Fri, 16 Aug 2024 07:17:15 +0200 [thread overview]
Message-ID: <2b40d7e4-10f5-406a-9ac9-f9292a52a659@kernel.org> (raw)
In-Reply-To: <PSAPR06MB4949F7F617DB4A371567EB4789812@PSAPR06MB4949.apcprd06.prod.outlook.com>
On 16/08/2024 06:08, Kevin Chen wrote:
> Hi Krzk,
>
>>> ---
>>> Documentation/devicetree/bindings/arm/aspeed/aspeed.yaml | 5 +++++
>>> 1 file changed, 5 insertions(+)
>>>
>>> diff --git a/Documentation/devicetree/bindings/arm/aspeed/aspeed.yaml
>>> b/Documentation/devicetree/bindings/arm/aspeed/aspeed.yaml
>>> index 71c31c08a8ad..b21551817f44 100644
>>> --- a/Documentation/devicetree/bindings/arm/aspeed/aspeed.yaml
>>> +++ b/Documentation/devicetree/bindings/arm/aspeed/aspeed.yaml
>>> @@ -99,4 +99,9 @@ properties:
>>> - ufispace,ncplite-bmc
>>> - const: aspeed,ast2600
>>>
>>> + - description: AST2700 based boards
>>> + items:
>>> + - enum:
>>> + - aspeed,ast2700-evb
>>
>> NAK, this cannot be alone. Look at all other examples. Why are you doing
>> things differently?
> Disagree, ast2700-evb is 7th generation IC in ASPEED.
I gave you argument and you just respond "disagree" with unrelated
statement?
> It not in the sub-set of AST2400/AST2500/AST2600 based boards.
? That does not answer at all my concerns. Bring me any example doing
this that way: single board compatible.
Answer to the argument, instead of bringing useless text "ast is a
board". Yeah, of course it is. What else could it be? A ship? A car?
Best regards,
Krzysztof
WARNING: multiple messages have this Message-ID (diff)
From: Krzysztof Kozlowski <krzk@kernel.org>
To: Kevin Chen <kevin_chen@aspeedtech.com>,
"robh@kernel.org" <robh@kernel.org>,
"krzk+dt@kernel.org" <krzk+dt@kernel.org>,
"conor+dt@kernel.org" <conor+dt@kernel.org>,
"joel@jms.id.au" <joel@jms.id.au>,
"andrew@codeconstruct.com.au" <andrew@codeconstruct.com.au>,
"lee@kernel.org" <lee@kernel.org>,
"catalin.marinas@arm.com" <catalin.marinas@arm.com>,
"will@kernel.org" <will@kernel.org>,
"arnd@arndb.de" <arnd@arndb.de>,
"olof@lixom.net" <olof@lixom.net>,
"soc@kernel.org" <soc@kernel.org>,
"mturquette@baylibre.com" <mturquette@baylibre.com>,
"sboyd@kernel.org" <sboyd@kernel.org>,
"p.zabel@pengutronix.de" <p.zabel@pengutronix.de>,
"quic_bjorande@quicinc.com" <quic_bjorande@quicinc.com>,
"geert+renesas@glider.be" <geert+renesas@glider.be>,
"dmitry.baryshkov@linaro.org" <dmitry.baryshkov@linaro.org>,
"shawnguo@kernel.org" <shawnguo@kernel.org>,
"neil.armstrong@linaro.org" <neil.armstrong@linaro.org>,
"m.szyprowski@samsung.com" <m.szyprowski@samsung.com>,
"nfraprado@collabora.com" <nfraprado@collabora.com>,
"u-kumar1@ti.com" <u-kumar1@ti.com>,
"devicetree@vger.kernel.org" <devicetree@vger.kernel.org>,
"linux-arm-kernel@lists.infradead.org"
<linux-arm-kernel@lists.infradead.org>,
"linux-aspeed@lists.ozlabs.org" <linux-aspeed@lists.ozlabs.org>,
"linux-kernel@vger.kernel.org" <linux-kernel@vger.kernel.org>,
"linux-clk@vger.kernel.org" <linux-clk@vger.kernel.org>,
BMC-SW <BMC-SW@aspeedtech.com>
Subject: Re: [PATCH v1 06/10] dt-bindings: arm: aspeed: Add aspeed,ast2700-evb compatible string
Date: Fri, 16 Aug 2024 07:17:15 +0200 [thread overview]
Message-ID: <2b40d7e4-10f5-406a-9ac9-f9292a52a659@kernel.org> (raw)
In-Reply-To: <PSAPR06MB4949F7F617DB4A371567EB4789812@PSAPR06MB4949.apcprd06.prod.outlook.com>
On 16/08/2024 06:08, Kevin Chen wrote:
> Hi Krzk,
>
>>> ---
>>> Documentation/devicetree/bindings/arm/aspeed/aspeed.yaml | 5 +++++
>>> 1 file changed, 5 insertions(+)
>>>
>>> diff --git a/Documentation/devicetree/bindings/arm/aspeed/aspeed.yaml
>>> b/Documentation/devicetree/bindings/arm/aspeed/aspeed.yaml
>>> index 71c31c08a8ad..b21551817f44 100644
>>> --- a/Documentation/devicetree/bindings/arm/aspeed/aspeed.yaml
>>> +++ b/Documentation/devicetree/bindings/arm/aspeed/aspeed.yaml
>>> @@ -99,4 +99,9 @@ properties:
>>> - ufispace,ncplite-bmc
>>> - const: aspeed,ast2600
>>>
>>> + - description: AST2700 based boards
>>> + items:
>>> + - enum:
>>> + - aspeed,ast2700-evb
>>
>> NAK, this cannot be alone. Look at all other examples. Why are you doing
>> things differently?
> Disagree, ast2700-evb is 7th generation IC in ASPEED.
I gave you argument and you just respond "disagree" with unrelated
statement?
> It not in the sub-set of AST2400/AST2500/AST2600 based boards.
? That does not answer at all my concerns. Bring me any example doing
this that way: single board compatible.
Answer to the argument, instead of bringing useless text "ast is a
board". Yeah, of course it is. What else could it be? A ship? A car?
Best regards,
Krzysztof
next prev parent reply other threads:[~2024-08-16 5:17 UTC|newest]
Thread overview: 96+ messages / expand[flat|nested] mbox.gz Atom feed top
2024-07-26 11:03 [PATCH v1 00/10] Introduce ASPEED AST27XX BMC SoC Kevin Chen
2024-07-26 11:09 ` Kevin Chen
2024-07-26 11:03 ` [PATCH v1 01/10] dt-binding: mfd: aspeed,ast2x00-scu: Add binding for ASPEED AST2700 SCU Kevin Chen
2024-07-26 11:09 ` Kevin Chen
2024-07-26 11:10 ` Krzysztof Kozlowski
2024-07-26 11:10 ` Krzysztof Kozlowski
2024-08-15 5:50 ` 回覆: " Kevin Chen
2024-08-15 5:50 ` Kevin Chen
2024-10-07 11:03 ` Krzysztof Kozlowski
2024-10-07 11:03 ` Krzysztof Kozlowski
2024-10-07 11:03 ` Krzysztof Kozlowski
2024-10-07 11:03 ` Krzysztof Kozlowski
2024-08-16 4:06 ` Kevin Chen
2024-08-16 4:06 ` Kevin Chen
2024-07-26 11:03 ` [PATCH v1 02/10] dt-binding: clk: ast2700: Add binding for Aspeed AST27xx Clock Kevin Chen
2024-07-26 11:09 ` Kevin Chen
2024-07-26 11:11 ` Krzysztof Kozlowski
2024-07-26 11:11 ` Krzysztof Kozlowski
2024-08-15 5:50 ` 回覆: " Kevin Chen
2024-08-15 5:50 ` Kevin Chen
2024-08-16 4:06 ` Kevin Chen
2024-08-16 4:06 ` Kevin Chen
2024-07-26 11:03 ` [PATCH v1 03/10] clk: ast2700: add clock controller Kevin Chen
2024-07-26 11:09 ` Kevin Chen
2024-07-26 11:13 ` Krzysztof Kozlowski
2024-07-26 11:13 ` Krzysztof Kozlowski
2024-08-15 5:50 ` 回覆: " Kevin Chen
2024-08-15 5:50 ` Kevin Chen
2024-08-16 4:06 ` Kevin Chen
2024-08-16 4:06 ` Kevin Chen
2024-07-26 11:03 ` [PATCH v1 04/10] dt-bindings: reset: ast2700: Add binding for ASPEED AST2700 Reset Kevin Chen
2024-07-26 11:09 ` Kevin Chen
2024-07-26 11:13 ` Krzysztof Kozlowski
2024-07-26 11:13 ` Krzysztof Kozlowski
2024-08-16 4:07 ` Kevin Chen
2024-08-16 4:07 ` Kevin Chen
2024-07-26 11:03 ` [PATCH v1 05/10] dt-bindings: arm: aspeed: Add maintainer Kevin Chen
2024-07-26 11:09 ` Kevin Chen
2024-07-26 11:14 ` Krzysztof Kozlowski
2024-07-26 11:14 ` Krzysztof Kozlowski
2024-08-15 5:50 ` 回覆: " Kevin Chen
2024-08-15 5:50 ` Kevin Chen
2024-08-16 4:07 ` Kevin Chen
2024-08-16 4:07 ` Kevin Chen
2024-07-26 11:03 ` [PATCH v1 06/10] dt-bindings: arm: aspeed: Add aspeed,ast2700-evb compatible string Kevin Chen
2024-07-26 11:09 ` Kevin Chen
2024-07-26 11:15 ` Krzysztof Kozlowski
2024-07-26 11:15 ` Krzysztof Kozlowski
2024-08-15 5:50 ` 回覆: " Kevin Chen
2024-08-15 5:50 ` Kevin Chen
2024-08-16 4:08 ` Kevin Chen
2024-08-16 4:08 ` Kevin Chen
2024-08-16 5:17 ` Krzysztof Kozlowski [this message]
2024-08-16 5:17 ` Krzysztof Kozlowski
2024-10-07 9:26 ` Kevin Chen
2024-10-07 9:26 ` Kevin Chen
2024-07-26 11:03 ` [PATCH v1 07/10] arm64: aspeed: Add support for ASPEED AST2700 BMC SoC Kevin Chen
2024-07-26 11:09 ` Kevin Chen
2024-07-26 11:16 ` Krzysztof Kozlowski
2024-07-26 11:16 ` Krzysztof Kozlowski
2024-08-15 5:51 ` 回覆: " Kevin Chen
2024-08-15 5:51 ` Kevin Chen
2024-08-16 4:07 ` Kevin Chen
2024-08-16 4:07 ` Kevin Chen
2024-07-26 11:03 ` [PATCH v1 08/10] arm64: dts: aspeed: Add initial AST27XX device tree Kevin Chen
2024-07-26 11:09 ` Kevin Chen
2024-07-26 11:19 ` Krzysztof Kozlowski
2024-07-26 11:19 ` Krzysztof Kozlowski
2024-08-15 5:50 ` 回覆: " Kevin Chen
2024-08-15 5:50 ` Kevin Chen
2024-08-16 5:08 ` Krzysztof Kozlowski
2024-08-16 5:08 ` Krzysztof Kozlowski
2024-10-07 9:26 ` Kevin Chen
2024-10-07 9:26 ` Kevin Chen
2024-10-07 11:05 ` Krzysztof Kozlowski
2024-10-07 11:05 ` Krzysztof Kozlowski
2024-08-16 4:07 ` Kevin Chen
2024-08-16 4:07 ` Kevin Chen
2024-07-26 11:03 ` [PATCH v1 09/10] arm64: dts: aspeed: Add initial AST2700 EVB " Kevin Chen
2024-07-26 11:09 ` Kevin Chen
2024-07-26 11:16 ` Krzysztof Kozlowski
2024-07-26 11:16 ` Krzysztof Kozlowski
2024-08-15 5:50 ` 回覆: " Kevin Chen
2024-08-15 5:50 ` Kevin Chen
2024-08-16 4:07 ` Kevin Chen
2024-08-16 4:07 ` Kevin Chen
2024-07-26 11:03 ` [PATCH v1 10/10] arm64: defconfig: Add ASPEED AST2700 family support Kevin Chen
2024-07-26 11:09 ` Kevin Chen
2024-07-26 13:09 ` [PATCH v1 00/10] Introduce ASPEED AST27XX BMC SoC Rob Herring
2024-07-26 13:09 ` Rob Herring (Arm)
2024-07-26 13:33 ` Krzysztof Kozlowski
2024-07-26 13:33 ` Krzysztof Kozlowski
2024-08-15 5:50 ` 回覆: " Kevin Chen
2024-08-15 5:50 ` Kevin Chen
2024-08-16 4:06 ` Kevin Chen
2024-08-16 4:06 ` Kevin Chen
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=2b40d7e4-10f5-406a-9ac9-f9292a52a659@kernel.org \
--to=krzk@kernel.org \
--cc=linux-aspeed@lists.ozlabs.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 an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.