Message-ID: <9adaa7c15ff08464c9ec8068f8d7a2d1dde46e23.camel@nxp.com>
Subject: Re: [PATCH] drm: Use of_property_present() for testing DT property
 presence
From:   Liu Ying <victor.liu@nxp.com>
To:     Rob Herring <robh@kernel.org>,
        Andrzej Hajda <andrzej.hajda@intel.com>,
        Neil Armstrong <neil.armstrong@linaro.org>,
        Robert Foss <rfoss@kernel.org>,
        Laurent Pinchart <Laurent.pinchart@ideasonboard.com>,
        Jonas Karlman <jonas@kwiboo.se>,
        Jernej Skrabec <jernej.skrabec@gmail.com>,
        David Airlie <airlied@gmail.com>,
        Daniel Vetter <daniel@ffwll.ch>,
        Shawn Guo <shawnguo@kernel.org>,
        Sascha Hauer <s.hauer@pengutronix.de>,
        Pengutronix Kernel Team <kernel@pengutronix.de>,
        Fabio Estevam <festevam@gmail.com>,
        NXP Linux Team <linux-imx@nxp.com>,
        Maarten Lankhorst <maarten.lankhorst@linux.intel.com>,
        Maxime Ripard <mripard@kernel.org>,
        Thomas Zimmermann <tzimmermann@suse.de>,
        Rob Clark <robdclark@gmail.com>,
        Abhinav Kumar <quic_abhinavk@quicinc.com>,
        Dmitry Baryshkov <dmitry.baryshkov@linaro.org>,
        Sean Paul <sean@poorly.run>, Chen-Yu Tsai <wens@csie.org>,
        Samuel Holland <samuel@sholland.org>,
        Emma Anholt <emma@anholt.net>
Cc:     devicetree@vger.kernel.org, dri-devel@lists.freedesktop.org,
        linux-arm-kernel@lists.infradead.org, linux-kernel@vger.kernel.org,
        linux-arm-msm@vger.kernel.org, freedreno@lists.freedesktop.org,
        linux-sunxi@lists.linux.dev
Date:   Mon, 13 Mar 2023 12:04:44 +0800
In-Reply-To: <20230310144705.1542207-1-robh@kernel.org>
References: <20230310144705.1542207-1-robh@kernel.org>
Content-Type: text/plain; charset="UTF-8"
User-Agent: Evolution 3.36.4-0ubuntu1 
Content-Transfer-Encoding: 7bit
MIME-Version: 1.0
X-MS-Exchange-AntiSpam-MessageData-ChunkCount: 1
X-MS-Exchange-AntiSpam-MessageData-0: =?utf-8?B?cEVhRTlaaVViWUF1MGF1eHQ3QlFBcG9KQmp2NFhZYVVFMEcvVzJrcWg5SDFC?=
 =?utf-8?B?R1k1T0Fid0txMmFPTzkzT2U1NzdMdXNKZUU4THpraDhzSlhKeTZJSlBQT1Zq?=
 =?utf-8?B?ODRyVHBQUUE4Q0xJejNWM0FsVVkxMXdSMyttN2o1NHBaOWxyenZVTklDanh0?=
 =?utf-8?B?RG5Bd1p0ditMNHUzbHhaM254REE1ZUlFaVp6MTJRd2dvTG9rK1h5anl2ZDAw?=
 =?utf-8?B?cGRSSFNMUUp2VzRkK3hhLzZ5Z0hadGp6MnNVUTlXV25IcXRvUWZObktCbHoz?=
 =?utf-8?B?RDhWeUs0VDZUVUJPSmV1SlNSWW9JakxhTTVnaThSLzA3YW5rRGllUWxKNHU0?=
 =?utf-8?B?WHM3ajltclZxNG53bGxLdGhqNTVla3dOZFZBajRJbWZ5WjdtME9xNnhNZFIz?=
 =?utf-8?B?OXRmRklocXgyV3B4NU9lYks4U3NmUWp4SS9BWkh4eTNvQlBCaUNhcjNaOEFs?=
 =?utf-8?B?QitFVkdJV2JjMThyM0wvUXMyTjI5ZEJMVmV2SDZIaVRlbjhWekZ6aWlxdllB?=
 =?utf-8?B?RlkvbEhiMFAzZ21Xa0tkU1ljTFRkYTJvSGhRVDREZGFaWE1VcTdxRkVNdko2?=
 =?utf-8?B?U3RVb0J6QjI2cUwxWXNCcVdQSVdrZi8wY25lVHpvR1l4YlpDSDlJR3NZUVV1?=
 =?utf-8?B?LzlRUGNkS3VUOHQxWlVDenN4UDlTTmJJWndnMjF6OTl1RG5OTW5UWnkwbFUz?=
 =?utf-8?B?VnphMnRVRXFocnBaOEh6T01DTCtkbW9kSkJIcWdwU0dSYmpPZWRmcDV1TWRE?=
 =?utf-8?B?UzkxUHQ5dVF4bHRaWk1JbnEzSTh4M0hMNGx1YldYWXgxQnJMQXRyQmhCWHFR?=
 =?utf-8?B?OHZXNkhXak84bDFFOFBDWVlxbzVZSWphbkFTZHI3amU5K0pxZGc5S1hMSkEx?=
 =?utf-8?B?bUZhZ3dOZ1cvakREM1ltUDZwTzNSbHFDcEpuYVRrWitDNnVGMkpuTFRBeThU?=
 =?utf-8?B?WFBtd3dEdFpSOTVnVTQ5dHJWbnJWamJSQ0VGbjcydEZ5bm92UXN4RmpnZ1J1?=
 =?utf-8?B?L1lpN05RRHMwKyt6OHJWTDljd2dnditVUFZiRjZ2Q3NPTDZpMi9IM0dJeDJS?=
 =?utf-8?B?ZEJsb0JGRnI1K0Z5M0NFdjdmZ3JsN1RJRlZOQ2dzeHExWCtHd1FoZ1NKbUVE?=
 =?utf-8?B?MTBWakFKREtPZFFNT3VEQ3hxVlBFdi9xVVROMG5IOThUV0lrZWxTMjcrNGdX?=
 =?utf-8?B?R3g3RXhLTXQ0QXFrYmhvTFladDkrOEZ1NmRaNHhFeG9pU1c2d29MN1BxNDlO?=
 =?utf-8?B?WUNPeFVQUmd0YmJqN254OUx4Y0dHZlIzQVd1dmJNeFdjdVpveTZoWDZZZ1lm?=
 =?utf-8?B?VFkyRXFpdjRwcW41T3hpQitwSlp1OTJuaTZNR2Njb2ZpL3BITk54aUZNYm1O?=
 =?utf-8?B?aHYyQkl3bXdsQXIzSnFWU04yQlN4MitEZUNmc2xjOHhINFA3SXovclhnajQv?=
 =?utf-8?B?Z3FybDdoZkFRWENRZ0RMSThaclBoZVR6OTdoNS9HU2drWEt6VDdhQ25aaW0x?=
 =?utf-8?B?OFk5UjE0WGIrY251VWMvbHdqZzZrUWFHWWRUOXZBaFl4dGxoanlQT1kvSXNK?=
 =?utf-8?B?MDgvaTFKcjVTdUJzcTJyY2c0R25tZjJVZkxlcDdGQlg2OGdMQ1F4ekVtZXhz?=
 =?utf-8?B?MmRTQWs5OVRMN2xrdUp6SFlXTzBRUENDMGF2SStUelRUOE5hU3lEV0ZYa0Fs?=
 =?utf-8?B?bTVia2hVZE8vNDVmblA5VkExaTMxUk5CYThyb1hVNVhFN21PajR5eVhMR3lp?=
 =?utf-8?B?NTYvRGxqcUZkanhhTkMrcFRGMUtNL1RVYUdqNnRaNkxOVmxNNnFBYVpNSjFp?=
 =?utf-8?B?UmNpU1lHYXAvQmgxWG1LZTN6UnlRRDRRbXFoeUR3aHl0Y21QVVhGMWRPSVFu?=
 =?utf-8?B?SzYxU1lGb1huTGtqTzRyUkJiNDIvbjU3VDE1SmRsVVorZ2RoYThOcFJ3SGNS?=
 =?utf-8?B?dTcwUWZCdGJJUUVrSVIySEM1bVVxRzM2UEl2UnpIa1J6WXp3UDhEQVh4TXRz?=
 =?utf-8?B?WDNKc3dhd3VjNWN1MG1Mem85R25Gc2NuYzRYejY0dTNSNHI5UTh6aGlNY2R1?=
 =?utf-8?B?UTJTSnBPZGtZbzc1SXNwVFdDKzl0ZGNWem40WFlwWXdoalFjaFJkOTRscklh?=
 =?utf-8?Q?1Sa5UBW/vS9SSNMcRXKuytBJU?=
