Received-SPF: pass (google.com: domain of linux-kernel-owner@vger.kernel.org designates 2620:137:e000::1:20 as permitted sender) client-ip=2620:137:e000::1:20;
Message-ID: <9da24c47-341b-1581-0043-a6c1cdce462d@amlogic.com>
Date:   Thu, 27 Jul 2023 16:45:48 +0800
User-Agent: Mozilla/5.0 (Windows NT 10.0; Win64; x64; rv:102.0) Gecko/20100101
 Thunderbird/102.13.0
Subject: Re: [PATCH 3/3] arm64: dts: Replace the IRQ number with the IRQID
 macro definition
To:     Kevin Hilman <khilman@baylibre.com>, robh+dt@kernel.org,
        krzysztof.kozlowski+dt@linaro.org, conor+dt@kernel.org,
        neil.armstrong@linaro.org, jbrunet@baylibre.com,
        martin.blumenstingl@googlemail.com, linus.walleij@linaro.org
Cc:     devicetree@vger.kernel.org, linux-arm-kernel@lists.infradead.org,
        linux-amlogic@lists.infradead.org, linux-kernel@vger.kernel.org,
        linux-gpio@vger.kernel.org
References: <20230721073214.1876417-1-huqiang.qin@amlogic.com>
 <20230721073214.1876417-4-huqiang.qin@amlogic.com>
 <7hcz0ei9r4.fsf@baylibre.com>
Content-Language: en-US
From:   Huqiang Qin <huqiang.qin@amlogic.com>
In-Reply-To: <7hcz0ei9r4.fsf@baylibre.com>
Content-Type: text/plain; charset=UTF-8
Content-Transfer-Encoding: 7bit
MIME-Version: 1.0
X-MS-Exchange-AntiSpam-MessageData-ChunkCount: 1
X-MS-Exchange-AntiSpam-MessageData-0: =?utf-8?B?WHJnWEFNZWdyOFE4aG52NTNmaW03ZGwyeHdYTXhrT2VJaFA4d3BsODNOVDlF?=
 =?utf-8?B?MmN6cnNkejhDTFRnR3E2YW4ycmxWdzZyNGYyWklkQjJseExISDB6ZHVCU0hJ?=
 =?utf-8?B?b3k1Rk83Z2FOSHVKWVRZK2dBR3FYZVNuUWRmU1NUWE1zV1lUaGxGNTVoMjJH?=
 =?utf-8?B?U29rTmdIWFlpTFVyVTlQbW15a2JqZ0JudlFQYXFTN0tYK3ZKNHdNazd6SURr?=
 =?utf-8?B?TlJkYklWUFp2WFMzdnVwYnJJKy9nS2tHSUdNRzFyVUhJZm5EMjZPUElRL05Y?=
 =?utf-8?B?VmxwK2UrMjdycXlsbWlYcGhZZnNPenpLUm9SekE0K0hMVjdaZWszc2FKYkh0?=
 =?utf-8?B?cVpnSU9CeWRrVU9kZlExeU5GSFBxdGRIbDFxejh0bTRHYi9LZXNzVE5XSTFa?=
 =?utf-8?B?d2JjN3dsd2EraTdZUW5OZkRsbnBnWHZSdVJCTlkyZUE4NGdLUmtpK0ViNkRh?=
 =?utf-8?B?dkc4UWw0NkE0MDkwWUtLZGU4eTEvaTZGd1EveGVzUnk4RVpEYnEyWUphbXRm?=
 =?utf-8?B?WkJDRGQ3YkpkQ3Z0SHlXQ01DeEVGSDh4MTJiK2NhMTVhVFRneFVqQ0I3MGRv?=
 =?utf-8?B?a1BRT1pSbTNMbENiK040V3BRQi9nZ3Q4dG1mZWhJWGJ0aG8xN3AvaldWblpK?=
 =?utf-8?B?dWNYdFlyalBxL0c5b2NGRCtINXZZQkFCT2JHY1hsYzJSdS90RE5RQ2VQV0pz?=
 =?utf-8?B?ZTdzSFh3M3JxbnJJOXNRaEY4UDVMOXF0dktieUYrWnQ1RFhCajZiRzRvMnVt?=
 =?utf-8?B?TGQ2TGJlcUVpTkdWeXdFTGs4Q3E0eEVEbDdIS3Y5VlJtSXBlV2gzOW51S0pK?=
 =?utf-8?B?bnBEVVVDNElvS2s3QTZCNFA0ZW9pYytwZ0UyeFFZc2FBMkx6ZEZ2aSs1SlR0?=
 =?utf-8?B?ZlV1bXFGdjZEWjIrdjNndEk5Y3YvQ0NnaGRRbGVyUytkb3hrcjMvUlhtNWsv?=
 =?utf-8?B?YUhOODJZK3ZxVVFxc3hSL2UwYUErUVZYc1Z4RTBVeDlIRXNJY1NEUEkxcVdp?=
 =?utf-8?B?T0Z0VEc5T0IxNjhsUWk3VzNIU3g5SWN5MDBZdVlMWUh4NEszTXU1SERlYU9Z?=
 =?utf-8?B?aVJ0Z2JLWHJwcWQrQ2hyUExkbGd0SDZrTHFSbENIaWhYNnRiWG1XNDI0ejhO?=
 =?utf-8?B?RmdId3dsU0xKcEdtd2VjbkQ1Ym5KcitteFhvMHRsRlV0aVBEai94blg0d0lZ?=
 =?utf-8?B?U2EvMWNTak04V2E3NDNJNWJEc0NpM0tZVmRoUXBucGxYczdoRTVWN3VkbkJz?=
 =?utf-8?B?eHlXVVppMEx0dENueE9WcEx0eWI3MGdsMjJUajJDMlA5WHIxMUFRQ25JZHp3?=
 =?utf-8?B?akxUeTNjK0JaNDFsV3pIVXFhcURqT2FYQXFjMWZpMzE5a29HbUtxQzRvcmdl?=
 =?utf-8?B?cjhFWE9FWDJJd3Z5VktCVlBrVkMyRWdmQ1lOQVZLclA1dGFqQ01pbUxXTGJE?=
 =?utf-8?B?SzhSWXdSMFl1emZxYmhXZzVvQ283dGFZVVVPRGpQa0JMbnBjNlAzZWRVT25p?=
 =?utf-8?B?Mkp3U21uQzloSlVLWC9KUHZNODczMC9yRTNYZmZKYU5XQ2FMY2lFc2F5OWFt?=
 =?utf-8?B?QmxBWExINnI3aFlpcGF3KzRiVUhsM1JsTUh5alEwQWpzc3ZqaElaM2Y5T3ly?=
 =?utf-8?B?WlhJdkEvY0VwUWphNVltRG5YQ3ZpZk84MDhpVThyMmQ3dGVsak9LK09UNDFo?=
 =?utf-8?B?R0x0K2E4LzFpSzJYRkRKTVlDWjhRcElCcmorUjRvK0FTYUNic2UycTlLNGs4?=
 =?utf-8?B?M3ZIY1haeVBJTXlDZGk0NWJyKzN1dWRKOGh3amJjWTBBMWxyK1VadGtHMkpU?=
 =?utf-8?B?eGxYVFF0bnQrZ0N4VGR6THVKRE5WNVV6OEtBM0lIYktyK2xra0E5TXpxTmdl?=
 =?utf-8?B?VEQ4RFZ0VFM2QlFEb01EZ3NsUVR2YnNlMzNjbzNvWllyaGpVb3VpM0FMZmwr?=
 =?utf-8?B?NnErUWpuL3FnQ0QySkRIQmlaNzRIUkM0a1dTT2VnWVZxdndFdDdFb1VHUThm?=
 =?utf-8?B?Um1JK3BPTzFybFg2VlJvQ21CMmhNVlNXelZvb3hZaWtkNWFtTElFMVAzU3lr?=
 =?utf-8?B?b25XZUtUaWhuTWNabjBkMytqWWplN2x3aUJWYi9zdzFvb2w4Ymd3Y1BoZkJZ?=
 =?utf-8?B?VlJkNnlqRkN0N3dNakFQaTkzOGNGUjFvc0I4ZkR6UzZta21XN00zMVBSYWZk?=
 =?utf-8?B?b3c9PQ==?=
X-OriginatorOrg: amlogic.com
X-MS-Exchange-CrossTenant-Network-Message-Id: 5b4f4a4b-f79e-42e5-0e7d-08db8e7de3fb
X-MS-Exchange-CrossTenant-AuthSource: KL1PR03MB7221.apcprd03.prod.outlook.com
X-MS-Exchange-CrossTenant-AuthAs: Internal
X-MS-Exchange-CrossTenant-OriginalArrivalTime: 27 Jul 2023 08:45:56.5713
 (UTC)
X-MS-Exchange-CrossTenant-FromEntityHeader: Hosted
X-MS-Exchange-CrossTenant-Id: 0df2add9-25ca-4b3a-acb4-c99ddf0b1114
X-MS-Exchange-CrossTenant-MailboxType: HOSTED
X-MS-Exchange-CrossTenant-UserPrincipalName: M2LuHYAdf46ssrpkI4HaBqy3FQF7W7Jfj0ry9pZYSSamEnLWdHLjcvOwU52jMoPoq3Zt4zpdpCtmsUHG89uPSg==
X-MS-Exchange-Transport-CrossTenantHeadersStamped: SEZPR03MB8232
X-Spam-Status: No, score=-2.2 required=5.0 tests=BAYES_00,DKIM_SIGNED,
        DKIM_VALID,DKIM_VALID_AU,DKIM_VALID_EF,NICE_REPLY_A,RCVD_IN_DNSWL_NONE,
        RCVD_IN_MSPIKE_H2,SPF_HELO_PASS,SPF_PASS,T_SCC_BODY_TEXT_LINE
        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

Hi Kevin Hilman,

On 2023/7/27 6:33, Kevin Hilman wrote:
>> Replacing IRQ numbers with IRQID macro definitions makes node properties
>> easier to understand and also makes GPIO interrupts easier to use.
>>
>> Associated platforms:
>> - Amlogic Meson-G12A
>> - Amlogic Meson-G12B
>> - Amlogic Meson-SM1
> Does this mean you tested/validated these changes on those platforms
> also?

Amlogic's G12A/G12B/SM1 have the same GPIO, and the GPIO interrupt number
is also the same, so I only tested on the SM1 platform, and the test results
prove that these changes are good.

Best Regards,
Huqiang Qin