From: Thomas Abraham <thomas.abraham@linaro.org> To: Jaehoon Chung <jh80.chung@samsung.com> Cc: linux-mmc@vger.kernel.org, devicetree-discuss@lists.ozlabs.org, linux-kernel@vger.kernel.org, linux-arm-kernel@lists.infradead.org, cjb@laptop.org, grant.likely@secretlab.ca, rob.herring@calxeda.com, linux-samsung-soc@vger.kernel.org, kgene.kim@samsung.com, patches@linaro.org Subject: Re: [PATCH v3 0/6] mmc: dw_mmc: add support for device tree based instantiation Date: Fri, 20 Jul 2012 00:29:08 +0530 [thread overview] Message-ID: <CAJuYYwQBgpJ1wiS3fSDS2cMi6kanuO47rc8SmniAqjRPuBGJHg@mail.gmail.com> (raw) In-Reply-To: <5008279B.4000001@samsung.com> On 19 July 2012 20:58, Jaehoon Chung <jh80.chung@samsung.com> wrote: > Hi Thomas, > > I think not good that added the samsung specific code into dw_mmc-pltfm.c > How about separating to dw-mmc-exynos.c? I am not sure of this. The only samsung specific code in dw_mmc-pltfm.c file is the data for of_device_id instances. The clock lookup added into this file in the 3rd patch does not cause any harm on non-samsung SoC's which might not define those clocks (on clock lookup failure, there are only warning printed, the driver's probe does not fail. I would prefer not to add separate file for Exynos SoC's for now. Splitting into different files will need to defined new callbacks which I fell is not really required. Thanks, Thomas. > > Best Regards, > Jaehoon Chung > > On 07/12/2012 09:54 PM, Thomas Abraham wrote: >> This patch series adds device tree support for Synopsis Designware Mobile >> Storage Host Controller. >> >> The first patch converts the copy of controller device instance into a >> reference. This is need to allow device resource management api to correctly >> manage the resources allocated by the driver. The second patch fixes the >> incorrect abort of the probe in case a slot initialization fails. This is >> fixed by allowing as many slots to be initialized successfully and failing >> only if there are no slots that were initialized. >> >> The third patch adds clock lookup in the driver and this is optional. Platforms >> that do not need any clock gating and control for the dw_mmc controllers will >> not be affected with this change. The fourth patch adds a quirk to notify the >> controller about the absence of the write protect line. >> >> The fifth patch adds device tree based discovery support for the dw_mmc driver. >> The sixth patch add Samsung Exynos5250 specific extentions to the driver. >> >> This patchset is based on Samsung kernel tree's for-next branch with the >> mmc tree's mmc-next branch merged. >> >> Thomas Abraham (6): >> mmc: dw_mmc: convert copy of struct device in struct dw_mci to a reference >> mmc: dw_mmc: allow probe to succeed even if one slot is initialized >> mmc: dw_mmc: lookup for optional biu and ciu clocks >> mmc: dw_mmc: add quirk to indicate missing write protect line >> mmc: dw_mmc: add device tree support >> mmc: dw_mmc: add samsung exynos5250 specific extentions >> >> .../devicetree/bindings/mmc/synposis-dw-mshc.txt | 144 ++++++++ >> drivers/mmc/host/dw_mmc-pci.c | 2 +- >> drivers/mmc/host/dw_mmc-pltfm.c | 41 +++- >> drivers/mmc/host/dw_mmc.c | 364 +++++++++++++++++--- >> drivers/mmc/host/dw_mmc.h | 23 ++ >> include/linux/mmc/dw_mmc.h | 17 +- >> 6 files changed, 538 insertions(+), 53 deletions(-) >> create mode 100644 Documentation/devicetree/bindings/mmc/synposis-dw-mshc.txt >> >> -- >> To unsubscribe from this list: send the line "unsubscribe linux-mmc" in >> the body of a message to majordomo@vger.kernel.org >> More majordomo info at http://vger.kernel.org/majordomo-info.html >> > >
WARNING: multiple messages have this Message-ID (diff)
From: thomas.abraham@linaro.org (Thomas Abraham) To: linux-arm-kernel@lists.infradead.org Subject: [PATCH v3 0/6] mmc: dw_mmc: add support for device tree based instantiation Date: Fri, 20 Jul 2012 00:29:08 +0530 [thread overview] Message-ID: <CAJuYYwQBgpJ1wiS3fSDS2cMi6kanuO47rc8SmniAqjRPuBGJHg@mail.gmail.com> (raw) In-Reply-To: <5008279B.4000001@samsung.com> On 19 July 2012 20:58, Jaehoon Chung <jh80.chung@samsung.com> wrote: > Hi Thomas, > > I think not good that added the samsung specific code into dw_mmc-pltfm.c > How about separating to dw-mmc-exynos.c? I am not sure of this. The only samsung specific code in dw_mmc-pltfm.c file is the data for of_device_id instances. The clock lookup added into this file in the 3rd patch does not cause any harm on non-samsung SoC's which might not define those clocks (on clock lookup failure, there are only warning printed, the driver's probe does not fail. I would prefer not to add separate file for Exynos SoC's for now. Splitting into different files will need to defined new callbacks which I fell is not really required. Thanks, Thomas. > > Best Regards, > Jaehoon Chung > > On 07/12/2012 09:54 PM, Thomas Abraham wrote: >> This patch series adds device tree support for Synopsis Designware Mobile >> Storage Host Controller. >> >> The first patch converts the copy of controller device instance into a >> reference. This is need to allow device resource management api to correctly >> manage the resources allocated by the driver. The second patch fixes the >> incorrect abort of the probe in case a slot initialization fails. This is >> fixed by allowing as many slots to be initialized successfully and failing >> only if there are no slots that were initialized. >> >> The third patch adds clock lookup in the driver and this is optional. Platforms >> that do not need any clock gating and control for the dw_mmc controllers will >> not be affected with this change. The fourth patch adds a quirk to notify the >> controller about the absence of the write protect line. >> >> The fifth patch adds device tree based discovery support for the dw_mmc driver. >> The sixth patch add Samsung Exynos5250 specific extentions to the driver. >> >> This patchset is based on Samsung kernel tree's for-next branch with the >> mmc tree's mmc-next branch merged. >> >> Thomas Abraham (6): >> mmc: dw_mmc: convert copy of struct device in struct dw_mci to a reference >> mmc: dw_mmc: allow probe to succeed even if one slot is initialized >> mmc: dw_mmc: lookup for optional biu and ciu clocks >> mmc: dw_mmc: add quirk to indicate missing write protect line >> mmc: dw_mmc: add device tree support >> mmc: dw_mmc: add samsung exynos5250 specific extentions >> >> .../devicetree/bindings/mmc/synposis-dw-mshc.txt | 144 ++++++++ >> drivers/mmc/host/dw_mmc-pci.c | 2 +- >> drivers/mmc/host/dw_mmc-pltfm.c | 41 +++- >> drivers/mmc/host/dw_mmc.c | 364 +++++++++++++++++--- >> drivers/mmc/host/dw_mmc.h | 23 ++ >> include/linux/mmc/dw_mmc.h | 17 +- >> 6 files changed, 538 insertions(+), 53 deletions(-) >> create mode 100644 Documentation/devicetree/bindings/mmc/synposis-dw-mshc.txt >> >> -- >> To unsubscribe from this list: send the line "unsubscribe linux-mmc" in >> the body of a message to majordomo at vger.kernel.org >> More majordomo info at http://vger.kernel.org/majordomo-info.html >> > >
next prev parent reply other threads:[~2012-07-19 18:59 UTC|newest] Thread overview: 53+ messages / expand[flat|nested] mbox.gz Atom feed top 2012-07-12 12:54 [PATCH v3 0/6] mmc: dw_mmc: add support for device tree based instantiation Thomas Abraham 2012-07-12 12:54 ` Thomas Abraham 2012-07-12 12:54 ` [PATCH v3 1/6] mmc: dw_mmc: convert copy of struct device in struct dw_mci to a reference Thomas Abraham 2012-07-12 12:54 ` Thomas Abraham 2012-07-12 12:54 ` [PATCH v3 2/6] mmc: dw_mmc: allow probe to succeed even if one slot is initialized Thomas Abraham 2012-07-12 12:54 ` Thomas Abraham 2012-07-12 12:54 ` Thomas Abraham 2012-07-12 12:54 ` [PATCH v3 3/6] mmc: dw_mmc: lookup for optional biu and ciu clocks Thomas Abraham 2012-07-12 12:54 ` Thomas Abraham 2012-07-12 12:54 ` Thomas Abraham 2012-07-13 9:11 ` Girish K S 2012-07-13 9:11 ` Girish K S 2012-07-17 10:38 ` [PATCH 3/9] " Thomas Abraham 2012-07-17 10:38 ` Thomas Abraham 2012-07-19 10:27 ` Seungwon Jeon 2012-07-19 10:27 ` Seungwon Jeon 2012-07-12 12:54 ` [PATCH v3 4/6] mmc: dw_mmc: add quirk to indicate missing write protect line Thomas Abraham 2012-07-12 12:54 ` Thomas Abraham 2012-07-12 12:54 ` [PATCH v3 5/6] mmc: dw_mmc: add device tree support Thomas Abraham 2012-07-12 12:54 ` Thomas Abraham 2012-07-16 4:15 ` Girish K S 2012-07-16 4:15 ` Girish K S 2012-07-16 4:30 ` Girish K S 2012-07-16 4:30 ` Girish K S 2012-07-17 10:19 ` Thomas Abraham 2012-07-17 10:19 ` Thomas Abraham 2012-07-16 4:23 ` Chris Ball 2012-07-16 4:23 ` Chris Ball 2012-07-17 10:21 ` Thomas Abraham 2012-07-17 10:21 ` Thomas Abraham 2012-07-17 10:21 ` Thomas Abraham 2012-07-17 10:37 ` [PATCH 5/9] " Thomas Abraham 2012-07-17 10:37 ` Thomas Abraham 2012-07-12 12:54 ` [PATCH v3 6/6] mmc: dw_mmc: add samsung exynos5250 specific extentions Thomas Abraham 2012-07-12 12:54 ` Thomas Abraham 2012-07-19 3:51 ` Seungwon Jeon 2012-07-19 3:51 ` Seungwon Jeon 2012-07-19 18:48 ` Thomas Abraham 2012-07-19 18:48 ` Thomas Abraham 2012-07-20 10:38 ` Seungwon Jeon 2012-07-20 10:38 ` Seungwon Jeon 2012-07-23 6:17 ` Thomas Abraham 2012-07-23 6:17 ` Thomas Abraham 2012-07-23 9:59 ` Seungwon Jeon 2012-07-23 9:59 ` Seungwon Jeon 2012-07-19 15:28 ` [PATCH v3 0/6] mmc: dw_mmc: add support for device tree based instantiation Jaehoon Chung 2012-07-19 15:28 ` Jaehoon Chung 2012-07-19 18:59 ` Thomas Abraham [this message] 2012-07-19 18:59 ` Thomas Abraham 2012-07-19 22:47 ` Kukjin Kim 2012-07-19 22:47 ` Kukjin Kim 2012-07-23 1:57 ` Jaehoon Chung 2012-07-23 1:57 ` Jaehoon Chung
Reply instructions: You may reply publicly to this message via plain-text email using any one of the following methods: * Save the following mbox file, import it into your mail client, and reply-to-all from there: mbox Avoid top-posting and favor interleaved quoting: https://en.wikipedia.org/wiki/Posting_style#Interleaved_style * Reply using the --to, --cc, and --in-reply-to switches of git-send-email(1): git send-email \ --in-reply-to=CAJuYYwQBgpJ1wiS3fSDS2cMi6kanuO47rc8SmniAqjRPuBGJHg@mail.gmail.com \ --to=thomas.abraham@linaro.org \ --cc=cjb@laptop.org \ --cc=devicetree-discuss@lists.ozlabs.org \ --cc=grant.likely@secretlab.ca \ --cc=jh80.chung@samsung.com \ --cc=kgene.kim@samsung.com \ --cc=linux-arm-kernel@lists.infradead.org \ --cc=linux-kernel@vger.kernel.org \ --cc=linux-mmc@vger.kernel.org \ --cc=linux-samsung-soc@vger.kernel.org \ --cc=patches@linaro.org \ --cc=rob.herring@calxeda.com \ /path/to/YOUR_REPLY https://kernel.org/pub/software/scm/git/docs/git-send-email.html * If your mail client supports setting the In-Reply-To header via mailto: links, try the mailto: linkBe sure your reply has a Subject: header at the top and a blank line before the message body.
This is an external index of several public inboxes, see mirroring instructions on how to clone and mirror all data and code used by this external index.