[<prev] [next>] [thread-next>] [day] [month] [year] [list]
Message-ID: <20241210073212.3917912-1-wenst@chromium.org>
Date: Tue, 10 Dec 2024 15:32:09 +0800
From: Chen-Yu Tsai <wenst@...omium.org>
To: Chaotian Jing <chaotian.jing@...iatek.com>,
Ulf Hansson <ulf.hansson@...aro.org>,
Matthias Brugger <matthias.bgg@...il.com>,
AngeloGioacchino Del Regno <angelogioacchino.delregno@...labora.com>,
Rob Herring <robh@...nel.org>,
Krzysztof Kozlowski <krzk+dt@...nel.org>,
Conor Dooley <conor+dt@...nel.org>,
Wenbin Mei <wenbin.mei@...iatek.com>
Cc: Chen-Yu Tsai <wenst@...omium.org>,
linux-mmc@...r.kernel.org,
linux-kernel@...r.kernel.org,
linux-arm-kernel@...ts.infradead.org,
linux-mediatek@...ts.infradead.org,
devicetree@...r.kernel.org,
Frank Wunderlich <frank-w@...lic-files.de>,
Andy-ld Lu <andy-ld.lu@...iatek.com>
Subject: [PATCH v2 0/2] mmc: mtk-sd: Limit second register range to SoCs requiring it
Changes since v1:
- Added MT7988 compatible string to list that needs second register
range
- Dropped comment about uncertainty of MT6779 from commit message
- Picked up Angelo's Reviewed-by
Currently the usage of a second register range for MediaTek's MMC
controller is seen only on a few platforms. However the driver tries
to grab the register base regardless, and then ignores any errors.
This makes all the other platforms that don't need it produce confusing
error messages.
Instead, make the driver only grab the register range for platforms that
actually require it, and correctly handle the error. Also fix up the
binding so that validation of the "reg" property is as strict as
possible.
Please take a look.
Thanks
ChenYu
Chen-Yu Tsai (2):
dt-bindings: mmc: mtk-sd: Document compatibles that need two register
ranges
mmc: mtk-sd: Limit getting top_base to SoCs that require it
Documentation/devicetree/bindings/mmc/mtk-sd.yaml | 10 +++++++++-
drivers/mmc/host/mtk-sd.c | 15 +++++++++++----
2 files changed, 20 insertions(+), 5 deletions(-)
--
2.47.0.338.g60cca15819-goog
Powered by blists - more mailing lists