From mboxrd@z Thu Jan 1 00:00:00 1970 Received: from mx0a-0031df01.pphosted.com (mx0a-0031df01.pphosted.com [205.220.168.131]) (using TLSv1.2 with cipher ECDHE-RSA-AES256-GCM-SHA384 (256/256 bits)) (No client certificate requested) by smtp.subspace.kernel.org (Postfix) with ESMTPS id 0D73E1D63C7 for ; Mon, 10 Nov 2025 14:41:46 +0000 (UTC) Authentication-Results: smtp.subspace.kernel.org; arc=none smtp.client-ip=205.220.168.131 ARC-Seal:i=1; a=rsa-sha256; d=subspace.kernel.org; s=arc-20240116; t=1762785708; cv=none; b=YbUl2g+OpdErOP41Z+zBYyQ2ZM3JYVim6f00jJJvpl9+YnhhG6wPJu6aKDFYYw9GWFKR3qwcbxmGCF2NPJMNbd0w323WmIUnT8sZ/3cAKu/8phhvfX9Sc4O9QvTuxWGpaJighIKgeGGRP5/bKWhQGWA9YoJoQIak0a12/tmstu0= ARC-Message-Signature:i=1; a=rsa-sha256; d=subspace.kernel.org; s=arc-20240116; t=1762785708; c=relaxed/simple; bh=EGsoQ001UBEf7yxavI8ZABTjuEBgxU+e7Nmf8KHADdI=; h=Message-ID:Date:MIME-Version:Subject:To:Cc:References:From: In-Reply-To:Content-Type; b=dJ5dFXieEm5K9/WTa6gwoaiVrrHvE1aMUCBBBrQG14Az/L1iEUxb6/UxGrtcnAXcjjQWiuJAAP95Z3hCedj4QC+GmlX49ah1mmY8JbqRg8qOmV5aWPkdDXTKdT763E8qJ3tCJa/b3esZ/SbAMVezl4Y5mvmSKiMag8w4iM8Q4Bg= ARC-Authentication-Results:i=1; smtp.subspace.kernel.org; dmarc=pass (p=reject dis=none) header.from=oss.qualcomm.com; spf=pass smtp.mailfrom=oss.qualcomm.com; dkim=pass (2048-bit key) header.d=qualcomm.com header.i=@qualcomm.com header.b=bfILjCMM; dkim=pass (2048-bit key) header.d=oss.qualcomm.com header.i=@oss.qualcomm.com header.b=LNDHg5NV; arc=none smtp.client-ip=205.220.168.131 Authentication-Results: smtp.subspace.kernel.org; dmarc=pass (p=reject dis=none) header.from=oss.qualcomm.com Authentication-Results: smtp.subspace.kernel.org; spf=pass smtp.mailfrom=oss.qualcomm.com Authentication-Results: smtp.subspace.kernel.org; dkim=pass (2048-bit key) header.d=qualcomm.com header.i=@qualcomm.com header.b="bfILjCMM"; dkim=pass (2048-bit key) header.d=oss.qualcomm.com header.i=@oss.qualcomm.com header.b="LNDHg5NV" Received: from pps.filterd (m0279864.ppops.net [127.0.0.1]) by mx0a-0031df01.pphosted.com (8.18.1.11/8.18.1.11) with ESMTP id 5AACOJnM2407325 for ; Mon, 10 Nov 2025 14:41:46 GMT DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=qualcomm.com; h= cc:content-transfer-encoding:content-type:date:from:in-reply-to :message-id:mime-version:references:subject:to; s=qcppdkim1; bh= HYPBMmPMrl74VcBNfaZK6P0Th/jmEX2D4ZwvG4S3pO4=; b=bfILjCMMjXR9ll5x cD6jtvxG6fD6T52Kv9T5eFgtaACKtgd2pxJrJYkaEhSdnaM0G+6qMOnEyxVw2yen nmcvAkwEwJggDXfQcRGcs/MT52ntoYywDMTghWYOAfQDTvazF2IIX6muvRRypGTt x8vgomIVGqrxB59uYFQIxGMx3TZWKb3fmh+SmVuUNocYD5V7cwCQsPZaZs5F2/RB z5OlpnaqX899nLntx4ApLqe42zY1vRkWgA0SMYsI+iKlLn2JwNGALeD+zMx57m3c hyhkTX25wcuYf3cNxj5vL8BW4eS4tRJhRvSUdnKFh8C33GUfWnHecHjyk0tCeDsj yWYvEw== Received: from mail-pl1-f200.google.com (mail-pl1-f200.google.com [209.85.214.200]) by mx0a-0031df01.pphosted.com (PPS) with ESMTPS id 4abatd9eyx-1 (version=TLSv1.3 cipher=TLS_AES_128_GCM_SHA256 bits=128 verify=NOT) for ; Mon, 10 Nov 2025 14:41:46 +0000 (GMT) Received: by mail-pl1-f200.google.com with SMTP id d9443c01a7336-297b35951b7so53905875ad.3 for ; Mon, 10 Nov 2025 06:41:46 -0800 (PST) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=oss.qualcomm.com; s=google; t=1762785706; x=1763390506; darn=vger.kernel.org; h=content-transfer-encoding:in-reply-to:from:references:cc:to :content-language:subject:user-agent:mime-version:date:message-id :from:to:cc:subject:date:message-id:reply-to; bh=HYPBMmPMrl74VcBNfaZK6P0Th/jmEX2D4ZwvG4S3pO4=; b=LNDHg5NVywSXuikZeB0U6w1nob8Hm8N8OBTwhUVJwsqVQyOmz/x4k319Xdl57EHMbE VBG0oO5jiROiPdUjPwo3pvzmQ8XoTI9UqR+DmIbnvohAL/H4f+m/bLsTjd5T27FutTq0 eJ7tjTPNqZcG2Tkco3IsRZyiyeqzYyD5Qkm5fz2Cpd8YwUpXv0dIlvJZveESHLJwPhUP SUTjUHYly6RkVwUiKN9b8J5H220wNibbnIKqxvc9hxOf5ShDAeFkNpYqVjAf2WXFVLYo fZ2Ogo1OisN+ATuni/6Eb0NPVdGBv6aUN2hYUBIqVgOCFtqHShHbukcJS7FkeV1q44Rm 7C9Q== X-Google-DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=1e100.net; s=20230601; t=1762785706; x=1763390506; h=content-transfer-encoding:in-reply-to:from:references:cc:to :content-language:subject:user-agent:mime-version:date:message-id :x-gm-gg:x-gm-message-state:from:to:cc:subject:date:message-id :reply-to; bh=HYPBMmPMrl74VcBNfaZK6P0Th/jmEX2D4ZwvG4S3pO4=; b=eKHv865AyonKsFlVRhWs1qVKRdJJ7gSswpnIdqnmr3cXdc2KqNFhmhDxvGJcFuqjAv 5wIN8aZ1kxeGN4+FpuvWA1OnQwjtJ1gh6hJ8Ibd/SoJXVIVNjkgu+arv5++vhrgaEHkM GLfA3zL9AZ+yTdXX7plKsy/99Mc4JLY7OwsLptacJ0vAMMiHmo9MciVsyPQPHb8GtnqV y/STJRLDcp4PtJ0Q0u3jd41yWlw9ZpK9/8y3s71xjwPMGe5gspaXpX0F6vU+3UcWTDUf cnI4HpwWxHME+Ts205/GRuzfdEz745NtQikzvpUkBDDhUxdNIALebOEjrlSTv4Cb7Nkt 0pWA== X-Forwarded-Encrypted: i=1; AJvYcCWqTMjavHWqId1/aXKNyQJIKTu7PfAfEpGDU/VasH8QsP6y1dIqW9QxEiyJg8GHWDvLfkvx83oWUw==@vger.kernel.org X-Gm-Message-State: AOJu0YyYUe5ZoAb5phdcgMYLj1Py3xPEpP+AqMs00SBdpOS+PHKE98o0 PQPCqVIXrCd0mgTIjblHOUFyDrphUlmIg7hMjuLMFnGFTOwCD754sv52pk9YlIGYq2u4ldq5ZtM X42ryAOK6YHF4PoOESsFf/huobgU5gAJufiH1W2BEtRc85tfvL4jz/jRbDgnZGQ== X-Gm-Gg: ASbGncvoTe7XMKwfIZ2YeXAvOWIUZdlwqT8nUmjdpj3vpoj9UOF6eWdjS31Pr8+Wf59 NFWdZC+kNelcBFk9tXIztKf/ie4Auc40g2oO679dWlO4stKeR/64hL7h7A3s0aoRAN24ahGNajH 9HqqkrzWhAxA22V1PxLOlcG2vCD0q0qyjbk4+/qXj7B6JCkNsWuLWFb/hncVUo809Ro8JUYtBeN Mox7U9dAwH7aJtYBd0eSRnVMZpPKhsTO2+mVONoybXYnKYduZ062kH+O3/hnCWXTXUV4r6gxgoW BkBoVeBkEPpOkI2BdAdDeDUdxA+HTimeTTYsbhH60DcIQUS8KQfx9Mc9fXgWrBD06n0ngyQmsY4 1+QoCzwEgECOs9Y60+wD0spvPWB5807xC X-Received: by 2002:a17:902:da4b:b0:27e:f018:d2fb with SMTP id d9443c01a7336-297e540dbf8mr103670365ad.6.1762785705561; Mon, 10 Nov 2025 06:41:45 -0800 (PST) X-Google-Smtp-Source: AGHT+IEzUGI3gfRP1roHcIKhz5BZsgRD+n11EwhNGgQ9IIMPUd06GGUlB6wwr56gPGTWX6jx/IrBZg== X-Received: by 2002:a17:902:da4b:b0:27e:f018:d2fb with SMTP id d9443c01a7336-297e540dbf8mr103669785ad.6.1762785705031; Mon, 10 Nov 2025 06:41:45 -0800 (PST) Received: from [10.219.57.23] ([202.46.23.19]) by smtp.gmail.com with ESMTPSA id 98e67ed59e1d1-34370ec58d9sm7675580a91.6.2025.11.10.06.41.35 (version=TLS1_3 cipher=TLS_AES_128_GCM_SHA256 bits=128/128); Mon, 10 Nov 2025 06:41:44 -0800 (PST) Message-ID: <821358a8-64ed-46c9-3fd4-538e194fa5b3@oss.qualcomm.com> Date: Mon, 10 Nov 2025 20:11:34 +0530 Precedence: bulk X-Mailing-List: linux-pm@vger.kernel.org List-Id: List-Subscribe: List-Unsubscribe: MIME-Version: 1.0 User-Agent: Mozilla/5.0 (Windows NT 10.0; Win64; x64; rv:102.0) Gecko/20100101 Thunderbird/102.5.1 Subject: Re: [PATCH v17 07/12] firmware: psci: Implement vendor-specific resets as reboot-mode Content-Language: en-US To: Kathiravan Thirumoorthy , Bartosz Golaszewski , Bjorn Andersson , Sebastian Reichel , Rob Herring , Sudeep Holla , Souvik Chakravarty , Krzysztof Kozlowski , Conor Dooley , Andy Yan , Mark Rutland , Lorenzo Pieralisi , Arnd Bergmann , Konrad Dybcio , cros-qcom-dts-watchers@chromium.org, Vinod Koul , Catalin Marinas , Will Deacon , Florian Fainelli , Moritz Fischer , John Stultz , Matthias Brugger , Krzysztof Kozlowski Cc: Dmitry Baryshkov , Mukesh Ojha , Stephen Boyd , Andre Draszik , linux-pm@vger.kernel.org, linux-kernel@vger.kernel.org, devicetree@vger.kernel.org, linux-arm-kernel@lists.infradead.org, linux-arm-msm@vger.kernel.org, Elliot Berman , Xin Liu , Srinivas Kandagatla , Umang Chheda , Nirmesh Kumar Singh References: <20251109-arm-psci-system_reset2-vendor-reboots-v17-0-46e085bca4cc@oss.qualcomm.com> <20251109-arm-psci-system_reset2-vendor-reboots-v17-7-46e085bca4cc@oss.qualcomm.com> From: Shivendra Pratap In-Reply-To: Content-Type: text/plain; charset=UTF-8 Content-Transfer-Encoding: 8bit X-Proofpoint-GUID: oS4MDuwWfbVQhJOLUlUhVVz_WgNuDXLg X-Authority-Analysis: v=2.4 cv=eZowvrEH c=1 sm=1 tr=0 ts=6911f9aa cx=c_pps a=IZJwPbhc+fLeJZngyXXI0A==:117 a=j4ogTh8yFefVWWEFDRgCtg==:17 a=IkcTkHD0fZMA:10 a=6UeiqGixMTsA:10 a=s4-Qcg_JpJYA:10 a=VkNPw1HP01LnGYTKEx00:22 a=yV86LA_xD0RmO51t7RcA:9 a=3ZKOabzyN94A:10 a=QEXdDO2ut3YA:10 a=uG9DUKGECoFWVXl0Dc02:22 X-Proofpoint-ORIG-GUID: oS4MDuwWfbVQhJOLUlUhVVz_WgNuDXLg X-Proofpoint-Spam-Details-Enc: AW1haW4tMjUxMTEwMDEyNyBTYWx0ZWRfX2kUFJS6yz4z5 4/3bFrTvQylHgilwB/8luWMB8e7/5fk8QfuHq2GpwbQuiZpZVXa8+iX4bf7hXjOaMXAlRcHqcQZ V3lN07BiM9HPJNIhRliY8XNwBkGVincKMbtaI0E1/BbAK3Po0wwxT10+dgOeKNrNgZIHAVitf8t 6S1zi9/7loLEN+pnnRhEHEJDB/+Mn2W8MzagZ1R1JZJMDHN3YQqCnfCjqNQw8SqQ38ZZabw1Dkz yclRd+Az9z7O/dcXwkCeA89pyYVgDk93aLhIoiJEBtPnNqVBQdDpUOUU+Zu9AdV+42qbIDSBdwh dgkrBbka+GeFE9q5pGrgRxoMSKCIkYS7c+a4jl467uuCF2zkCFYY1AAy55Og/0MycapYKHjmzII xuu3VE8WUGIeKd/z4fS91cw+ef7rqg== X-Proofpoint-Virus-Version: vendor=baseguard engine=ICAP:2.0.293,Aquarius:18.0.1121,Hydra:6.1.9,FMLib:17.12.100.49 definitions=2025-11-10_05,2025-11-10_01,2025-10-01_01 X-Proofpoint-Spam-Details: rule=outbound_notspam policy=outbound score=0 suspectscore=0 spamscore=0 adultscore=0 impostorscore=0 lowpriorityscore=0 clxscore=1015 malwarescore=0 priorityscore=1501 bulkscore=0 phishscore=0 classifier=typeunknown authscore=0 authtc= authcc= route=outbound adjust=0 reason=mlx scancount=1 engine=8.22.0-2510240001 definitions=main-2511100127 On 11/10/2025 10:10 AM, Kathiravan Thirumoorthy wrote: > > On 11/9/2025 8:07 PM, Shivendra Pratap wrote: >> +static int __init psci_init_vendor_reset(void) >> +{ >> +    struct reboot_mode_driver *reboot; >> +    struct device_node *psci_np; >> +    struct device_node *np; > > > We can take advantage of __cleanup() attribute to simply the code paths. > > Declare the variables like below > >     struct device_node *psci_np __free(device_node) = NULL; >     struct device_node *np __free(device_node) = NULL; > > and get rid of the explicit of_node_put(). > > I think, we can take up this an improvement once this series landed. But if you happen to respin to address other issues, please take care of this as well. Let me evaluate this for next respin. thanks, Shivendra