From mboxrd@z Thu Jan 1 00:00:00 1970 Received: from mail-wm1-f45.google.com (mail-wm1-f45.google.com [209.85.128.45]) (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 4790533B97B for ; Thu, 30 Oct 2025 11:11:44 +0000 (UTC) Authentication-Results: smtp.subspace.kernel.org; arc=none smtp.client-ip=209.85.128.45 ARC-Seal:i=1; a=rsa-sha256; d=subspace.kernel.org; s=arc-20240116; t=1761822705; cv=none; b=bHcsNTPm9eCLt+kzYXstfJE2pnGEMoBrS+bKaCUZOywg70GmQM9uI1Vx/+w0LGhAvpcmH3pJCxSZjuyyU4kr9PyFE+ZZQPv6tMPHAwpg9TeQJHvAaLE6pgbfzz2Ii501JZiZB5rjXiI2deHZ1cAfi8GsZQ8crY5AnJfD8jjzoHU= ARC-Message-Signature:i=1; a=rsa-sha256; d=subspace.kernel.org; s=arc-20240116; t=1761822705; c=relaxed/simple; bh=RmUR81Vp+1u3hIuZWAFJx55ZbSzWgS/AvpxG9kE/93k=; h=Message-ID:Date:From:To:Cc:Subject:References:MIME-Version: Content-Type:Content-Disposition:In-Reply-To; b=oIpXUkAkwpchFj124lJpdokgEkQLr6aFNysN6kIM7S3UEZKQWja+qcyZy5Zq6dVWG+KhRb2UL7FF8nYJ5ZW31w3PzuDMumIBqdphapkLoWCOgoKgK9lx2S4ZvyqW9Me+x2h4SOfgtizE8g6zNKx6Lmt150DELzPb+sn2EcOvgxY= ARC-Authentication-Results:i=1; smtp.subspace.kernel.org; dmarc=pass (p=none dis=none) header.from=gmail.com; spf=pass smtp.mailfrom=gmail.com; dkim=pass (2048-bit key) header.d=gmail.com header.i=@gmail.com header.b=QAJwvwqU; arc=none smtp.client-ip=209.85.128.45 Authentication-Results: smtp.subspace.kernel.org; dmarc=pass (p=none dis=none) header.from=gmail.com Authentication-Results: smtp.subspace.kernel.org; spf=pass smtp.mailfrom=gmail.com Authentication-Results: smtp.subspace.kernel.org; dkim=pass (2048-bit key) header.d=gmail.com header.i=@gmail.com header.b="QAJwvwqU" Received: by mail-wm1-f45.google.com with SMTP id 5b1f17b1804b1-4770c2cd96fso6837445e9.3 for ; Thu, 30 Oct 2025 04:11:44 -0700 (PDT) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=gmail.com; s=20230601; t=1761822703; x=1762427503; darn=vger.kernel.org; h=in-reply-to:content-disposition:mime-version:references:subject:cc :to:from:date:message-id:from:to:cc:subject:date:message-id:reply-to; bh=f0yQfjJh3fnP4ZqkUoVVr4XyZOI4wGNOEVBXCVxV97Q=; b=QAJwvwqUnYRoTyfxcGAdOFy65vlKoGrDd6UiwTELdGIVSnQN4YMlI+lbpthUhACL/i TXgVJWYrHIuJdLXI+TPGDLCnPUOD6F3PQ7YWWyQn28d2rRKI25aQnF9uODpvbkKlMhWG NqwwTpU/3Y1JzB+u+sk9Dc6iHGWgVGwLTwovHIAq7FCu7Wv9yT4r0g4oNbE1AuHs+cfD d7QcWaFwAOZSyRH0Uar+wjUkN4mnsm9I0jQe//2NFjMg8QU2b5po883KIU8SLqhSoB6M SvfBBQO4UX8LUHw7E8mLc3ouYzxjFngCxJuF7EDCeJC9Zdi6X8IUR3PMOa2r6+T2glpi XsJg== X-Google-DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=1e100.net; s=20230601; t=1761822703; x=1762427503; h=in-reply-to:content-disposition:mime-version:references:subject:cc :to:from:date:message-id:x-gm-message-state:from:to:cc:subject:date :message-id:reply-to; bh=f0yQfjJh3fnP4ZqkUoVVr4XyZOI4wGNOEVBXCVxV97Q=; b=R8bcYEKLEh3I5gv+tid7UmoVoeEijyTy5endVQcg9erehB/yan9aZR40iytWCeVhPK pKqxbqjcBvGLPtRd4GOfjvLxTw74SEVVNilNIWFjlysvfjIwJIpEX6bGLb4uYtp0u4xz F7c2kYImt44wMi894d6r1ReOtNMXhG4FQ8vSpOdmx4fMUvvqu3QX3aonlRnveHperHRN 1KMNuKMF94HxJLzTN37MbrkD3sPumusuYrbS93CQ7DOdn5lzbUUSmiq/Lqrvm5fAMCxG VE1W9Z2AqzzRP3nyPtxYt/tGWsxPq3/lLFAmWceFevUV7N5BwJB175aAD0iJq99xSupV +qaQ== X-Forwarded-Encrypted: i=1; AJvYcCXKn5uAv7VutBAbk2ihcVj+a0TvwMZiSMQ6Ctbp/CubQO0+FFpgRmDb2aCdzVr0crBPfxxkqzg20JMU1e/5@vger.kernel.org X-Gm-Message-State: AOJu0YwPs52hvbXvVSuLofpLguqpdusSn2HwZwaU0nQXEre1yokJOMiv DQF+pVvI3l/g62hgfvgnRqtCyPfSxvgnpQ/0TVvVVZ1yJXqRFC3o48CL X-Gm-Gg: ASbGncu/vzC6acu+Hn+4n8SkgwOdEbTBFgvbv1h0saNLaWBqXgh+TaIbbRSsW5sq6+3 ydM2KOjU/k8MDrxkIeMQvbySQCTtdKp/yC+KmMxb+FUukppWc2w4gyfhqpNeoBXo7MTcQapUUXn Fu3o+X5WLWz+8O1vqw5DntytPDyzWK0CylHTAwhMSP4mjPoOiY69MA94Ce67GejNo/l1ymPksK6 FD2Z+QKf5T+iNBg0FxzPri3iVPJ/ec0UqX86qtpgCSHOgfisk08pxxSrJEOWy5DpUhlX8RvWyzJ zFXunErXDh8UxzlB6dMdGgH+RiaHQXIdD/bniQlu5O3NxW8JUAjSOIMAOvbuYV/cD6GDUdp6Pmb jKA6UgNcE3+Q1WUVFz77YslulR6lmG51zwgD4WnTx3Z9Q6D4UMzMxNpYbRRXwfvBJtHe0FUB2kG VF1X/55YuvcQGr0e/dFeF0hY61Qjix X-Google-Smtp-Source: AGHT+IHxhf8stS61kYSKITBJZM63qOEpp0Q9HeczTOxUA5TPtzWHnwZWNRGkjWICXEC9IRZOfnm8Dw== X-Received: by 2002:a05:600c:19d3:b0:471:c72:c807 with SMTP id 5b1f17b1804b1-47726005eaemr24937345e9.22.1761822702490; Thu, 30 Oct 2025 04:11:42 -0700 (PDT) Received: from Ansuel-XPS. (93-34-90-37.ip49.fastwebnet.it. [93.34.90.37]) by smtp.gmail.com with ESMTPSA id 5b1f17b1804b1-47728999a4bsm33080805e9.2.2025.10.30.04.11.41 (version=TLS1_3 cipher=TLS_AES_256_GCM_SHA384 bits=256/256); Thu, 30 Oct 2025 04:11:42 -0700 (PDT) Message-ID: <690347ee.050a0220.21ee29.8092@mx.google.com> X-Google-Original-Message-ID: Date: Thu, 30 Oct 2025 12:11:40 +0100 From: Christian Marangi To: Konrad Dybcio Cc: Ilia Lin , "Rafael J. Wysocki" , Viresh Kumar , Bjorn Andersson , Konrad Dybcio , Raag Jadav , Arnd Bergmann , Andy Shevchenko , linux-arm-msm@vger.kernel.org, linux-pm@vger.kernel.org, linux-kernel@vger.kernel.org Subject: Re: [PATCH 3/3] cpufreq: qcom-nvmem: add compatible fallback for ipq806x for no SMEM References: <20251029133323.24565-1-ansuelsmth@gmail.com> <20251029133323.24565-3-ansuelsmth@gmail.com> <9343108b-7796-4783-afb1-afe758f78414@oss.qualcomm.com> <69033de7.050a0220.34ea6.780b@mx.google.com> Precedence: bulk X-Mailing-List: linux-arm-msm@vger.kernel.org List-Id: List-Subscribe: List-Unsubscribe: MIME-Version: 1.0 Content-Type: text/plain; charset=us-ascii Content-Disposition: inline In-Reply-To: On Thu, Oct 30, 2025 at 11:54:41AM +0100, Konrad Dybcio wrote: > On 10/30/25 11:28 AM, Christian Marangi wrote: > > On Thu, Oct 30, 2025 at 09:56:24AM +0100, Konrad Dybcio wrote: > >> On 10/29/25 2:33 PM, Christian Marangi wrote: > >>> On some IPQ806x SoC SMEM might be not initialized by SBL. This is the > >>> case for some Google devices (the OnHub family) that can't make use of > >>> SMEM to detect the SoC ID. > >> > >> Oh this is (the unpleasant kind of ) interesting.. Is there any sort > >> of uboot/kernel tree for these machines available? > >> > > > > There is some sort of source but quite confusing. From the info they use > > coreboot and chromeos. > > > > Looking at the source they comment everything related to SMEM > > (confirming the fact that they actually don't init it) > > > > [1] https://chromium.googlesource.com/chromiumos/platform/depthcharge/+/refs/heads/firmware-storm-6315.B/src/board/storm > > [2] https://chromium.googlesource.com/chromiumos/third_party/coreboot/+/firmware-storm-6315.B > > Hmm odd.. > > The patch itself looks mostly good, although you e.g. assign > qcom,ipq8069 -> QCOM_ID_IPQ8065 even though QCOM_ID_IPQ8069 exists > > This doesn't cause any difference in behavior within this driver but > looks slightly funky > Well yes I did to simplify the logic. > Should we perhaps do this patching in smem.c instead, in case other > drivers try to retrieve the ID in the future? > Well we would hide the fact that SMEM is not available. SMEM gives precise info while this operates on some kind of fallback measure. If someone wrongly sets the compatible and use the most generic one (qcom,ipq8064) then we would parse the wrong ID. Also looking at the user of those API it's really just cpufreq and apss for ipq60xx so maybe not worth? (we would also have to add additional logic to fallback only for some specific SoC) -- Ansuel