From patchwork Mon Jan 29 07:52:03 2024 Content-Type: text/plain; charset="utf-8" MIME-Version: 1.0 Content-Transfer-Encoding: 7bit X-Patchwork-Submitter: Manivannan Sadhasivam X-Patchwork-Id: 13535104 Received: from mail-pf1-f181.google.com (mail-pf1-f181.google.com [209.85.210.181]) (using TLSv1.2 with cipher ECDHE-RSA-AES128-GCM-SHA256 (128/128 bits)) (No client certificate requested) by smtp.subspace.kernel.org (Postfix) with ESMTPS id C60FF4EB23 for ; Mon, 29 Jan 2024 07:52:22 +0000 (UTC) Authentication-Results: smtp.subspace.kernel.org; arc=none smtp.client-ip=209.85.210.181 ARC-Seal: i=1; a=rsa-sha256; d=subspace.kernel.org; s=arc-20240116; t=1706514744; cv=none; b=V8pI0ZUcINd8JQyg4oL3L6mc+Jjb2YZvAQwK/BPIiwYAkdux8F/aRh44RqugBM5ftALtu/ihQHlfXcyPNrHJ+brdcKBEPPU6g16aV+puI3X8g/3KrUprnJjjx1V4SsRcUkhgFcPoCFJGwirpUNNIBywJLfl2/CUCqlfDJd1GMeE= ARC-Message-Signature: i=1; a=rsa-sha256; d=subspace.kernel.org; s=arc-20240116; t=1706514744; c=relaxed/simple; bh=zsixOtgoxOAaktuLOVRNS+8gFKTlnwz3zGLUZ3WnNGo=; h=From:Subject:Date:Message-Id:MIME-Version:Content-Type:To:Cc; b=RJ+jQZuOoa/0oCyalR+3i2kyCMNeUK1h104TTkcl5+0ckNHrwR1M+w0S1mW/QeyXJkKwjane32R8UFzMbZXpP4VoH6QTMuUlSo6HHhw4GFRp8WG+ngN+oXJ/t7/9foCG8UmdmQzxpEIFftGAisF7gxyd8qxxUEutU5Rk7JAsciE= ARC-Authentication-Results: i=1; smtp.subspace.kernel.org; dmarc=pass (p=none dis=none) header.from=linaro.org; spf=pass smtp.mailfrom=linaro.org; dkim=pass (2048-bit key) header.d=linaro.org header.i=@linaro.org header.b=jU+pRh35; arc=none smtp.client-ip=209.85.210.181 Authentication-Results: smtp.subspace.kernel.org; dmarc=pass (p=none dis=none) header.from=linaro.org Authentication-Results: smtp.subspace.kernel.org; spf=pass smtp.mailfrom=linaro.org Authentication-Results: smtp.subspace.kernel.org; dkim=pass (2048-bit key) header.d=linaro.org header.i=@linaro.org header.b="jU+pRh35" Received: by mail-pf1-f181.google.com with SMTP id d2e1a72fcca58-6de0ba30994so1238114b3a.1 for ; Sun, 28 Jan 2024 23:52:22 -0800 (PST) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=linaro.org; s=google; t=1706514742; x=1707119542; darn=vger.kernel.org; h=cc:to:content-transfer-encoding:mime-version:message-id:date :subject:from:from:to:cc:subject:date:message-id:reply-to; bh=fNguBtqcgFqP7NZKo6lpCS4yOVt7MJ3Er1hgEXdMUg8=; b=jU+pRh35erKYerFZJlVghxYfHwFqEALAxQuSONfv141q3V4zT16f4ij5FyiVY6/xKd 4dJWBUyOkD1c+feluQtL1EswghR12syxUjA0MkR+bTPOpWOkGUQHcp6WsGiYJ9gjp8qX 9b6igq0EwvySW6CLMBIJ5qp52xV5i1sSn2lhJghvsT0gFteRpxGIR7EpHJHlG/RMcSx2 g9M3ywj1uQ3McMMyFAiaW6Pq+eshV4KNHxSKQU4doG6tDFPfprp7x+6VHoWbCPMry5Ek tDrZ67qJ5M04JA6Fs013OkSjs0E/yWIEt3hPp+ZTHuIYSSJSTd2/ZEFuvafkxtNzOsYN 8iNw== X-Google-DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=1e100.net; s=20230601; t=1706514742; x=1707119542; h=cc:to:content-transfer-encoding:mime-version:message-id:date :subject:from:x-gm-message-state:from:to:cc:subject:date:message-id :reply-to; bh=fNguBtqcgFqP7NZKo6lpCS4yOVt7MJ3Er1hgEXdMUg8=; b=DmbMd2xteAfC/9gtxw5HvSNZ5XF2JTbLDMNGD+JLhcV0xvqojicQ/+5tbSEKxxHEL7 452m3t7a71Rd6wzJGi+Hd1RZyxnAsCSXJh7TVEiS3rmnlVRnuOZYvcul5Yo8Vv1wWlQy cMZPjXyVQGci8SaPG5eNDBAV6x3b0rgVxjeiRTfvN9CGMvcjL4ft8NKAqEAvco6Q2umN c5UXNbzKGJNRb0HTTe3GLqZdR9swo5eXgbPZZNiWJrv7qCp0LcZdcAV46hoP9oxZ94rY 8muhbVKrKSfoqqTH6tILzvBUrIFFvHF9wrHrA4iQObcXPlYE9MceSqidsVAOijx2wLhF hoeA== X-Gm-Message-State: AOJu0YwNUe/x3LcX6R/QIHtuECTbZFKe7y7K344humNR+ET0CCI/xxet UFYYCsuWC5rJtPsquLtAxF8KYDbo6TZZ4E/hwG4q3unSMjbaPFpvD4TfXLaovg== X-Google-Smtp-Source: AGHT+IGLj5HGoGuy0l3yI/NV57KQmeoJRLi2YW0sBBeEqN3seWdetPb4xNwGFcfa76uqzDgLcB640w== X-Received: by 2002:a62:b60d:0:b0:6de:1095:4b9b with SMTP id j13-20020a62b60d000000b006de10954b9bmr3538877pff.7.1706514742083; Sun, 28 Jan 2024 23:52:22 -0800 (PST) Received: from [127.0.1.1] ([117.193.214.109]) by smtp.gmail.com with ESMTPSA id t19-20020a62d153000000b006dddd685bbesm5467329pfl.122.2024.01.28.23.52.15 (version=TLS1_3 cipher=TLS_AES_256_GCM_SHA384 bits=256/256); Sun, 28 Jan 2024 23:52:21 -0800 (PST) From: Manivannan Sadhasivam Subject: [PATCH 0/3] ufs: qcom: Mark "core_reset" as required property Date: Mon, 29 Jan 2024 13:22:03 +0530 Message-Id: <20240129-ufs-core-reset-fix-v1-0-7ac628aa735f@linaro.org> Precedence: bulk X-Mailing-List: linux-arm-msm@vger.kernel.org List-Id: List-Subscribe: List-Unsubscribe: MIME-Version: 1.0 X-B4-Tracking: v=1; b=H4sIACNZt2UC/x2MSQqAMAwAvyI5G2hLKehXxINLqrm0kqgIxb9bP M7ATAElYVLomwJCNyvnVMG2DSz7lDZCXiuDM84b6zq8ouKShVBI6cTID9p59dPsQojBQw0Poar /6TC+7wdFQJPqZAAAAA== To: Bjorn Andersson , Konrad Dybcio , Alim Akhtar , Avri Altman , Bart Van Assche , Rob Herring , Krzysztof Kozlowski , Conor Dooley , Andy Gross , Andy Gross , "James E.J. Bottomley" , "Martin K. Petersen" Cc: linux-arm-msm@vger.kernel.org, linux-scsi@vger.kernel.org, devicetree@vger.kernel.org, linux-kernel@vger.kernel.org, Manivannan Sadhasivam X-Mailer: b4 0.12.4 X-Developer-Signature: v=1; a=openpgp-sha256; l=1197; i=manivannan.sadhasivam@linaro.org; h=from:subject:message-id; bh=zsixOtgoxOAaktuLOVRNS+8gFKTlnwz3zGLUZ3WnNGo=; b=owEBbQGS/pANAwAKAVWfEeb+kc71AcsmYgBlt1kukOiTQDGcIeoelZVUHLMR2QZqqCSV88qj/ CmxdYiFC9+JATMEAAEKAB0WIQRnpUMqgUjL2KRYJ5dVnxHm/pHO9QUCZbdZLgAKCRBVnxHm/pHO 9Y1IB/0ePDpYszqZJSgILa+x2JyZQMEuYoiet03wzEowK3tUc5prM87SkXDuBsTGDGXuU6ircTY BOqsSZnJzbZGCQT6a18kHAUKdd9hRzqUjNWFARhLdAydYDkMZxViId3LmeRhfZ+0RjyfqMPTtv3 drA+inetP/z0Ly7WZhLNrmsDbId9xkp0NbhYuXgfx+JlrT0Jy0yYMXe5Vin3piNw/qLQ5CtwR+s +w/n2c6Fvrbw0X8RSBiR9PaM2OPe2zhuDbR5XIatxo0FZOBDxKR9hIAKk43DhkR6KV6oZfKOKPw AdhOnaLI7cYEMBVW9CeKsC8CIM+aZBrycpik9ckgYg1pfXbY X-Developer-Key: i=manivannan.sadhasivam@linaro.org; a=openpgp; fpr=C668AEC3C3188E4C611465E7488550E901166008 As concluded in [1], mark the "core_reset" (UFS host controller reset from GCC) as required in the binding and also add the missing property in MSM8996 DT. However, to keep backwards compatibility with old DT, the driver cannot make it required. So it is still an optional property for the driver, but a comment has been added to clarify this. - Mani [1] https://lore.kernel.org/linux-scsi/190651ad-6aeb-69eb-89c5-ed18221b5a7a@quicinc.com/ Signed-off-by: Manivannan Sadhasivam --- Manivannan Sadhasivam (3): dt-bindings: ufs: qcom: Make reset properties as required scsi: ufs: qcom: Clarify the comment of core_reset property arm64: dts: qcom: msm8996: Add missing UFS host controller reset Documentation/devicetree/bindings/ufs/qcom,ufs.yaml | 2 ++ arch/arm64/boot/dts/qcom/msm8996.dtsi | 2 ++ drivers/ufs/host/ufs-qcom.c | 6 +++++- 3 files changed, 9 insertions(+), 1 deletion(-) --- base-commit: 6613476e225e090cc9aad49be7fa504e290dd33d change-id: 20240129-ufs-core-reset-fix-1bd4ab266f64 Best regards,