Received-SPF: pass (google.com: domain of linux-kernel-owner@vger.kernel.org designates 23.128.96.34 as permitted sender) client-ip=23.128.96.34;
From:   Inochi Amaoto <inochiama@outlook.com>
To:     Krzysztof Kozlowski <krzysztof.kozlowski@linaro.org>
Cc:     aou@eecs.berkeley.edu, chao.wei@sophgo.com, conor@kernel.org,
        evicetree@vger.kernel.org, emil.renner.berthing@canonical.com,
        guoren@kernel.org, jszhang@kernel.org,
        krzysztof.kozlowski+dt@linaro.org, linux-kernel@vger.kernel.org,
        linux-riscv@lists.infradead.org, palmer@dabbelt.com,
        paul.walmsley@sifive.com, robh+dt@kernel.org,
        xiaoguang.xing@sophgo.com, Chen Wang <wangchen20@iscas.ac.cn>,
        Inochi Amaoto <inochiama@outlook.com>
Subject: Re: [PATCH v2 06/11] dt-bindings: timer: Add Sophgo sg2042 clint
Date:   Wed, 20 Sep 2023 20:15:12 +0800
Message-ID: <IA1PR20MB4953117947BA7DAA60C1F435BBF9A@IA1PR20MB4953.namprd20.prod.outlook.com>
In-Reply-To: <0b84ef92-f6b7-f3bc-fd51-8f61cbc91ff8@linaro.org>
References: <0b84ef92-f6b7-f3bc-fd51-8f61cbc91ff8@linaro.org>
Content-Transfer-Encoding: 8bit
Content-Type: text/plain
MIME-Version: 1.0
X-MS-Exchange-MessageSentRepresentingType: 1
X-MS-PublicTrafficType: Email
X-MS-TrafficTypeDiagnostic: IA1PR20MB4953:EE_|CH3PR20MB6491:EE_
X-MS-Office365-Filtering-Correlation-Id: 57bcb162-2b57-4e90-ed4e-08dbb9d34117
X-Microsoft-Antispam: BCL:0;
X-Microsoft-Antispam-Message-Info: tbzSeT4IdxxtmrmWyunKnkcTX2uhXvwF/bszEXrFFQK0V+VpuSHbohAQiNJc4A9s1BsyJA9WPxq1mPtes/DI4mDUE78ZpbxGUk6sT0xSSo21+NYzldOGVQ0r3eJlLKpg+LiiO41MvqlzPQoWFlQo906D2SvAV6yOD6KODa7JIMRY16kCY2m//kohceCFTl0OUfUqHUdPJVlTxzy3hEyyT1goUkOwi9IMRjfY0ZiPPH1P1Tdi7UmdeitSJjeP2se1BfXI2LbSWtqQGwAIcKnIkiEnFGBGUvSJH1gRD9cYk7//t1txM6Y3u5HN3tS49mL+OMpKMhIcW43DDwbVgVOjyUKrPjdyFdxldQwYkes4r5pdx56m7RgjMbsCxo+QL0+Ml0ooOJPbEl4d+iUPdxWRxmmGhLUCI7LB2wlatdgRNr22SH9jeaNZzMpuikMVZivrJCpwAK+EYZCs7KbS4Bq1Gjo/Y9VzJ9wZ1ByKq+oo2Aw2+/TTgyFWxKyWxZMQRFEPeJfG0IVDTl55MWSoUm80zZFY5+h/x1oIqfBfDVBRzub+oDM9YCi6GvFcix8OXmPshwBerkD2XLjlEnkuw6cXmAeOTYww5DruIsqhyawGgQMUtmjB/NpRAOAzvL9o7JgzXQyOe8axKBivVbeva90Fyg==
X-MS-Exchange-AntiSpam-MessageData-ChunkCount: 1
X-MS-Exchange-AntiSpam-MessageData-0: =?us-ascii?Q?w493NV8UQdYgAMo5e5Sqcc549pCyggTq0jYY0R74SynDkKm69cjmnjGpck9e?=
 =?us-ascii?Q?sc893pZlW3IM1PIuh2E5E7uQBL3p37OxCDqMhRF6ub+IZ6HmlubdzmqIhvxP?=
 =?us-ascii?Q?8IBF+xiEaBx5pxPSyh5ciUvbIBOZwlqG0IMdp3osy3nxfX/ngqQC/BEYX4XC?=
 =?us-ascii?Q?fM9/psuOmtNsUdedcl/ttQR5mg1/f0LWrtXZMcoAzEHLi4xT236iErbKgZMi?=
 =?us-ascii?Q?mIJq+SD+MLcFLXGA/6acFIPf5UkC+H2+l3+JiA5R/qlr1M+8g8HyerE2d+Qq?=
 =?us-ascii?Q?gGkAjEg/9Vn+K30wLVoFOTjcR+7CgGllW4F+kDQHoIC79U3YeaBBJ/WcR45y?=
 =?us-ascii?Q?PTXCMbY0D/QDwrcylyPSrUU5CTX4tVzS3mlptQxvjwTBqj9zB+4LFeO4rhvo?=
 =?us-ascii?Q?U/eRB7rDvXDDLQOnAFdXxFFZKbwyIpmV7DRJbQs1VhVKaq4qSq3PGRrmij+S?=
 =?us-ascii?Q?rBzey05zvHa8yVQwpzIG4ctsOmGWfORHkQeaJ1gXyv6rKfWhVVROLUk3XIYN?=
 =?us-ascii?Q?ystC7r9a2GWOb2Y+9KAg+QiT9oJE36q30SibncFqTJEFpN++silmaisSZtZt?=
 =?us-ascii?Q?e9TLpyx+mFY/7uPDi/2QdDQ2UGMDIYhhkypT5aaa59qd1VbGSFk1dHfEp+PL?=
 =?us-ascii?Q?7cM45gTlXldReQfU0m8ES60AvCTqTa31Js/awv2KcpdLUDg1XuvkAQ41+eEv?=
 =?us-ascii?Q?R9fbZgdHs1EctDqbe3PRJunaYK3x9M4ekKdS8zIiyRA+saR5be3uuOUUk4aR?=
 =?us-ascii?Q?yWk6y+aqzEnTx42+EKc+3raPasZnRo/l/eR+lX6022rs98w9GVqtC/Cr6S2u?=
 =?us-ascii?Q?48I1fhgcMfxfAT0maD3iJmLcMmv6rT6hHIf/zF4ZsiXj1ShJudxEBO14pCwf?=
 =?us-ascii?Q?KdiAf588fZsKX78XfpsWm0DxCqZL7Cifwd4NPk8tMQMmMHh3wtjOSj1b8ovC?=
 =?us-ascii?Q?GEiUp4PMIXHVgLYr7n8EFm3AjQzyQ6rDrCPpvOPe/Atcr9orc9MUP9mh75mf?=
 =?us-ascii?Q?ermxunLAHx3gXa3ABkE9k+R1H+reP7WtdsJXAr5CUaaZbjfU8GyZ7S2NgF88?=
 =?us-ascii?Q?v8wsn6F6se/V0oCD+uRcs3PruGtpS7lpCxicBGJlbHShFx4vGz5Bp8jA/SZY?=
 =?us-ascii?Q?W3VsMgbjSDK6KciyRy0Fy85tcH1khqHMSByu/WtWSDZwMSnxlR5ZZOMkVjMC?=
 =?us-ascii?Q?AWIZboayuzgNom7o+ZQXIaRgRu/jPPAdNHxANmv1AQsB5QD27H0t3P2f2nw?=
 =?us-ascii?Q?=3D?=
