* [PATCH v1 0/1] mmc: sdhci-of-arasan: Specify .clk_ops for Keem Bay SOC @ 2020-11-06 12:09 muhammad.husaini.zulkifli 2020-11-06 12:09 ` [PATCH v1 1/1] " muhammad.husaini.zulkifli 2020-11-12 14:15 ` [PATCH v1 0/1] " Zulkifli, Muhammad Husaini 0 siblings, 2 replies; 10+ messages in thread From: muhammad.husaini.zulkifli @ 2020-11-06 12:09 UTC (permalink / raw) To: linux-mmc, linux-arm-kernel, linux-kernel Cc: adrian.hunter, lakshmi.bai.raja.subramanian, wan.ahmad.zainie.wan.mohamad, muhammad.husaini.zulkifli, david.e.box From: Muhammad Husaini Zulkifli <muhammad.husaini.zulkifli@intel.com> Hi. This patch declares the .clk_ops for Keem Bay SOC due to the introduction of new clock operations handling. This was tested with Keem Bay evaluation module board. Kindly help to review it. Thank you. Muhammad Husaini Zulkifli (1): mmc: sdhci-of-arasan: Specify .clk_ops for Keem Bay SOC drivers/mmc/host/sdhci-of-arasan.c | 3 +++ 1 file changed, 3 insertions(+) -- 2.17.1 ^ permalink raw reply [flat|nested] 10+ messages in thread
* [PATCH v1 1/1] mmc: sdhci-of-arasan: Specify .clk_ops for Keem Bay SOC 2020-11-06 12:09 [PATCH v1 0/1] mmc: sdhci-of-arasan: Specify .clk_ops for Keem Bay SOC muhammad.husaini.zulkifli @ 2020-11-06 12:09 ` muhammad.husaini.zulkifli 2020-11-16 11:41 ` Ulf Hansson 2020-11-12 14:15 ` [PATCH v1 0/1] " Zulkifli, Muhammad Husaini 1 sibling, 1 reply; 10+ messages in thread From: muhammad.husaini.zulkifli @ 2020-11-06 12:09 UTC (permalink / raw) To: linux-mmc, linux-arm-kernel, linux-kernel Cc: adrian.hunter, lakshmi.bai.raja.subramanian, wan.ahmad.zainie.wan.mohamad, muhammad.husaini.zulkifli, david.e.box From: Muhammad Husaini Zulkifli <muhammad.husaini.zulkifli@intel.com> Commit 16ada730a759 ("mmc: sdhci-of-arasan: Modify clock operations handling") introduces platform specific SDHCI clock operation. This patch declares the clock operation for Keem Bay. Add clk_ops for SD, EMMC and SDIO operations. Fixes: 36c6aadaae86 ("mmc: sdhci-of-arasan: Add support for Intel Keem Bay") Signed-off-by: Muhammad Husaini Zulkifli <muhammad.husaini.zulkifli@intel.com> Reviewed-by: Adrian Hunter <adrian.hunter@intel.com> --- drivers/mmc/host/sdhci-of-arasan.c | 3 +++ 1 file changed, 3 insertions(+) diff --git a/drivers/mmc/host/sdhci-of-arasan.c b/drivers/mmc/host/sdhci-of-arasan.c index 829ccef87426..012d52e1abee 100644 --- a/drivers/mmc/host/sdhci-of-arasan.c +++ b/drivers/mmc/host/sdhci-of-arasan.c @@ -1199,16 +1199,19 @@ static struct sdhci_arasan_of_data sdhci_arasan_versal_data = { static struct sdhci_arasan_of_data intel_keembay_emmc_data = { .soc_ctl_map = &intel_keembay_soc_ctl_map, .pdata = &sdhci_keembay_emmc_pdata, + .clk_ops = &arasan_clk_ops, }; static struct sdhci_arasan_of_data intel_keembay_sd_data = { .soc_ctl_map = &intel_keembay_soc_ctl_map, .pdata = &sdhci_keembay_sd_pdata, + .clk_ops = &arasan_clk_ops, }; static struct sdhci_arasan_of_data intel_keembay_sdio_data = { .soc_ctl_map = &intel_keembay_soc_ctl_map, .pdata = &sdhci_keembay_sdio_pdata, + .clk_ops = &arasan_clk_ops, }; static const struct of_device_id sdhci_arasan_of_match[] = { -- 2.17.1 ^ permalink raw reply related [flat|nested] 10+ messages in thread
* Re: [PATCH v1 1/1] mmc: sdhci-of-arasan: Specify .clk_ops for Keem Bay SOC 2020-11-06 12:09 ` [PATCH v1 1/1] " muhammad.husaini.zulkifli @ 2020-11-16 11:41 ` Ulf Hansson 2020-11-16 14:22 ` Zulkifli, Muhammad Husaini 0 siblings, 1 reply; 10+ messages in thread From: Ulf Hansson @ 2020-11-16 11:41 UTC (permalink / raw) To: Zulkifli, Muhammad Husaini Cc: linux-mmc, Linux ARM, Linux Kernel Mailing List, Adrian Hunter, Raja Subramanian, Lakshmi Bai, Wan Ahmad Zainie, David E. Box On Fri, 6 Nov 2020 at 05:10, <muhammad.husaini.zulkifli@intel.com> wrote: > > From: Muhammad Husaini Zulkifli <muhammad.husaini.zulkifli@intel.com> > > Commit 16ada730a759 ("mmc: sdhci-of-arasan: Modify clock operations handling") > introduces platform specific SDHCI clock operation. > > This patch declares the clock operation for Keem Bay. > Add clk_ops for SD, EMMC and SDIO operations. The above commit message doesn't really tell why or what goes on here. Can please try to clarify that. > > Fixes: 36c6aadaae86 ("mmc: sdhci-of-arasan: Add support for Intel Keem Bay") Is $subject patch fixing a bug/regression? Kind regards Uffe > > Signed-off-by: Muhammad Husaini Zulkifli <muhammad.husaini.zulkifli@intel.com> > Reviewed-by: Adrian Hunter <adrian.hunter@intel.com> > --- > drivers/mmc/host/sdhci-of-arasan.c | 3 +++ > 1 file changed, 3 insertions(+) > > diff --git a/drivers/mmc/host/sdhci-of-arasan.c b/drivers/mmc/host/sdhci-of-arasan.c > index 829ccef87426..012d52e1abee 100644 > --- a/drivers/mmc/host/sdhci-of-arasan.c > +++ b/drivers/mmc/host/sdhci-of-arasan.c > @@ -1199,16 +1199,19 @@ static struct sdhci_arasan_of_data sdhci_arasan_versal_data = { > static struct sdhci_arasan_of_data intel_keembay_emmc_data = { > .soc_ctl_map = &intel_keembay_soc_ctl_map, > .pdata = &sdhci_keembay_emmc_pdata, > + .clk_ops = &arasan_clk_ops, > }; > > static struct sdhci_arasan_of_data intel_keembay_sd_data = { > .soc_ctl_map = &intel_keembay_soc_ctl_map, > .pdata = &sdhci_keembay_sd_pdata, > + .clk_ops = &arasan_clk_ops, > }; > > static struct sdhci_arasan_of_data intel_keembay_sdio_data = { > .soc_ctl_map = &intel_keembay_soc_ctl_map, > .pdata = &sdhci_keembay_sdio_pdata, > + .clk_ops = &arasan_clk_ops, > }; > > static const struct of_device_id sdhci_arasan_of_match[] = { > -- > 2.17.1 > ^ permalink raw reply [flat|nested] 10+ messages in thread
* RE: [PATCH v1 1/1] mmc: sdhci-of-arasan: Specify .clk_ops for Keem Bay SOC 2020-11-16 11:41 ` Ulf Hansson @ 2020-11-16 14:22 ` Zulkifli, Muhammad Husaini 2020-11-17 11:27 ` Ulf Hansson 0 siblings, 1 reply; 10+ messages in thread From: Zulkifli, Muhammad Husaini @ 2020-11-16 14:22 UTC (permalink / raw) To: Ulf Hansson Cc: linux-mmc, Linux ARM, Linux Kernel Mailing List, Hunter, Adrian, Raja Subramanian, Lakshmi Bai, Wan Mohamad, Wan Ahmad Zainie, David E. Box Hi Ulf, Thanks for your review comments. I replied inline >-----Original Message----- >From: Ulf Hansson <ulf.hansson@linaro.org> >Sent: Monday, November 16, 2020 7:41 PM >To: Zulkifli, Muhammad Husaini <muhammad.husaini.zulkifli@intel.com> >Cc: linux-mmc@vger.kernel.org; Linux ARM <linux-arm- >kernel@lists.infradead.org>; Linux Kernel Mailing List <linux- >kernel@vger.kernel.org>; Hunter, Adrian <adrian.hunter@intel.com>; Raja >Subramanian, Lakshmi Bai <lakshmi.bai.raja.subramanian@intel.com>; Wan >Mohamad, Wan Ahmad Zainie ><wan.ahmad.zainie.wan.mohamad@intel.com>; David E. Box ><david.e.box@linux.intel.com> >Subject: Re: [PATCH v1 1/1] mmc: sdhci-of-arasan: Specify .clk_ops for Keem >Bay SOC > >On Fri, 6 Nov 2020 at 05:10, <muhammad.husaini.zulkifli@intel.com> wrote: >> >> From: Muhammad Husaini Zulkifli <muhammad.husaini.zulkifli@intel.com> >> >> Commit 16ada730a759 ("mmc: sdhci-of-arasan: Modify clock operations >> handling") introduces platform specific SDHCI clock operation. >> >> This patch declares the clock operation for Keem Bay. >> Add clk_ops for SD, EMMC and SDIO operations. > >The above commit message doesn't really tell why or what goes on here. >Can please try to clarify that. We missed out the clock operation handling patch (Commit 16ada730a759) and did not handle the clk_ops for Keem Bay SOC devices. These 2 patches (Commit 16ada730a759) and (Commit 36c6aadaae86) are merged at around the same time. We catch the issue later when trying to boot into v5.10-rc1 with Keem Bay EVM. That is why I created this patch to handle the clk_ops for Keem Bay devices. > >> >> Fixes: 36c6aadaae86 ("mmc: sdhci-of-arasan: Add support for Intel Keem >> Bay") > >Is $subject patch fixing a bug/regression? This is to fix issue on previous commit. It is a bug fix. > >Kind regards >Uffe > >> >> Signed-off-by: Muhammad Husaini Zulkifli >> <muhammad.husaini.zulkifli@intel.com> >> Reviewed-by: Adrian Hunter <adrian.hunter@intel.com> >> --- >> drivers/mmc/host/sdhci-of-arasan.c | 3 +++ >> 1 file changed, 3 insertions(+) >> >> diff --git a/drivers/mmc/host/sdhci-of-arasan.c >> b/drivers/mmc/host/sdhci-of-arasan.c >> index 829ccef87426..012d52e1abee 100644 >> --- a/drivers/mmc/host/sdhci-of-arasan.c >> +++ b/drivers/mmc/host/sdhci-of-arasan.c >> @@ -1199,16 +1199,19 @@ static struct sdhci_arasan_of_data >> sdhci_arasan_versal_data = { static struct sdhci_arasan_of_data >intel_keembay_emmc_data = { >> .soc_ctl_map = &intel_keembay_soc_ctl_map, >> .pdata = &sdhci_keembay_emmc_pdata, >> + .clk_ops = &arasan_clk_ops, >> }; >> >> static struct sdhci_arasan_of_data intel_keembay_sd_data = { >> .soc_ctl_map = &intel_keembay_soc_ctl_map, >> .pdata = &sdhci_keembay_sd_pdata, >> + .clk_ops = &arasan_clk_ops, >> }; >> >> static struct sdhci_arasan_of_data intel_keembay_sdio_data = { >> .soc_ctl_map = &intel_keembay_soc_ctl_map, >> .pdata = &sdhci_keembay_sdio_pdata, >> + .clk_ops = &arasan_clk_ops, >> }; >> >> static const struct of_device_id sdhci_arasan_of_match[] = { >> -- >> 2.17.1 >> ^ permalink raw reply [flat|nested] 10+ messages in thread
* Re: [PATCH v1 1/1] mmc: sdhci-of-arasan: Specify .clk_ops for Keem Bay SOC 2020-11-16 14:22 ` Zulkifli, Muhammad Husaini @ 2020-11-17 11:27 ` Ulf Hansson 2020-11-17 13:46 ` Zulkifli, Muhammad Husaini 0 siblings, 1 reply; 10+ messages in thread From: Ulf Hansson @ 2020-11-17 11:27 UTC (permalink / raw) To: Zulkifli, Muhammad Husaini Cc: linux-mmc, Linux ARM, Linux Kernel Mailing List, Hunter, Adrian, Raja Subramanian, Lakshmi Bai, Wan Mohamad, Wan Ahmad Zainie, David E. Box On Mon, 16 Nov 2020 at 15:22, Zulkifli, Muhammad Husaini <muhammad.husaini.zulkifli@intel.com> wrote: > > Hi Ulf, > > Thanks for your review comments. I replied inline > > >-----Original Message----- > >From: Ulf Hansson <ulf.hansson@linaro.org> > >Sent: Monday, November 16, 2020 7:41 PM > >To: Zulkifli, Muhammad Husaini <muhammad.husaini.zulkifli@intel.com> > >Cc: linux-mmc@vger.kernel.org; Linux ARM <linux-arm- > >kernel@lists.infradead.org>; Linux Kernel Mailing List <linux- > >kernel@vger.kernel.org>; Hunter, Adrian <adrian.hunter@intel.com>; Raja > >Subramanian, Lakshmi Bai <lakshmi.bai.raja.subramanian@intel.com>; Wan > >Mohamad, Wan Ahmad Zainie > ><wan.ahmad.zainie.wan.mohamad@intel.com>; David E. Box > ><david.e.box@linux.intel.com> > >Subject: Re: [PATCH v1 1/1] mmc: sdhci-of-arasan: Specify .clk_ops for Keem > >Bay SOC > > > >On Fri, 6 Nov 2020 at 05:10, <muhammad.husaini.zulkifli@intel.com> wrote: > >> > >> From: Muhammad Husaini Zulkifli <muhammad.husaini.zulkifli@intel.com> > >> > >> Commit 16ada730a759 ("mmc: sdhci-of-arasan: Modify clock operations > >> handling") introduces platform specific SDHCI clock operation. > >> > >> This patch declares the clock operation for Keem Bay. > >> Add clk_ops for SD, EMMC and SDIO operations. > > > >The above commit message doesn't really tell why or what goes on here. > >Can please try to clarify that. > > We missed out the clock operation handling patch (Commit 16ada730a759) > and did not handle the clk_ops for Keem Bay SOC devices. > These 2 patches (Commit 16ada730a759) and (Commit 36c6aadaae86) are merged at around the same time. > We catch the issue later when trying to boot into v5.10-rc1 with Keem Bay EVM. > That is why I created this patch to handle the clk_ops for Keem Bay devices. Alright, so the previous changes were simply not sufficient to complete the support. > > > > >> > >> Fixes: 36c6aadaae86 ("mmc: sdhci-of-arasan: Add support for Intel Keem > >> Bay") > > > >Is $subject patch fixing a bug/regression? > > This is to fix issue on previous commit. It is a bug fix. Has it ever worked? Did the commit you point to with the fixes tag, break some other existing functionality? [...] Kind regards Uffe ^ permalink raw reply [flat|nested] 10+ messages in thread
* RE: [PATCH v1 1/1] mmc: sdhci-of-arasan: Specify .clk_ops for Keem Bay SOC 2020-11-17 11:27 ` Ulf Hansson @ 2020-11-17 13:46 ` Zulkifli, Muhammad Husaini 2020-11-17 17:14 ` Ulf Hansson 0 siblings, 1 reply; 10+ messages in thread From: Zulkifli, Muhammad Husaini @ 2020-11-17 13:46 UTC (permalink / raw) To: Ulf Hansson Cc: linux-mmc, Linux ARM, Linux Kernel Mailing List, Hunter, Adrian, Raja Subramanian, Lakshmi Bai, Wan Mohamad, Wan Ahmad Zainie, David E. Box Hi Ulf, >-----Original Message----- >From: Ulf Hansson <ulf.hansson@linaro.org> >Sent: Tuesday, November 17, 2020 7:27 PM >To: Zulkifli, Muhammad Husaini <muhammad.husaini.zulkifli@intel.com> >Cc: linux-mmc@vger.kernel.org; Linux ARM <linux-arm- >kernel@lists.infradead.org>; Linux Kernel Mailing List <linux- >kernel@vger.kernel.org>; Hunter, Adrian <adrian.hunter@intel.com>; Raja >Subramanian, Lakshmi Bai <lakshmi.bai.raja.subramanian@intel.com>; Wan >Mohamad, Wan Ahmad Zainie ><wan.ahmad.zainie.wan.mohamad@intel.com>; David E. Box ><david.e.box@linux.intel.com> >Subject: Re: [PATCH v1 1/1] mmc: sdhci-of-arasan: Specify .clk_ops for Keem Bay >SOC > >On Mon, 16 Nov 2020 at 15:22, Zulkifli, Muhammad Husaini ><muhammad.husaini.zulkifli@intel.com> wrote: >> >> Hi Ulf, >> >> Thanks for your review comments. I replied inline >> >> >-----Original Message----- >> >From: Ulf Hansson <ulf.hansson@linaro.org> >> >Sent: Monday, November 16, 2020 7:41 PM >> >To: Zulkifli, Muhammad Husaini <muhammad.husaini.zulkifli@intel.com> >> >Cc: linux-mmc@vger.kernel.org; Linux ARM <linux-arm- >> >kernel@lists.infradead.org>; Linux Kernel Mailing List <linux- >> >kernel@vger.kernel.org>; Hunter, Adrian <adrian.hunter@intel.com>; >> >Raja Subramanian, Lakshmi Bai >> ><lakshmi.bai.raja.subramanian@intel.com>; Wan Mohamad, Wan Ahmad >> >Zainie <wan.ahmad.zainie.wan.mohamad@intel.com>; David E. Box >> ><david.e.box@linux.intel.com> >> >Subject: Re: [PATCH v1 1/1] mmc: sdhci-of-arasan: Specify .clk_ops >> >for Keem Bay SOC >> > >> >On Fri, 6 Nov 2020 at 05:10, <muhammad.husaini.zulkifli@intel.com> wrote: >> >> >> >> From: Muhammad Husaini Zulkifli >> >> <muhammad.husaini.zulkifli@intel.com> >> >> >> >> Commit 16ada730a759 ("mmc: sdhci-of-arasan: Modify clock operations >> >> handling") introduces platform specific SDHCI clock operation. >> >> >> >> This patch declares the clock operation for Keem Bay. >> >> Add clk_ops for SD, EMMC and SDIO operations. >> > >> >The above commit message doesn't really tell why or what goes on here. >> >Can please try to clarify that. >> >> We missed out the clock operation handling patch (Commit 16ada730a759) >> and did not handle the clk_ops for Keem Bay SOC devices. >> These 2 patches (Commit 16ada730a759) and (Commit 36c6aadaae86) are >merged at around the same time. >> We catch the issue later when trying to boot into v5.10-rc1 with Keem Bay >EVM. >> That is why I created this patch to handle the clk_ops for Keem Bay devices. > >Alright, so the previous changes were simply not sufficient to complete the >support. > >> >> > >> >> >> >> Fixes: 36c6aadaae86 ("mmc: sdhci-of-arasan: Add support for Intel >> >> Keem >> >> Bay") >> > >> >Is $subject patch fixing a bug/regression? >> >> This is to fix issue on previous commit. It is a bug fix. > >Has it ever worked? Yes it is working. Tested with Keem Bay EVM. SD/EMMC/SDIO devices registered successfully with this fix. > >Did the commit you point to with the fixes tag, break some other existing >functionality? It should not break other existing functionality. This is specific for Keem Bay SOC only. > >[...] > >Kind regards >Uffe ^ permalink raw reply [flat|nested] 10+ messages in thread
* Re: [PATCH v1 1/1] mmc: sdhci-of-arasan: Specify .clk_ops for Keem Bay SOC 2020-11-17 13:46 ` Zulkifli, Muhammad Husaini @ 2020-11-17 17:14 ` Ulf Hansson 2020-11-18 1:04 ` Zulkifli, Muhammad Husaini 0 siblings, 1 reply; 10+ messages in thread From: Ulf Hansson @ 2020-11-17 17:14 UTC (permalink / raw) To: Zulkifli, Muhammad Husaini Cc: linux-mmc, Linux ARM, Linux Kernel Mailing List, Hunter, Adrian, Raja Subramanian, Lakshmi Bai, Wan Mohamad, Wan Ahmad Zainie, David E. Box On Tue, 17 Nov 2020 at 14:46, Zulkifli, Muhammad Husaini <muhammad.husaini.zulkifli@intel.com> wrote: > > Hi Ulf, > > >-----Original Message----- > >From: Ulf Hansson <ulf.hansson@linaro.org> > >Sent: Tuesday, November 17, 2020 7:27 PM > >To: Zulkifli, Muhammad Husaini <muhammad.husaini.zulkifli@intel.com> > >Cc: linux-mmc@vger.kernel.org; Linux ARM <linux-arm- > >kernel@lists.infradead.org>; Linux Kernel Mailing List <linux- > >kernel@vger.kernel.org>; Hunter, Adrian <adrian.hunter@intel.com>; Raja > >Subramanian, Lakshmi Bai <lakshmi.bai.raja.subramanian@intel.com>; Wan > >Mohamad, Wan Ahmad Zainie > ><wan.ahmad.zainie.wan.mohamad@intel.com>; David E. Box > ><david.e.box@linux.intel.com> > >Subject: Re: [PATCH v1 1/1] mmc: sdhci-of-arasan: Specify .clk_ops for Keem Bay > >SOC > > > >On Mon, 16 Nov 2020 at 15:22, Zulkifli, Muhammad Husaini > ><muhammad.husaini.zulkifli@intel.com> wrote: > >> > >> Hi Ulf, > >> > >> Thanks for your review comments. I replied inline > >> > >> >-----Original Message----- > >> >From: Ulf Hansson <ulf.hansson@linaro.org> > >> >Sent: Monday, November 16, 2020 7:41 PM > >> >To: Zulkifli, Muhammad Husaini <muhammad.husaini.zulkifli@intel.com> > >> >Cc: linux-mmc@vger.kernel.org; Linux ARM <linux-arm- > >> >kernel@lists.infradead.org>; Linux Kernel Mailing List <linux- > >> >kernel@vger.kernel.org>; Hunter, Adrian <adrian.hunter@intel.com>; > >> >Raja Subramanian, Lakshmi Bai > >> ><lakshmi.bai.raja.subramanian@intel.com>; Wan Mohamad, Wan Ahmad > >> >Zainie <wan.ahmad.zainie.wan.mohamad@intel.com>; David E. Box > >> ><david.e.box@linux.intel.com> > >> >Subject: Re: [PATCH v1 1/1] mmc: sdhci-of-arasan: Specify .clk_ops > >> >for Keem Bay SOC > >> > > >> >On Fri, 6 Nov 2020 at 05:10, <muhammad.husaini.zulkifli@intel.com> wrote: > >> >> > >> >> From: Muhammad Husaini Zulkifli > >> >> <muhammad.husaini.zulkifli@intel.com> > >> >> > >> >> Commit 16ada730a759 ("mmc: sdhci-of-arasan: Modify clock operations > >> >> handling") introduces platform specific SDHCI clock operation. > >> >> > >> >> This patch declares the clock operation for Keem Bay. > >> >> Add clk_ops for SD, EMMC and SDIO operations. > >> > > >> >The above commit message doesn't really tell why or what goes on here. > >> >Can please try to clarify that. > >> > >> We missed out the clock operation handling patch (Commit 16ada730a759) > >> and did not handle the clk_ops for Keem Bay SOC devices. > >> These 2 patches (Commit 16ada730a759) and (Commit 36c6aadaae86) are > >merged at around the same time. > >> We catch the issue later when trying to boot into v5.10-rc1 with Keem Bay > >EVM. > >> That is why I created this patch to handle the clk_ops for Keem Bay devices. > > > >Alright, so the previous changes were simply not sufficient to complete the > >support. > > > >> > >> > > >> >> > >> >> Fixes: 36c6aadaae86 ("mmc: sdhci-of-arasan: Add support for Intel > >> >> Keem > >> >> Bay") > >> > > >> >Is $subject patch fixing a bug/regression? > >> > >> This is to fix issue on previous commit. It is a bug fix. > > > >Has it ever worked? > > Yes it is working. Tested with Keem Bay EVM. > SD/EMMC/SDIO devices registered successfully with this fix. I am not asking if $subject patch makes it work, I understand that it does. > > > > >Did the commit you point to with the fixes tag, break some other existing > >functionality? > > It should not break other existing functionality. > This is specific for Keem Bay SOC only. So, you are saying that it has never worked for Keem Bay? But $subject patch makes it work? Can you please re-spin and update the commit message to better reflect what goes on, then I can apply. Kind regards Uffe ^ permalink raw reply [flat|nested] 10+ messages in thread
* RE: [PATCH v1 1/1] mmc: sdhci-of-arasan: Specify .clk_ops for Keem Bay SOC 2020-11-17 17:14 ` Ulf Hansson @ 2020-11-18 1:04 ` Zulkifli, Muhammad Husaini 0 siblings, 0 replies; 10+ messages in thread From: Zulkifli, Muhammad Husaini @ 2020-11-18 1:04 UTC (permalink / raw) To: Ulf Hansson Cc: linux-mmc, Linux ARM, Linux Kernel Mailing List, Hunter, Adrian, Raja Subramanian, Lakshmi Bai, Wan Mohamad, Wan Ahmad Zainie, David E. Box Hi Ulf, >-----Original Message----- >From: Ulf Hansson <ulf.hansson@linaro.org> >Sent: Wednesday, November 18, 2020 1:14 AM >To: Zulkifli, Muhammad Husaini <muhammad.husaini.zulkifli@intel.com> >Cc: linux-mmc@vger.kernel.org; Linux ARM <linux-arm- >kernel@lists.infradead.org>; Linux Kernel Mailing List <linux- >kernel@vger.kernel.org>; Hunter, Adrian <adrian.hunter@intel.com>; Raja >Subramanian, Lakshmi Bai <lakshmi.bai.raja.subramanian@intel.com>; Wan >Mohamad, Wan Ahmad Zainie ><wan.ahmad.zainie.wan.mohamad@intel.com>; David E. Box ><david.e.box@linux.intel.com> >Subject: Re: [PATCH v1 1/1] mmc: sdhci-of-arasan: Specify .clk_ops for Keem Bay >SOC > >On Tue, 17 Nov 2020 at 14:46, Zulkifli, Muhammad Husaini ><muhammad.husaini.zulkifli@intel.com> wrote: >> >> Hi Ulf, >> >> >-----Original Message----- >> >From: Ulf Hansson <ulf.hansson@linaro.org> >> >Sent: Tuesday, November 17, 2020 7:27 PM >> >To: Zulkifli, Muhammad Husaini <muhammad.husaini.zulkifli@intel.com> >> >Cc: linux-mmc@vger.kernel.org; Linux ARM <linux-arm- >> >kernel@lists.infradead.org>; Linux Kernel Mailing List <linux- >> >kernel@vger.kernel.org>; Hunter, Adrian <adrian.hunter@intel.com>; >> >Raja Subramanian, Lakshmi Bai >> ><lakshmi.bai.raja.subramanian@intel.com>; Wan Mohamad, Wan Ahmad >> >Zainie <wan.ahmad.zainie.wan.mohamad@intel.com>; David E. Box >> ><david.e.box@linux.intel.com> >> >Subject: Re: [PATCH v1 1/1] mmc: sdhci-of-arasan: Specify .clk_ops >> >for Keem Bay SOC >> > >> >On Mon, 16 Nov 2020 at 15:22, Zulkifli, Muhammad Husaini >> ><muhammad.husaini.zulkifli@intel.com> wrote: >> >> >> >> Hi Ulf, >> >> >> >> Thanks for your review comments. I replied inline >> >> >> >> >-----Original Message----- >> >> >From: Ulf Hansson <ulf.hansson@linaro.org> >> >> >Sent: Monday, November 16, 2020 7:41 PM >> >> >To: Zulkifli, Muhammad Husaini >> >> ><muhammad.husaini.zulkifli@intel.com> >> >> >Cc: linux-mmc@vger.kernel.org; Linux ARM <linux-arm- >> >> >kernel@lists.infradead.org>; Linux Kernel Mailing List <linux- >> >> >kernel@vger.kernel.org>; Hunter, Adrian <adrian.hunter@intel.com>; >> >> >Raja Subramanian, Lakshmi Bai >> >> ><lakshmi.bai.raja.subramanian@intel.com>; Wan Mohamad, Wan >Ahmad >> >> >Zainie <wan.ahmad.zainie.wan.mohamad@intel.com>; David E. Box >> >> ><david.e.box@linux.intel.com> >> >> >Subject: Re: [PATCH v1 1/1] mmc: sdhci-of-arasan: Specify .clk_ops >> >> >for Keem Bay SOC >> >> > >> >> >On Fri, 6 Nov 2020 at 05:10, <muhammad.husaini.zulkifli@intel.com> >wrote: >> >> >> >> >> >> From: Muhammad Husaini Zulkifli >> >> >> <muhammad.husaini.zulkifli@intel.com> >> >> >> >> >> >> Commit 16ada730a759 ("mmc: sdhci-of-arasan: Modify clock >> >> >> operations >> >> >> handling") introduces platform specific SDHCI clock operation. >> >> >> >> >> >> This patch declares the clock operation for Keem Bay. >> >> >> Add clk_ops for SD, EMMC and SDIO operations. >> >> > >> >> >The above commit message doesn't really tell why or what goes on here. >> >> >Can please try to clarify that. >> >> >> >> We missed out the clock operation handling patch (Commit >> >> 16ada730a759) and did not handle the clk_ops for Keem Bay SOC devices. >> >> These 2 patches (Commit 16ada730a759) and (Commit 36c6aadaae86) >> >> are >> >merged at around the same time. >> >> We catch the issue later when trying to boot into v5.10-rc1 with >> >> Keem Bay >> >EVM. >> >> That is why I created this patch to handle the clk_ops for Keem Bay >devices. >> > >> >Alright, so the previous changes were simply not sufficient to >> >complete the support. >> > >> >> >> >> > >> >> >> >> >> >> Fixes: 36c6aadaae86 ("mmc: sdhci-of-arasan: Add support for >> >> >> Intel Keem >> >> >> Bay") >> >> > >> >> >Is $subject patch fixing a bug/regression? >> >> >> >> This is to fix issue on previous commit. It is a bug fix. >> > >> >Has it ever worked? >> >> Yes it is working. Tested with Keem Bay EVM. >> SD/EMMC/SDIO devices registered successfully with this fix. > >I am not asking if $subject patch makes it work, I understand that it does. > >> >> > >> >Did the commit you point to with the fixes tag, break some other >> >existing functionality? >> >> It should not break other existing functionality. >> This is specific for Keem Bay SOC only. > >So, you are saying that it has never worked for Keem Bay? But $subject patch >makes it work? It is working for Keem Bay before until the Commit 16ada730a759 ("mmc: sdhci-of-arasan: Modify clock operations handling") came into the picture. Both Commit 16ada730a759 ("mmc: sdhci-of-arasan: Modify clock operations handling") and Commit 36c6aadaae86 ("mmc: sdhci-of-arasan: Add support for Intel Keem") are merged about the same time. We did not have "Modify clock operations handling" patch when rebasing and testing during upstreaming process. I noticed of this issue when using latest mainline v5.10-rcx recently. > >Can you please re-spin and update the commit message to better reflect what >goes on, then I can apply. Noted. I will update the commit message and re-send again. Thanks > >Kind regards >Uffe ^ permalink raw reply [flat|nested] 10+ messages in thread
* RE: [PATCH v1 0/1] mmc: sdhci-of-arasan: Specify .clk_ops for Keem Bay SOC 2020-11-06 12:09 [PATCH v1 0/1] mmc: sdhci-of-arasan: Specify .clk_ops for Keem Bay SOC muhammad.husaini.zulkifli 2020-11-06 12:09 ` [PATCH v1 1/1] " muhammad.husaini.zulkifli @ 2020-11-12 14:15 ` Zulkifli, Muhammad Husaini [not found] ` <DM6PR11MB28767CEEAD0FA1E20614B797B8E60@DM6PR11MB2876.namprd11.prod.outlook.com> 1 sibling, 1 reply; 10+ messages in thread From: Zulkifli, Muhammad Husaini @ 2020-11-12 14:15 UTC (permalink / raw) To: linux-mmc, linux-arm-kernel, linux-kernel Cc: Hunter, Adrian, Raja Subramanian, Lakshmi Bai, Wan Mohamad, Wan Ahmad Zainie, david.e.box Hi, I would appreciate hearing your opinion/review on this patch. Thanks >-----Original Message----- >From: Zulkifli, Muhammad Husaini <muhammad.husaini.zulkifli@intel.com> >Sent: Friday, November 6, 2020 8:10 PM >To: linux-mmc@vger.kernel.org; linux-arm-kernel@lists.infradead.org; linux- >kernel@vger.kernel.org >Cc: Hunter, Adrian <adrian.hunter@intel.com>; Raja Subramanian, Lakshmi >Bai <lakshmi.bai.raja.subramanian@intel.com>; Wan Mohamad, Wan Ahmad >Zainie <wan.ahmad.zainie.wan.mohamad@intel.com>; Zulkifli, Muhammad >Husaini <muhammad.husaini.zulkifli@intel.com>; >david.e.box@linux.intel.com >Subject: [PATCH v1 0/1] mmc: sdhci-of-arasan: Specify .clk_ops for Keem Bay >SOC > >From: Muhammad Husaini Zulkifli <muhammad.husaini.zulkifli@intel.com> > >Hi. > >This patch declares the .clk_ops for Keem Bay SOC due to the introduction of >new clock operations handling. > >This was tested with Keem Bay evaluation module board. > >Kindly help to review it. > >Thank you. > >Muhammad Husaini Zulkifli (1): > mmc: sdhci-of-arasan: Specify .clk_ops for Keem Bay SOC > > drivers/mmc/host/sdhci-of-arasan.c | 3 +++ > 1 file changed, 3 insertions(+) > >-- >2.17.1 ^ permalink raw reply [flat|nested] 10+ messages in thread
[parent not found: <DM6PR11MB28767CEEAD0FA1E20614B797B8E60@DM6PR11MB2876.namprd11.prod.outlook.com>]
* RE: [PATCH v1 0/1] mmc: sdhci-of-arasan: Specify .clk_ops for Keem Bay SOC [not found] ` <DM6PR11MB28767CEEAD0FA1E20614B797B8E60@DM6PR11MB2876.namprd11.prod.outlook.com> @ 2020-11-13 0:15 ` Zulkifli, Muhammad Husaini 0 siblings, 0 replies; 10+ messages in thread From: Zulkifli, Muhammad Husaini @ 2020-11-13 0:15 UTC (permalink / raw) To: linux-mmc, linux-arm-kernel, linux-kernel, Ulf Hansson Cc: Hunter, Adrian, Raja Subramanian, Lakshmi Bai, Wan Mohamad, Wan Ahmad Zainie, david.e.box >-----Original Message----- >From: Zulkifli, Muhammad Husaini >Sent: Friday, November 13, 2020 8:14 AM >To: 'linux-mmc@vger.kernel.org' <linux-mmc@vger.kernel.org>; 'linux-arm- >kernel@lists.infradead.org' <linux-arm-kernel@lists.infradead.org>; 'linux- >kernel@vger.kernel.org' <linux-kernel@vger.kernel.org>; 'Ulf Hansson ><ulf.hansson@linaro.org>' <Ulf Hansson <ulf.hansson@linaro.org>> >Cc: Hunter, Adrian <adrian.hunter@intel.com>; Raja Subramanian, Lakshmi >Bai <lakshmi.bai.raja.subramanian@intel.com>; Wan Mohamad, Wan Ahmad >Zainie <wan.ahmad.zainie.wan.mohamad@intel.com>; >'david.e.box@linux.intel.com' <david.e.box@linux.intel.com> >Subject: RE: [PATCH v1 0/1] mmc: sdhci-of-arasan: Specify .clk_ops for Keem >Bay SOC > >++ Ulf > >>-----Original Message----- >>From: Zulkifli, Muhammad Husaini >>Sent: Thursday, November 12, 2020 10:15 PM >>To: linux-mmc@vger.kernel.org; linux-arm-kernel@lists.infradead.org; >>linux- kernel@vger.kernel.org >>Cc: Hunter, Adrian <adrian.hunter@intel.com>; Raja Subramanian, Lakshmi >>Bai <lakshmi.bai.raja.subramanian@intel.com>; Wan Mohamad, Wan >Ahmad >>Zainie <wan.ahmad.zainie.wan.mohamad@intel.com>; >>david.e.box@linux.intel.com >>Subject: RE: [PATCH v1 0/1] mmc: sdhci-of-arasan: Specify .clk_ops for >>Keem Bay SOC >> >>Hi, >> >>I would appreciate hearing your opinion/review on this patch. >> >>Thanks >> >>>-----Original Message----- >>>From: Zulkifli, Muhammad Husaini <muhammad.husaini.zulkifli@intel.com> >>>Sent: Friday, November 6, 2020 8:10 PM >>>To: linux-mmc@vger.kernel.org; linux-arm-kernel@lists.infradead.org; >>>linux- kernel@vger.kernel.org >>>Cc: Hunter, Adrian <adrian.hunter@intel.com>; Raja Subramanian, >>>Lakshmi Bai <lakshmi.bai.raja.subramanian@intel.com>; Wan Mohamad, >Wan >>Ahmad >>>Zainie <wan.ahmad.zainie.wan.mohamad@intel.com>; Zulkifli, Muhammad >>>Husaini <muhammad.husaini.zulkifli@intel.com>; >>>david.e.box@linux.intel.com >>>Subject: [PATCH v1 0/1] mmc: sdhci-of-arasan: Specify .clk_ops for >>>Keem Bay SOC >>> >>>From: Muhammad Husaini Zulkifli <muhammad.husaini.zulkifli@intel.com> >>> >>>Hi. >>> >>>This patch declares the .clk_ops for Keem Bay SOC due to the >>>introduction of new clock operations handling. >>> >>>This was tested with Keem Bay evaluation module board. >>> >>>Kindly help to review it. >>> >>>Thank you. >>> >>>Muhammad Husaini Zulkifli (1): >>> mmc: sdhci-of-arasan: Specify .clk_ops for Keem Bay SOC >>> >>> drivers/mmc/host/sdhci-of-arasan.c | 3 +++ >>> 1 file changed, 3 insertions(+) >>> >>>-- >>>2.17.1 ^ permalink raw reply [flat|nested] 10+ messages in thread
end of thread, other threads:[~2020-11-18 1:05 UTC | newest]
Thread overview: 10+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2020-11-06 12:09 [PATCH v1 0/1] mmc: sdhci-of-arasan: Specify .clk_ops for Keem Bay SOC muhammad.husaini.zulkifli
2020-11-06 12:09 ` [PATCH v1 1/1] " muhammad.husaini.zulkifli
2020-11-16 11:41 ` Ulf Hansson
2020-11-16 14:22 ` Zulkifli, Muhammad Husaini
2020-11-17 11:27 ` Ulf Hansson
2020-11-17 13:46 ` Zulkifli, Muhammad Husaini
2020-11-17 17:14 ` Ulf Hansson
2020-11-18 1:04 ` Zulkifli, Muhammad Husaini
2020-11-12 14:15 ` [PATCH v1 0/1] " Zulkifli, Muhammad Husaini
[not found] ` <DM6PR11MB28767CEEAD0FA1E20614B797B8E60@DM6PR11MB2876.namprd11.prod.outlook.com>
2020-11-13 0:15 ` Zulkifli, Muhammad Husaini
This is a public inbox, see mirroring instructions for how to clone and mirror all data and code used for this inbox; as well as URLs for NNTP newsgroup(s).