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: 767753 Received: from mail-pf1-f178.google.com (mail-pf1-f178.google.com [209.85.210.178]) (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 DFE394EB49 for ; Mon, 29 Jan 2024 07:52:22 +0000 (UTC) Authentication-Results: smtp.subspace.kernel.org; arc=none smtp.client-ip=209.85.210.178 ARC-Seal: i=1; a=rsa-sha256; d=subspace.kernel.org; s=arc-20240116; t=1706514744; cv=none; b=j4RL3yVCAEFuCVr3fLWGnDl5Y+JXHPXRuQDStwY+D1Y7qvDm4WbKZwS6AohIgkmYYGtvfmx/CQCYY8qmUnZijo+JJn4Z7RvxP7fpOfsnD/DyK038YM1bkjkQs0XBvRtWOW8t86wd4NATY7Y2l3sh4nixJHy+zJ/4xuDXaBAeldc= 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.178 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-f178.google.com with SMTP id d2e1a72fcca58-6d9b37f4804so2380180b3a.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=PKpg3ESxJvK9X1t22t7uzKwkI3QmF+cmEB1x5I0EpgefORgoJR1Us0pt+F4FDEO1VX A9QaZ1GxMKBPoHede/FO6FjXMXa6TJ0w9jHHhnEd/vACKyOLO4ALFY+W01AwbQGmEFhB GJs2vvu9iNhkkzQAlbuJpd5TaFWjvAnVAtqIv2bPakj71IUli/cu8E0THdR52llIXsC3 +lnku8evvsz378m09VfXrlKdRTFA1++SUtCcC8JFKQ8ryTBesaRCwPLKUtocgmwZYOFE Re03IVf7eHGS47z/j2diOJnq2W4smjMGDBXGc1S4hbriT4ZB3s2S94pGZ/lO3lsREptP +E6g== X-Gm-Message-State: AOJu0Yzjvhc4VWt8BTXMSRhcJ1X4mv2m3AvxByYL7mX6CL07Lk/E5zTm Z2Wgpv7z7u9kdIgJXNQ4fkqGIAHYzeXI/GELeYvFi7tZy2GwUNPelqMUFMTgWA== 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: devicetree@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,