Received-SPF: pass (google.com: domain of linux-kernel-owner@vger.kernel.org designates 23.128.96.18 as permitted sender) client-ip=23.128.96.18;
IronPort-SDR: S3bT1xzbnBuEoNX+9Z0NH553IqVw2cY7wIATP2k+3sQiSjp14IPlFGCUmEr/U5tyD6b9k+Szfc
 NyUwRi+yKm/+I+5aTrmO/+KqN7n6hb1aNOuLOqdaDwp8Yw/WvMc7JK29CIHmdUte9qT7U99yak
 7rTZ75hqBiwejlCtc5U7XRLN6ITcw2ff1YJRONe7yQ6KxbvWXgI3ntPnVdgMFNaEOBEvGn80+M
 eprZkn/QjU3/qMdXAx/V5of0ZQmXfg1IzlC0+LQlFk8j1wMlWJondn7arpVLtZ1Z7qBDdlI6eb
 zFw=
From:   Avri Altman <Avri.Altman@wdc.com>
To:     Stanley Chu <stanley.chu@mediatek.com>,
        "linux-scsi@vger.kernel.org" <linux-scsi@vger.kernel.org>,
        "martin.petersen@oracle.com" <martin.petersen@oracle.com>,
        "alim.akhtar@samsung.com" <alim.akhtar@samsung.com>,
        "jejb@linux.ibm.com" <jejb@linux.ibm.com>
CC:     "matthias.bgg@gmail.com" <matthias.bgg@gmail.com>,
        "bvanassche@acm.org" <bvanassche@acm.org>,
        "linux-mediatek@lists.infradead.org" 
        <linux-mediatek@lists.infradead.org>,
        "linux-arm-kernel@lists.infradead.org" 
        <linux-arm-kernel@lists.infradead.org>,
        "linux-kernel@vger.kernel.org" <linux-kernel@vger.kernel.org>,
        "kuohong.wang@mediatek.com" <kuohong.wang@mediatek.com>,
        "peter.wang@mediatek.com" <peter.wang@mediatek.com>,
        "chun-hung.wu@mediatek.com" <chun-hung.wu@mediatek.com>,
        "andy.teng@mediatek.com" <andy.teng@mediatek.com>,
        "chaotian.jing@mediatek.com" <chaotian.jing@mediatek.com>,
        "cc.chou@mediatek.com" <cc.chou@mediatek.com>
Subject: RE: [PATCH] scsi: ufs-mediatek: Modify the minimum RX/TX lane count
 to 2
Thread-Topic: [PATCH] scsi: ufs-mediatek: Modify the minimum RX/TX lane count
 to 2
Thread-Index: AQHWdgTc2Id9u3Ln3Ua1qwoK877ZRalAgmqA
Date:   Thu, 20 Aug 2020 06:03:45 +0000
Message-ID: <BY5PR04MB67057C0623170CFABE9E8AFBFC5A0@BY5PR04MB6705.namprd04.prod.outlook.com>
References: <20200819084340.7021-1-stanley.chu@mediatek.com>
In-Reply-To: <20200819084340.7021-1-stanley.chu@mediatek.com>
Accept-Language: en-US
Content-Language: en-US
wdcipoutbound: EOP-TRUE
Content-Type: text/plain; charset="us-ascii"
Content-Transfer-Encoding: quoted-printable
MIME-Version: 1.0
X-MS-Exchange-CrossTenant-Network-Message-Id: 8db1184b-285a-4a59-dfb6-08d844cecc84
X-MS-Exchange-CrossTenant-originalarrivaltime: 20 Aug 2020 06:03:45.7290
 (UTC)
X-MS-Exchange-CrossTenant-fromentityheader: Hosted
X-MS-Exchange-CrossTenant-id: b61c8803-16f3-4c35-9b17-6f65f441df86
X-MS-Exchange-CrossTenant-mailboxtype: HOSTED
X-MS-Exchange-CrossTenant-userprincipalname: q6e1elYAWlc/NWkq2qYda2Yr2FwSVLwXAYnwAcZOeAiGMOSopLhwrHPIItbV/C+FDbaKYFw+nVRqSyOESXFLFg==
X-MS-Exchange-Transport-CrossTenantHeadersStamped: BYAPR04MB5447
Sender: linux-kernel-owner@vger.kernel.org
Precedence: bulk
List-ID: <linux-kernel.vger.kernel.org>
X-Mailing-List: linux-kernel@vger.kernel.org

=20
>=20
> From: Andy Teng <andy.teng@mediatek.com>
>=20
> MediaTek UFS host starts to support 2 lanes, thus modify the
> minimum lane count to 2.
>=20
> This modification shall not impact old 1-lane host because
> PA_CONNECTEDRXDATALANES and PA_CONNECTEDTXDATALANES will limit
> the
> target lanes properly during power mode change. So we could relax
> the limitation in ufs_dev_params.
>=20
> Reviewed-by: Stanley Chu <stanley.chu@mediatek.com>
> Signed-off-by: Andy Teng <andy.teng@mediatek.com>
> Singed-off-by: Stanley Chu <stanley.chu@mediatek.com>
Reviewed-by: Avri Altman <avri.altman@wdc.com>

But then again, why those constants needs to be re-negotiated every pre-cha=
nge?

Thanks,
Avri