From mboxrd@z Thu Jan 1 00:00:00 1970 Received: from mail-pf1-f172.google.com (mail-pf1-f172.google.com [209.85.210.172]) (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 4DBBE158D8B for ; Wed, 24 Jul 2024 13:28:55 +0000 (UTC) Authentication-Results: smtp.subspace.kernel.org; arc=none smtp.client-ip=209.85.210.172 ARC-Seal:i=1; a=rsa-sha256; d=subspace.kernel.org; s=arc-20240116; t=1721827736; cv=none; b=uaLm+enYX0c24MQr/ksESLHuEM/H36PdSrnLXMpSFVVi4lN8qcYbetpz86iIzY7LKH+f8HNUtVIdl3UiOxJVPy3ynY+gxCShUDvhUODOc2rrETCwljQryi/+xuOjdn0V5DFLqa8AhsABnTp5TcEle/iyTvo9tHkVzBZF68xaQII= ARC-Message-Signature:i=1; a=rsa-sha256; d=subspace.kernel.org; s=arc-20240116; t=1721827736; c=relaxed/simple; bh=NbryZMNi++9IyWDKCW43oXUTr404zUv0RT78qB3Do6I=; h=Date:From:To:Cc:Subject:Message-ID:References:MIME-Version: Content-Type:Content-Disposition:In-Reply-To; b=nNo/ujVl6ID82Hgemt+zagkShA8K4zvl/pXBjyBRkmHHv8+z5L5q4VASS8J8CrlDhbJxe32U+8n54AOWD4Z1znaKw2z7GPbvA4E/69wtEl3N4eouWQUZut7B4x0yrJeEAINRdpWlVzXfPPeW/tJKMWHSoLuheGBVJ7M8YOHxQL4= 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=LgDdXhAY; arc=none smtp.client-ip=209.85.210.172 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="LgDdXhAY" Received: by mail-pf1-f172.google.com with SMTP id d2e1a72fcca58-70d18d4b94cso2451672b3a.2 for ; Wed, 24 Jul 2024 06:28:55 -0700 (PDT) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=linaro.org; s=google; t=1721827734; x=1722432534; darn=vger.kernel.org; h=in-reply-to:content-transfer-encoding:content-disposition :mime-version:references:message-id:subject:cc:to:from:date:from:to :cc:subject:date:message-id:reply-to; bh=EK7J1tmmD0LeVUI1TCcORLh7QZq/9jUNRzduMUl+rUQ=; b=LgDdXhAYUCPAgQ/clGLOA5YPymW32ytYmvwjnGxYDsje7yLKqaXjN2ghOVosjavZwp gCKMIAwR2SLPgUkKiLJM+5fhXwAzcjWGysuvamryNsuC8/+B27ZMznp5WiqFSeNqXZw5 waS9YE+ZFmuyV6F6YEhiCRBda5ozs1QzKjdyru+utGpZICR7LYYsAlejaKMMqEbseCsH t8ya5OGY1NyMnVuycXKCWxLX9De9G9XWffIgY5cEuUN92dLyWhtFgr9T1OHj4LcsCVIx ivGbam+R9SwR4vi/aZ7AcC8UfwKay1mPr2/bBak8FNRqV8r3kQ1Gafmh4YXPEYkKVCXc ySAQ== X-Google-DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=1e100.net; s=20230601; t=1721827734; x=1722432534; h=in-reply-to:content-transfer-encoding:content-disposition :mime-version:references:message-id:subject:cc:to:from:date :x-gm-message-state:from:to:cc:subject:date:message-id:reply-to; bh=EK7J1tmmD0LeVUI1TCcORLh7QZq/9jUNRzduMUl+rUQ=; b=C1+WDvZn7te5aPFeVXP3Yg/VLIZUh30oFwP/8QjYKdQ3i0CUUzeVtuhY1i18NnMGlQ 0PAPgnzEitcHt1a5lpVrY3hybkKnAAdS9CBN20pjsyHZWBIC/OUFcSPw+3aR34vt7gPU tz+v/XP2fZXwn7MjdHCPj1s4KP21gaDNBNBV7Xxzi8+bl56iTOLAlHUhgL8gTgSa7+gt PozTB14M1KTMg/3UpGGL39KMQRcPItOdymd6fP3bd//CtbN4svKzC3Gy+RfOkLBzBz92 vVLBnnaL81s4g2dAloJbA3HB3BI8DF8oCR4OHKcOamsv0/++MzjZeJV5qzSyimxEYYS6 EZ0Q== X-Forwarded-Encrypted: i=1; AJvYcCX+4opdx5BaPwldQ+B7viEYghCDUoO7k8iqhH0Ian6KrbcxBgxq5ivoPft0qYo1fgc+HgxLGtDZumAwpcVnYQe9rkD8+kw311qSZUCJ X-Gm-Message-State: AOJu0Yw6tO0Z3ynHnF5N5iVo+FgHqHujpgqYizXD0xIPILEMKC10i85B tqt9TA8oN5DGG31w4tsxp1WWC3ji4MfCOOTFX5gTMrFcGxOjrXPYaCwawgshEQ== X-Google-Smtp-Source: AGHT+IGrrJr+Oo0dBAP1RDU3YHrRkUwB1oBOMPFPB4/1gpiTJwDY4uP4ocgeH+a+h16PfoQZz0jhTQ== X-Received: by 2002:a05:6a21:7884:b0:1c4:17e1:14d0 with SMTP id adf61e73a8af0-1c461a04b51mr2600716637.47.1721827734299; Wed, 24 Jul 2024 06:28:54 -0700 (PDT) Received: from thinkpad ([103.244.168.26]) by smtp.gmail.com with ESMTPSA id d2e1a72fcca58-70d2628be4fsm5226884b3a.80.2024.07.24.06.28.52 (version=TLS1_3 cipher=TLS_AES_256_GCM_SHA384 bits=256/256); Wed, 24 Jul 2024 06:28:53 -0700 (PDT) Date: Wed, 24 Jul 2024 18:58:50 +0530 From: Manivannan Sadhasivam To: Slark Xiao Cc: Dmitry Baryshkov , mhi@lists.linux.dev, linux-arm-msm@vger.kernel.org, linux-kernel@vger.kernel.org Subject: Re: Re:Re: Re: [PATCH] bus: mhi: host: Add firehose support for Foxconn SDX24/SDX55/SDX65 Message-ID: <20240724132850.GA3349@thinkpad> References: <20240709015818.110384-1-slark_xiao@163.com> <17f0f426.4faf.190b4edaadd.Coremail.slark_xiao@163.com> <7036d57d.5904.190b512d00b.Coremail.slark_xiao@163.com> <7b69f589.982a.190e4735077.Coremail.slark_xiao@163.com> 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=utf-8 Content-Disposition: inline Content-Transfer-Encoding: 8bit In-Reply-To: <7b69f589.982a.190e4735077.Coremail.slark_xiao@163.com> On Wed, Jul 24, 2024 at 07:14:40PM +0800, Slark Xiao wrote: > > At 2024-07-15 14:27:07, "Slark Xiao" wrote: > > > >At 2024-07-15 14:16:57, "Dmitry Baryshkov" wrote: > >>On Mon, 15 Jul 2024 at 08:46, Slark Xiao wrote: > >>> > >>> > >>> At 2024-07-13 23:09:47, "Dmitry Baryshkov" wrote: > >>> >On Tue, Jul 09, 2024 at 09:58:18AM GMT, Slark Xiao wrote: > >>> >> Since we implement the FIREHOSE channel support in foxconn mhi > >>> >> channels, that means each product which use this channel config > >>> >> would support FIREHOSE. But according to the trigger_edl feature, > >>> >> we need to enable it by adding '.edl_trigger = true' in device > >>> >> info struct. > >>> >> Also, we update all edl image path from 'qcom' to 'fox' in case of > >>> >> conflicting with other vendors. > >>> > > >>> >Separate patches please. Also don't use "we", just an imerative style: > >>> >do this and that. > >>> > > >>> > >>> Do you mean use 2 patches (1 for enabling trigger edl and 1 for > >>> modifying path)? Though these changes are aimed to make > >>> firehose download successfully. > >> > >>Yes. "Do this. Also do that" is usually a sign that the patch should be split. > > > >Will do a update in next version. > > > >> > >>> > >>> >> > >>> >> Signed-off-by: Slark Xiao > >>> >> --- > >>> >> drivers/bus/mhi/host/pci_generic.c | 20 ++++++++++++++------ > >>> >> 1 file changed, 14 insertions(+), 6 deletions(-) > >>> >> > >>> >> diff --git a/drivers/bus/mhi/host/pci_generic.c b/drivers/bus/mhi/host/pci_generic.c > >>> >> index 14a11880bcea..440609b81e57 100644 > >>> >> --- a/drivers/bus/mhi/host/pci_generic.c > >>> >> +++ b/drivers/bus/mhi/host/pci_generic.c > >>> >> @@ -433,8 +433,8 @@ static const struct mhi_controller_config modem_foxconn_sdx72_config = { > >>> >> > >>> >> static const struct mhi_pci_dev_info mhi_foxconn_sdx55_info = { > >>> >> .name = "foxconn-sdx55", > >>> >> - .fw = "qcom/sdx55m/sbl1.mbn", > >>> >> - .edl = "qcom/sdx55m/edl.mbn", > >>> >> + .edl = "fox/sdx55m/prog_firehose_sdx55.mbn", > >>> > > >>> >qcom/sdx55m/foxconn/prog_firehose_sdx55.mbn > >>> > >>> what's your opinion?Mani > > This format mismatch with Foxconn SDX72 edl path"fox/sdx72m/edl.mbn". > I think we should align with that changes we just committed. > What's your opinion, Mani? > This really sneaked through. I shouldn't have allowed this path. We should bite the bullet and use a standard path for all modems as Dmitry suggested. So the path will become: "qcom///" where chip is 'sdx72m' and vendor is 'foxconn'. If you need to use product specific firmware, then "qcom///" Since the firmware itself is not upstreamed to linux-firmware, changing the path is not a big deal. - Mani -- மணிவண்ணன் சதாசிவம்