X-OriginatorOrg: nxp.com
X-MS-Exchange-CrossTenant-Network-Message-Id: c782a274-8f10-4e1d-3f6b-08db237830ee
X-MS-Exchange-CrossTenant-AuthSource: AM7PR04MB7046.eurprd04.prod.outlook.com
X-MS-Exchange-CrossTenant-AuthAs: Internal
X-MS-Exchange-CrossTenant-OriginalArrivalTime: 13 Mar 2023 04:05:32.8587
 (UTC)
X-MS-Exchange-CrossTenant-FromEntityHeader: Hosted
X-MS-Exchange-CrossTenant-Id: 686ea1d3-bc2b-4c6f-a92c-d99c5c301635
X-MS-Exchange-CrossTenant-MailboxType: HOSTED
X-MS-Exchange-CrossTenant-UserPrincipalName: 8KVTb2SzrnoDgX/qDhXwRBDBEKfppjf0yyQ1JaOwOA6pjXKvQfcA6g4pyUWWSmm/xczJLjEhs27Ln1Ow/viiGg==
X-MS-Exchange-Transport-CrossTenantHeadersStamped: AM9PR04MB7650
Precedence: bulk
List-ID: <linux-kernel.vger.kernel.org>
X-Mailing-List: linux-kernel@vger.kernel.org

On Fri, 2023-03-10 at 08:47 -0600, Rob Herring wrote:
> It is preferred to use typed property access functions (i.e.
> of_property_read_<type> functions) rather than low-level
> of_get_property/of_find_property functions for reading properties. As
> part of this, convert of_get_property/of_find_property calls to the
> recently added of_property_present() helper when we just want to test
> for presence of a property and nothing more.
> 
> Signed-off-by: Rob Herring <robh@kernel.org>
> ---
>  drivers/gpu/drm/bridge/imx/imx8qxp-pixel-link.c | 2 +-

Reviewed-by: Liu Ying <victor.liu@nxp.com> # i.MX bridge

>  drivers/gpu/drm/drm_mipi_dsi.c                  | 2 +-
>  drivers/gpu/drm/msm/adreno/adreno_gpu.c         | 2 +-
>  drivers/gpu/drm/sun4i/sun4i_backend.c           | 2 +-
>  drivers/gpu/drm/sun4i/sun8i_mixer.c             | 2 +-
>  drivers/gpu/drm/vc4/vc4_hdmi.c                  | 2 +-
>  6 files changed, 6 insertions(+), 6 deletions(-)