On Tue, 10 Dec 2024 at 08:32, Chen-Yu Tsai <wenst@xxxxxxxxxxxx> wrote: > > 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(-) > Both patches applied for next, thanks! Kind regards Uffe
