public inbox for linux-arm-kernel@lists.infradead.org
 help / color / mirror / Atom feed
From: Konrad Dybcio <konrad.dybcio@oss.qualcomm.com>
To: Krzysztof Kozlowski <krzk@kernel.org>,
	Taniya Das <quic_tdas@quicinc.com>,
	Taniya Das <taniya.das@oss.qualcomm.com>,
	Bjorn Andersson <andersson@kernel.org>,
	linux-arm-msm@vger.kernel.org
Cc: linux-arm-kernel@lists.infradead.org,
	Ajit Pandey <ajit.pandey@oss.qualcomm.com>,
	Imran Shaik <imran.shaik@oss.qualcomm.com>,
	Jagadeesh Kona <jagadeesh.kona@oss.qualcomm.com>,
	linux-kernel@vger.kernel.org
Subject: Re: [PATCH] arm64: defconfig: Change CONFIG_SM_TCSRCC_8750 from m to y
Date: Fri, 17 Oct 2025 10:15:14 +0200	[thread overview]
Message-ID: <0a9874d4-29e4-477e-a675-c4534658f9d9@oss.qualcomm.com> (raw)
In-Reply-To: <8580ae8e-50e9-481c-b9f3-125b6d1cb494@kernel.org>

On 10/17/25 8:54 AM, Krzysztof Kozlowski wrote:
> On 17/10/2025 07:56, Krzysztof Kozlowski wrote:
>> On 17/10/2025 07:49, Taniya Das wrote:
>>>
>>>
>>> On 10/17/2025 10:51 AM, Krzysztof Kozlowski wrote:
>>>> On 17/10/2025 07:16, Taniya Das wrote:

[...]

>>> We have tried booting up recently and and that is what we observed. The
>>> patch from 'm' to 'y' helps the UFS probe is successful and the rootfs
>>> is picked from ufs partitions. I will add these fail & success log
>>> snippets in the commit text.
>>
>> That's not enough. You need to explain why UFS fails. After explaining
>> this, I guess bug in UFS would be exposed thus that one should be fixed.
>> You just provided band-aid without fixing the real problem.
>>
>> NAK
> 
> 
> ... and to prove your analysis is wrong (because your setup is likely
> having issues) I even tested now linux next with defconfig. Works all
> fine on next-20251013. You did not share which kernel even has this
> issue, maybe some downstream tree?

Is there a chance you have the TCSR module packed in initramfs and
Taniya doesn't?

Konrad


  parent reply	other threads:[~2025-10-17  8:15 UTC|newest]

Thread overview: 18+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2025-10-16 18:53 [PATCH] arm64: defconfig: Change CONFIG_SM_TCSRCC_8750 from m to y Taniya Das
2025-10-17  4:30 ` Krzysztof Kozlowski
2025-10-17  5:16   ` Taniya Das
2025-10-17  5:21     ` Krzysztof Kozlowski
2025-10-17  5:49       ` Taniya Das
2025-10-17  5:56         ` Krzysztof Kozlowski
2025-10-17  6:54           ` Krzysztof Kozlowski
2025-10-17  6:57             ` Taniya Das
2025-10-17  7:37               ` Krzysztof Kozlowski
2025-10-17  8:15             ` Konrad Dybcio [this message]
2025-10-17  8:34               ` Krzysztof Kozlowski
2025-10-17  6:55           ` Taniya Das
2025-10-17  7:36             ` Krzysztof Kozlowski
2025-10-17 21:47             ` Bjorn Andersson
2025-10-17 21:41         ` Bjorn Andersson
2025-10-21  9:12           ` Taniya Das
2025-10-19 11:57     ` Dmitry Baryshkov
2025-10-21  9:15       ` Taniya Das

Reply instructions:

You may reply publicly to this message via plain-text email
using any one of the following methods:

* Save the following mbox file, import it into your mail client,
  and reply-to-all from there: mbox

  Avoid top-posting and favor interleaved quoting:
  https://en.wikipedia.org/wiki/Posting_style#Interleaved_style

* Reply using the --to, --cc, and --in-reply-to
  switches of git-send-email(1):

  git send-email \
    --in-reply-to=0a9874d4-29e4-477e-a675-c4534658f9d9@oss.qualcomm.com \
    --to=konrad.dybcio@oss.qualcomm.com \
    --cc=ajit.pandey@oss.qualcomm.com \
    --cc=andersson@kernel.org \
    --cc=imran.shaik@oss.qualcomm.com \
    --cc=jagadeesh.kona@oss.qualcomm.com \
    --cc=krzk@kernel.org \
    --cc=linux-arm-kernel@lists.infradead.org \
    --cc=linux-arm-msm@vger.kernel.org \
    --cc=linux-kernel@vger.kernel.org \
    --cc=quic_tdas@quicinc.com \
    --cc=taniya.das@oss.qualcomm.com \
    /path/to/YOUR_REPLY

  https://kernel.org/pub/software/scm/git/docs/git-send-email.html

* If your mail client supports setting the In-Reply-To header
  via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line before the message body.
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox