From mboxrd@z Thu Jan 1 00:00:00 1970 Received: from mail-wr1-f48.google.com (mail-wr1-f48.google.com [209.85.221.48]) (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 9B018420864 for ; Mon, 6 Jul 2026 12:55:39 +0000 (UTC) Authentication-Results: smtp.subspace.kernel.org; arc=none smtp.client-ip=209.85.221.48 ARC-Seal:i=1; a=rsa-sha256; d=subspace.kernel.org; s=arc-20240116; t=1783342542; cv=none; b=MwcUOAHzP9lmrKpC7Z95d47UQ0tMFMNYU0sI55Q8eM8boaJaRCQhJQqR03UEmh/kVFJi9PT6aabh2lXVJuPNU6/vUl/MwoIgS9vMc68wxDykPK/hy1KMI4cWiagp7FHjERQgE+aGeVv5RydKDZe4LQSvW8FxYxOX0RusXS4vG1o= ARC-Message-Signature:i=1; a=rsa-sha256; d=subspace.kernel.org; s=arc-20240116; t=1783342542; c=relaxed/simple; bh=bT+72sp86CIVIp/ogp9isiWyW9mszXa47KCTY8allM0=; h=Date:From:To:Cc:Subject:Message-ID:References:MIME-Version: Content-Type:Content-Disposition:In-Reply-To; b=eOw1FaAy7iJCflLJfMT+PEf3ThO6DyaZYxCPGKj0xcfO0wTWLbsWvKDgxC/dfvM9GHPBHRq9EnkXjR+XXlsz1Euh2TIn3n7UBOriD6nmyoElKZXWiNTcO+65ishOjF6fEZPASfu52hrPciHFw/N0tYi95upH1PTPi6KdulOiXQ4= 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=MMUj76yf; arc=none smtp.client-ip=209.85.221.48 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="MMUj76yf" Received: by mail-wr1-f48.google.com with SMTP id ffacd0b85a97d-4629051c9d1so1463766f8f.2 for ; Mon, 06 Jul 2026 05:55:39 -0700 (PDT) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=linaro.org; s=google; t=1783342538; x=1783947338; darn=vger.kernel.org; h=in-reply-to:content-transfer-encoding:content-disposition :content-type:mime-version:references:message-id:subject:cc:to:from :date:from:to:cc:subject:date:message-id:reply-to:content-type; bh=qF6R64HmcT1c2a3xEhtCiEIwl/af807a27WtqelA0FM=; b=MMUj76yfCLTiRtx5Ir6Rh2186YluuIkSQgyEMLvFSYXkczgMoFLGCMMnmaY0uY0SeD 8RY+ECBhdxzTS9AXNG+yU/uhfVmU/wl/Jja6sjNT/0FE4xnHcV7pHTCrE2F9y4Yy5Avu gFGQ/tjhuug7Bmqjp5N2FDc1jUmzD1a3OEfS7q61WxqMpHx6rQTpZs7UT9jxdAB1UF3w LNOtrTDXtbxh6ArBhE6o1IbivSudMvfia8AIT9/59E9jPPegqdzHBpIvgVgeDrgFn4eU lZYqNYcolOdOTXdfqWS+C0EVmeV/ZPnfrGBrCHGw81MEjwgHA1FCBFXzkCb4+uWdvw8J nZiA== X-Google-DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=1e100.net; s=20251104; t=1783342538; x=1783947338; h=in-reply-to:content-transfer-encoding:content-disposition :content-type:mime-version:references:message-id:subject:cc:to:from :date:x-gm-gg:x-gm-message-state:from:to:cc:subject:date:message-id :reply-to:content-type; bh=qF6R64HmcT1c2a3xEhtCiEIwl/af807a27WtqelA0FM=; b=DLWOCTKbIGMvX2mwmZo2DwYR8S5nlkz6hwLX5TcIBIGOTg6zTIFa0WsIA3hiwUcE5O v1OS+FY0UGJvBljscmai9+4DJ7JmA8JGuG26osbAccopwtryehEVTNlfj7mjmRa19Vd6 S+lemJ7bWt0ip3IFoekY1HsEY4G3XoNErhSl0Ffwdl5/bNfOUs0S8+kLz22+WAbyJlJm 3zn/SDZTUS4AqOv/g1ghtfZIHyqqnvWW5miW4+MGD4el42kBOfEWSygc8Jjt52hjWh4O 7KBQfBmHnY26ElGMEY9mMZWkFyfUr0T8CKoB3H+5e3jQBj273/MYwobrVX0zPTY2fbB5 IcEg== X-Forwarded-Encrypted: i=1; AHgh+RqoN/wKsfZlL8c3u8bM6HGTgI1ktqUTgLb7Jxalqxlp1DTwZsXHLPYxHFcUYfEDWvcvIAYbTpinkXn1JVo=@vger.kernel.org X-Gm-Message-State: AOJu0YxCGIe35S1dpknm9q0xoi33KuxA7Cx4gX9mBC0D1q58XXT2aI5d Fn5/9qFLjHbOqHbnJjSR6O2SjJAgvtwU/05ujHbZCWjnZ8FOH0ak+YmmaVvGNiJLC70HDuSIpU7 b+S3b/3o= X-Gm-Gg: AfdE7cnCQajcLANg4oJaFw3Pp7ktuISPlEDyGH0wSCF9b0WdUXAVS4vV1DZzFi4POlB lRQMAg2Wu5OWTvTpnT9iib1weeVxmE5rparCpzIK7/4ZHjFusvwFwNS6UF9XSzJ7ha+pV6gYz3Q C2OrpIRVnvXZQg9yexyRGJAu5d5Yr3JwxckCA0o2RhsGRzAASJGE9EfLiX881sUROz4jRUQlAiD AUavsWCxRG6ZzCZHiSRjZxy9p5PCzhf+QEtYEVaG89ZmjtNT8ObaWEJ3NVu+GaFOc08inTGm8sA O6H8fDPbqXPuQoPIq9pkLM8a24HGwo7lWSsoCKuJ7P8Z0f7dk/UgmA1yVkhN2ZZhJLQYvFIlvgI TQLyJyXWAAscWLyTJf9PRyw8UGdO9dYR0WqEyYRsuy4zVChDbguQp8RgEkkRmNYe1HWs6X12sPE jZydrfkg57cXbrivwqckmVHDqX X-Received: by 2002:a5d:564b:0:b0:470:2fb1:3dca with SMTP id ffacd0b85a97d-47aac5e17damr8744560f8f.30.1783342538035; Mon, 06 Jul 2026 05:55:38 -0700 (PDT) Received: from linaro.org ([2a02:2454:ff24:7210:1bb5:914d:5d25:d048]) by smtp.gmail.com with ESMTPSA id ffacd0b85a97d-47a9e4d8410sm24762493f8f.15.2026.07.06.05.55.37 (version=TLS1_3 cipher=TLS_AES_256_GCM_SHA384 bits=256/256); Mon, 06 Jul 2026 05:55:37 -0700 (PDT) Date: Mon, 6 Jul 2026 14:55:26 +0200 From: Stephan Gerhold To: Miquel Raynal Cc: Richard Weinberger , Vignesh Raghavendra , Rob Herring , Krzysztof Kozlowski , Conor Dooley , linux-mtd@lists.infradead.org, linux-arm-msm@vger.kernel.org, devicetree@vger.kernel.org, linux-kernel@vger.kernel.org, Manivannan Sadhasivam Subject: Re: [PATCH 0/4] mtd: rawnand: qcom: Add MDM9607 Message-ID: References: <20260608-qcom-nandc-mdm9607-v1-0-4639a0492274@linaro.org> <4kdjxrn3bxg7rhkdovidxv2b2f6evnknng7gjtbz7pahyqaakh@qkgxaz6xlav2> Precedence: bulk X-Mailing-List: linux-kernel@vger.kernel.org List-Id: List-Subscribe: List-Unsubscribe: MIME-Version: 1.0 Content-Type: text/plain; charset=iso-8859-1 Content-Disposition: inline Content-Transfer-Encoding: 8bit In-Reply-To: <4kdjxrn3bxg7rhkdovidxv2b2f6evnknng7gjtbz7pahyqaakh@qkgxaz6xlav2> Hi Miquèl, On Mon, Jun 29, 2026 at 05:46:57PM +0200, Manivannan Sadhasivam wrote: > On Mon, Jun 08, 2026 at 03:20:21PM +0200, Stephan Gerhold wrote: > > MDM9607 has QPIC v1.5 that supports the OP_PAGE_READ_ONFI_READ command, but > > is missing the rest of the hardware changes in QPIC v2. There is also only > > a single clock that can be controlled using the RPM firmware. Document and > > add the new qcom,mdm9607-nand compatible for this setup. > > > > Signed-off-by: Stephan Gerhold > > You could ammend patch 1's commit message with the information I shared in the > reply. But nevertheless: > > Reviewed-by: Manivannan Sadhasivam > Do you want me to resend the series with patch 1 commit message adjusted? There were no other changes requested as far as I can tell. I think the current commit message there is fine, especially if you add the Link: tag during applying. The extra context will be there. If you want me to resend, I would just replace the second paragraph in patch 1 with the following: --- On MDM9607 and other recent SoCs, the QPIC hardware requires 3 clocks (ahb, core, aon). However, the access to these clocks is restricted to the RPM firmware that controls the shared power resources for the whole SoC. The clocks cannot be controlled separately, there is only a single RPM_SMD_QPIC_CLK clock that implicitly enables all of the 3 clocks. The only exception to this are some IPQ* SoC that are not using RPM, there the clocks are directly controlled by the kernel via the clock controller (GCC). Require only one clock in the dt-bindings for MDM9607 to avoid having to define dummy clock entries. --- Thanks, Stephan