From mboxrd@z Thu Jan 1 00:00:00 1970 Received: from mail-lf1-f49.google.com (mail-lf1-f49.google.com [209.85.167.49]) (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 5E26D152172 for ; Tue, 7 May 2024 11:43:17 +0000 (UTC) Authentication-Results: smtp.subspace.kernel.org; arc=none smtp.client-ip=209.85.167.49 ARC-Seal:i=1; a=rsa-sha256; d=subspace.kernel.org; s=arc-20240116; t=1715082198; cv=none; b=Nd++wGjU1eRWoeb8qBVo/87BNj/58ELDzzZPtKTSA/UFOSA6xp9Zcq0Iubv7g7XzW6ZI/WwxcD+Tm0nTw/WeKeIR+LvyMgpiwOgyDIlj9MwJEigP+UR6iP8z1F9kk5UIHR8cs0siu0RlY7BdRa1sxrfRCPpqx54jkDCk4mBG7G4= ARC-Message-Signature:i=1; a=rsa-sha256; d=subspace.kernel.org; s=arc-20240116; t=1715082198; c=relaxed/simple; bh=nJmQfa/G63amUHNoyNPPruCS1yE59OplOKYnfKXgiM8=; h=Message-ID:Date:MIME-Version:Subject:To:Cc:References:From: In-Reply-To:Content-Type; b=bmN00kZveyIFnCFumyUKOuuAorZewM6Z1Bh6XjB7q1G3EfZSnDKibxEF+802heSKxIgLbufRhqkLFi+NjSii3x5QxNvIpQuEL0njZcm8w/+xE5DdQ4CizmAVFQEcP/BJNkb44+BjSAOdbGviVjZDWiia13B1TXWwsPgSjLrXfh0= 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=YX7+LXtT; arc=none smtp.client-ip=209.85.167.49 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="YX7+LXtT" Received: by mail-lf1-f49.google.com with SMTP id 2adb3069b0e04-51f17ac14daso3272982e87.1 for ; Tue, 07 May 2024 04:43:17 -0700 (PDT) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=linaro.org; s=google; t=1715082195; x=1715686995; darn=vger.kernel.org; h=content-transfer-encoding:in-reply-to:from:content-language :references:cc:to:subject:user-agent:mime-version:date:message-id :from:to:cc:subject:date:message-id:reply-to; bh=vCCDWmEOxdrYyg03QGM9HCNXy8fqBDtdzIoCPdDROW0=; b=YX7+LXtTF5V7RKwwlL+PLYY/0BXJ5KWbXgprejd3wFVd1HU7X4eIEzoQ0ZC+j0aydy RtGtkscZ2CThkKXGSugkEnZJ+3cqlo85qD8ZpFT0EIa6Jc+WwJ63iGdbHWy3+rQufRFd ncOIZu2HTRd12OjUTCY9pYSH+aRQ5Qa8gdSBD1DCBx3JJq+k/IiVPKkxYvfxknbuokch aSxsYdB3Q/JxxYK97vRv8+KQ4Q1B+/z63JNdAzs2xii0BAO5WocndJcXo6WjB5/4ipad /7nWI75U61ZCqSPOyXofn3ZsQ4GKxu7mGnvN1i4l7xs0fq4/H4dv0IoSaHUUoWHAWdf6 9v4g== X-Google-DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=1e100.net; s=20230601; t=1715082195; x=1715686995; h=content-transfer-encoding:in-reply-to:from:content-language :references:cc:to:subject:user-agent:mime-version:date:message-id :x-gm-message-state:from:to:cc:subject:date:message-id:reply-to; bh=vCCDWmEOxdrYyg03QGM9HCNXy8fqBDtdzIoCPdDROW0=; b=WVes5fDAnxM7tCgsgoWJh4R2JWf+tObi5gSpNGOgVHRQs6M/oz2bkslhoAAN4obQBB mkRRFcotTESrgaT65WOkbn8CNoC0XG+XGC72KSiTkTiLP+WCF5J9/5SAjlXBz+L3Spud DhR9ezbv3EvFKlnehusFFhxjtnPRB6mxr4IvcmspMCNNVzYkedUwwvwvOG1R7YZU3CQ9 RYKtOpsVpmyIFUD/1th8V2hEe0nKEVEn9xU9Us36av/XroFyLEx/lUzsMFowQomrXP/E syb3Yx/OF8NYj7rinDULO0ef8TDRQyJSpFXlPySMBQztqN/n165l7n9hNHYdou3WF0s6 D5yw== X-Forwarded-Encrypted: i=1; AJvYcCWJo261P/sUuJSMtdbWeiVm084a1y4TKe0qpCOJBwcGo9pMm28795z8gJsLTFdfwvh9SzzxG52HQX+DPo18WvpgZ/RzXVFn8X0RIJ6Lyw== X-Gm-Message-State: AOJu0YyfriiInl2IUR77Vm9+UzCFe7fCasHBMiGBku7GFX1SOXEaRgsp y9plncUan4SH54mIyevjA7tP6m2PR4e5+aA44EOAQZ0MtRYNpNVHajb08Wc0fnYB6nymbKF9Rtj OM3Y= X-Google-Smtp-Source: AGHT+IEatV3ixiUCg3FxqsVCLWRFRBRCA8gGTMCNaxxmiDHfSI+2iThDhqWAlx33/+UViLds+bPqkg== X-Received: by 2002:ac2:58cc:0:b0:515:cfaf:737 with SMTP id u12-20020ac258cc000000b00515cfaf0737mr6627843lfo.9.1715082195460; Tue, 07 May 2024 04:43:15 -0700 (PDT) Received: from [172.30.205.144] (UNUSED.212-182-62-129.lubman.net.pl. [212.182.62.129]) by smtp.gmail.com with ESMTPSA id h2-20020a19ca42000000b005196e3e8a84sm2121235lfj.177.2024.05.07.04.43.14 (version=TLS1_3 cipher=TLS_AES_128_GCM_SHA256 bits=128/128); Tue, 07 May 2024 04:43:15 -0700 (PDT) Message-ID: <90a5b641-af5b-4640-b2ad-85dbbab523bf@linaro.org> Date: Tue, 7 May 2024 13:43:13 +0200 Precedence: bulk X-Mailing-List: linux-arm-msm@vger.kernel.org List-Id: List-Subscribe: List-Unsubscribe: MIME-Version: 1.0 User-Agent: Mozilla Thunderbird Subject: Re: [PATCH] phy: qcom-snps-femto-v2: Add load and voltage setting for LDO's used To: Dmitry Baryshkov , Udipto Goswami Cc: Bjorn Andersson , Vinod Koul , Kishon Vijay Abraham I , linux-arm-msm@vger.kernel.org, linux-phy@lists.infradead.org, linux-kernel@vger.kernel.org References: <20240502123312.31083-1-quic_ugoswami@quicinc.com> Content-Language: en-US From: Konrad Dybcio In-Reply-To: Content-Type: text/plain; charset=UTF-8; format=flowed Content-Transfer-Encoding: 7bit On 5/2/24 15:12, Dmitry Baryshkov wrote: > On Thu, 2 May 2024 at 15:33, Udipto Goswami wrote: >> >> The Femto phy depends on 0.88/ 1.8/ 3.3V regulators for its operation. >> If one of the regulators is not voted to the required minimum voltage >> for phy to operate, then High speed mode of operation will fail. >> >> On certain targets like (qcm6490_rb3gen2) where the minimum voltage >> of the regulator is lower than the operating voltage of the phy. >> If not voted properly, the phy supply would be limited to the min value >> of the LDO thereby rendering the phy non-operational. >> >> The current implementation of the regulators in the Femto PHY is not >> setting the load and voltage for each LDO. The appropriate voltages and >> loads required for the PHY to operate should be set. > > Please move min/max voltages to the DTS. There is no need to set them > from the driver. > > Also, is there any reason why you can't use `regulator-initial-mode = > ;` like other boards do? The point is to aggregate the values and switch to HPM if a threshold is crossed (or stay in LPM otherwise) Konrad