Linux kernel stable tree (mirror) https://git.radix-linux.su/kernel/linux.git/atom?h=v5.16 2021-12-09T09:30:11+00:00 2021-12-09T09:30:11+00:00 Wenbin Mei wenbin.mei@mediatek.com 2021-12-07T07:50:13+00:00 urn:sha1:d594b35d3b31bc04b6ef36589f38135d3acb8df5 If mmc_get_ext_csd success, the ext_csd are not freed. Add the missing kfree() calls. Signed-off-by: Wenbin Mei <wenbin.mei@mediatek.com> Fixes: c4ac38c6539b ("mmc: mtk-sd: Add HS400 online tuning support") Link: https://lore.kernel.org/r/20211207075013.22911-1-wenbin.mei@mediatek.com Signed-off-by: Ulf Hansson <ulf.hansson@linaro.org> 2021-10-28T09:59:34+00:00 Ulf Hansson ulf.hansson@linaro.org 2021-10-28T09:59:34+00:00 urn:sha1:237ecf1be300fed6275742ba63c6472abd16322e 2021-10-28T09:55:20+00:00 Wenbin Mei wenbin.mei@mediatek.com 2021-10-28T02:20:49+00:00 urn:sha1:e8a1ff65927080278e6826f797b7c197fb2611a6 We must enable clock before cqhci init, because crypto needs read information from CQHCI registers, otherwise, it will hang in MediaTek mmc host controller. Signed-off-by: Wenbin Mei <wenbin.mei@mediatek.com> Fixes: 88bd652b3c74 ("mmc: mediatek: command queue support") Cc: stable@vger.kernel.org Acked-by: Chaotian Jing <chaotian.jing@mediatek.com> Link: https://lore.kernel.org/r/20211028022049.22129-1-wenbin.mei@mediatek.com Signed-off-by: Ulf Hansson <ulf.hansson@linaro.org> 2021-10-12T08:21:18+00:00 Wenbin Mei wenbin.mei@mediatek.com 2021-09-17T12:48:03+00:00 urn:sha1:c4ac38c6539b6cccfda7e8cf8da50edf7877c865 According to JEDEC Spec, there is no need to do tuning under HS400 mode since the Rx signal is aligned with the DS signal. However, MediaTek's IC need set its "DS delay" internally to ensure it can latch Rx signal correctly. In previous version, We provide an "hs400-ds-delay" in device tree to cover different chipset/PCB design, and it works fine in most cases. But, with the development of process technology and the big VCore voltage scale range(may have 0.7V/0.6V/0.55V), it is difficult to find a suitable "hs400-ds-delay" to cover all of IC corner cases(SSSS/TTTT/FFFF). So that We must have the ability to do hs400 online tuning. Signed-off-by: Wenbin Mei <wenbin.mei@mediatek.com> Reviewed-by: Chaotian Jing <chaotian.jing@mediatek.com> Link: https://lore.kernel.org/r/20210917124803.22871-4-wenbin.mei@mediatek.com Signed-off-by: Ulf Hansson <ulf.hansson@linaro.org> 2021-10-12T08:21:16+00:00 ChanWoo Lee cw9316.lee@samsung.com 2021-08-30T02:17:49+00:00 urn:sha1:d74179b86925165562aec81e0ba269ebc453dad2 Remove unused parameters 1. msdc_start_data() - struct mmc_request *mrq 2. msdc_track_cmd_data() - struct mmc_data *data Signed-off-by: ChanWoo Lee <cw9316.lee@samsung.com> Link: https://lore.kernel.org/r/20210830021749.5947-1-cw9316.lee@samsung.com Signed-off-by: Ulf Hansson <ulf.hansson@linaro.org> 2021-10-12T08:21:16+00:00 ChanWoo Lee cw9316.lee@samsung.com 2021-08-27T09:31:19+00:00 urn:sha1:961e40f714f6ef958e008b7ee4e66df1ea0fac89 The mmc_request structure(*mrq) is not used. //msdc_cmd_find_resp I remove the unnecessary code related to the mmc_request structure. Signed-off-by: ChanWoo Lee <cw9316.lee@samsung.com> Link: https://lore.kernel.org/r/20210827093119.32481-1-cw9316.lee@samsung.com Signed-off-by: Ulf Hansson <ulf.hansson@linaro.org> 2021-10-12T08:21:16+00:00 Derong Liu derong.liu@mediatek.com 2021-08-27T07:15:37+00:00 urn:sha1:43e5fee317f4b0a48992b8b07935b1a3ac20ce84 We found this issue on a 5G platform, during CMDQ error handling, if DMA status is active when it call msdc_reset_hw(), it means mmc host hw reset and DMA transfer will be parallel, mmc host may access sram region unexpectedly. According to the programming guide of mtk-sd host, it needs to wait for dma stop done after set dma stop. This change should be applied to all SoCs. Signed-off-by: Derong Liu <derong.liu@mediatek.com> Cc: stable@vger.kernel.org Link: https://lore.kernel.org/r/20210827071537.1034-1-derong.liu@mediatek.com Signed-off-by: Ulf Hansson <ulf.hansson@linaro.org> 2021-06-14T11:57:40+00:00 Yue Hu huyue2@yulong.com 2021-05-17T10:54:32+00:00 urn:sha1:f0ed43edb4cb793f4d9909feef32510ed77ed35a We do not use the 'data' in msdc_data_xfer_next(). Signed-off-by: Yue Hu <huyue2@yulong.com> Link: https://lore.kernel.org/r/20210517105432.1682-1-zbestahu@gmail.com Signed-off-by: Ulf Hansson <ulf.hansson@linaro.org> 2021-06-14T11:57:39+00:00 Yue Hu huyue2@yulong.com 2021-05-17T10:09:00+00:00 urn:sha1:151071351bb6f3d1861e99a22c4cebadf81911a0 We already have 'mrq->data' before calling these two functions, no need to find it again via 'mrq->data' internally. Also remove local data variable accordingly. Signed-off-by: Yue Hu <huyue2@yulong.com> Link: https://lore.kernel.org/r/20210517100900.1620-1-zbestahu@gmail.com Signed-off-by: Ulf Hansson <ulf.hansson@linaro.org> 2021-02-01T10:54:43+00:00 Chaotian Jing chaotian.jing@mediatek.com 2020-12-18T07:16:11+00:00 urn:sha1:0354ca6edd464a2cf332f390581977b8699ed081 when get request SW timeout, if CMD/DAT xfer done irq coming right now, then there is race between the msdc_request_timeout work and irq handler, and the host->cmd and host->data may set to NULL in irq handler. also, current flow ensure that only one path can go to msdc_request_done(), so no need check the return value of cancel_delayed_work(). Signed-off-by: Chaotian Jing <chaotian.jing@mediatek.com> Link: https://lore.kernel.org/r/20201218071611.12276-1-chaotian.jing@mediatek.com Signed-off-by: Ulf Hansson <ulf.hansson@linaro.org>