X-OriginatorOrg: outlook.com
X-MS-Exchange-CrossTenant-Network-Message-Id: 57bcb162-2b57-4e90-ed4e-08dbb9d34117
X-MS-Exchange-CrossTenant-AuthSource: IA1PR20MB4953.namprd20.prod.outlook.com
X-MS-Exchange-CrossTenant-AuthAs: Internal
X-MS-Exchange-CrossTenant-OriginalArrivalTime: 20 Sep 2023 12:15:18.8774
 (UTC)
X-MS-Exchange-CrossTenant-FromEntityHeader: Hosted
X-MS-Exchange-CrossTenant-Id: 84df9e7f-e9f6-40af-b435-aaaaaaaaaaaa
X-MS-Exchange-CrossTenant-RMS-PersistedConsumerOrg: 00000000-0000-0000-0000-000000000000
X-MS-Exchange-Transport-CrossTenantHeadersStamped: CH3PR20MB6491
X-Spam-Status: No, score=-2.1 required=5.0 tests=BAYES_00,DKIM_SIGNED,
        DKIM_VALID,DKIM_VALID_AU,DKIM_VALID_EF,FREEMAIL_FROM,
        RCVD_IN_DNSWL_BLOCKED,RCVD_IN_MSPIKE_H2,SPF_HELO_PASS,SPF_PASS
        autolearn=ham autolearn_force=no version=3.4.6
X-Spam-Checker-Version: SpamAssassin 3.4.6 (2021-04-09) on
        lindbergh.monkeyblade.net
Precedence: bulk
List-ID: <linux-kernel.vger.kernel.org>
X-Mailing-List: linux-kernel@vger.kernel.org
X-Greylist: Sender passed SPF test, not delayed by milter-greylist-4.6.4 (howler.vger.email [0.0.0.0]); Wed, 20 Sep 2023 05:15:34 -0700 (PDT)

>On 20/09/2023 08:39, Chen Wang wrote:
>> From: Inochi Amaoto <inochiama@outlook.com>
>>
>> Add two new compatible string formatted like `C9xx-clint-xxx` to identify
>> the timer and ipi device separately, and do not allow c900-clint as the
>
>Why?
>

If use the same compatible, SBI will process this twice in both ipi and
timer, use different compatible will allow SBI to treat these as different.
AFAIK, the aclint in SBI use the same concepts, which make hard to use the
second register range. I have explained in another response.

https://lore.kernel.org/all/IA1PR20MB495313B7E9B2FC529BE0BB2ABBF9A@IA1PR20MB4953.namprd20.prod.outlook.com/

>You received comment about it, so please provide proper explanation in
>the commit msg.
>
>Same device does not get two different compatibles.
>
>You also did not respond to my comments, so you basically ignored it and
>send the same.
>
>NAK
>
>Best regards,
>Krzysztof
>

Sorry for this, as I mistake the idea of the last message. All of this
will be fixed in the next patch.