* Re: [PATCH 5/7] mmc: sdhci-of: support generic OF controllers [not found] ` <1289939635-30742-6-git-send-email-robherring2@gmail.com> @ 2010-11-16 21:32 ` Wolfram Sang 0 siblings, 0 replies; 4+ messages in thread From: Wolfram Sang @ 2010-11-16 21:32 UTC (permalink / raw) To: Rob Herring; +Cc: devicetree-discuss, Rob Herring, linux-mmc [-- Attachment #1: Type: text/plain, Size: 1666 bytes --] On Tue, Nov 16, 2010 at 02:33:53PM -0600, Rob Herring wrote: > From: Rob Herring <rob.herring@calxeda.com> > > The base sdhci driver requires a valid ops struct. Add empty struct to > sdhci-of to allow generic controllers which don't need custom ops functions. > > Signed-off-by: Rob Herring <rob.herring@calxeda.com> Acked-by: Wolfram Sang <w.sang@pengutronix.de> For the sdhci-patches, you should add linux-mmc@vger.kernel.org to the CC-list. Done now... > --- > drivers/mmc/host/sdhci-of-core.c | 5 +++++ > 1 files changed, 5 insertions(+), 0 deletions(-) > > diff --git a/drivers/mmc/host/sdhci-of-core.c b/drivers/mmc/host/sdhci-of-core.c > index dd84124..de292fe 100644 > --- a/drivers/mmc/host/sdhci-of-core.c > +++ b/drivers/mmc/host/sdhci-of-core.c > @@ -30,6 +30,9 @@ > #include "sdhci-of.h" > #include "sdhci.h" > > +static struct sdhci_ops sdhci_of_ops = { > +}; > + > #ifdef CONFIG_MMC_SDHCI_BIG_ENDIAN_32BIT_BYTE_SWAPPER > > /* > @@ -161,6 +164,8 @@ static int __devinit sdhci_of_probe(struct platform_device *ofdev, > if (sdhci_of_data) { > host->quirks = sdhci_of_data->quirks; > host->ops = &sdhci_of_data->ops; > + } else { > + host->ops = &sdhci_of_ops; > } > > if (of_get_property(np, "sdhci,auto-cmd12", NULL)) > -- > 1.7.1 > > _______________________________________________ > devicetree-discuss mailing list > devicetree-discuss@lists.ozlabs.org > https://lists.ozlabs.org/listinfo/devicetree-discuss -- Pengutronix e.K. | Wolfram Sang | Industrial Linux Solutions | http://www.pengutronix.de/ | [-- Attachment #2: Digital signature --] [-- Type: application/pgp-signature, Size: 197 bytes --] ^ permalink raw reply [flat|nested] 4+ messages in thread
[parent not found: <1289939635-30742-5-git-send-email-robherring2@gmail.com>]
* Re: [PATCH 4/7] mmc: sdhci-of: fix build on non-powerpc platforms [not found] ` <1289939635-30742-5-git-send-email-robherring2@gmail.com> @ 2010-11-16 21:44 ` Wolfram Sang 2010-11-16 22:34 ` Rob Herring 0 siblings, 1 reply; 4+ messages in thread From: Wolfram Sang @ 2010-11-16 21:44 UTC (permalink / raw) To: Rob Herring; +Cc: devicetree-discuss, Rob Herring, linux-mmc [-- Attachment #1: Type: text/plain, Size: 1956 bytes --] On Tue, Nov 16, 2010 at 02:33:52PM -0600, Rob Herring wrote: > From: Rob Herring <rob.herring@calxeda.com> > > Explicitly include err.h, of_address.h and of_irq.h. > Make use of machine_is() conditional on PPC. > > Signed-off-by: Rob Herring <rob.herring@calxeda.com> Hmm, sins of the past :/ I wonder if we can get away with less #ifdeffery, will think about it... > --- > drivers/mmc/host/sdhci-of-core.c | 9 +++++++++ > 1 files changed, 9 insertions(+), 0 deletions(-) > > diff --git a/drivers/mmc/host/sdhci-of-core.c b/drivers/mmc/host/sdhci-of-core.c > index fa19d84..dd84124 100644 > --- a/drivers/mmc/host/sdhci-of-core.c > +++ b/drivers/mmc/host/sdhci-of-core.c > @@ -13,6 +13,7 @@ > * your option) any later version. > */ > > +#include <linux/err.h> > #include <linux/module.h> > #include <linux/init.h> > #include <linux/io.h> > @@ -20,8 +21,12 @@ > #include <linux/delay.h> > #include <linux/of.h> > #include <linux/of_platform.h> > +#include <linux/of_address.h> > +#include <linux/of_irq.h> > #include <linux/mmc/host.h> > +#ifdef CONFIG_PPC > #include <asm/machdep.h> > +#endif > #include "sdhci-of.h" > #include "sdhci.h" > > @@ -112,7 +117,11 @@ static bool __devinit sdhci_of_wp_inverted(struct device_node *np) > return true; > > /* Old device trees don't have the wp-inverted property. */ > +#ifdef CONFIG_PPC > return machine_is(mpc837x_rdb) || machine_is(mpc837x_mds); > +#else > + return false; > +#endif > } > > static int __devinit sdhci_of_probe(struct platform_device *ofdev, > -- > 1.7.1 > > _______________________________________________ > devicetree-discuss mailing list > devicetree-discuss@lists.ozlabs.org > https://lists.ozlabs.org/listinfo/devicetree-discuss -- Pengutronix e.K. | Wolfram Sang | Industrial Linux Solutions | http://www.pengutronix.de/ | [-- Attachment #2: Digital signature --] [-- Type: application/pgp-signature, Size: 197 bytes --] ^ permalink raw reply [flat|nested] 4+ messages in thread
* Re: [PATCH 4/7] mmc: sdhci-of: fix build on non-powerpc platforms 2010-11-16 21:44 ` [PATCH 4/7] mmc: sdhci-of: fix build on non-powerpc platforms Wolfram Sang @ 2010-11-16 22:34 ` Rob Herring 2010-11-17 5:32 ` Grant Likely 0 siblings, 1 reply; 4+ messages in thread From: Rob Herring @ 2010-11-16 22:34 UTC (permalink / raw) To: Wolfram Sang; +Cc: devicetree-discuss, Rob Herring, linux-mmc On 11/16/2010 03:44 PM, Wolfram Sang wrote: > On Tue, Nov 16, 2010 at 02:33:52PM -0600, Rob Herring wrote: >> From: Rob Herring<rob.herring@calxeda.com> >> >> Explicitly include err.h, of_address.h and of_irq.h. >> Make use of machine_is() conditional on PPC. >> >> Signed-off-by: Rob Herring<rob.herring@calxeda.com> > > Hmm, sins of the past :/ I wonder if we can get away with less #ifdeffery, will > think about it... > I don't want to start a long debate, but is updating a kernel without updating the dtb really something to worry about? Isn't a year enough of a transition period. Do these machines have a machine level compatible property that could be used instead? Rob ^ permalink raw reply [flat|nested] 4+ messages in thread
* Re: [PATCH 4/7] mmc: sdhci-of: fix build on non-powerpc platforms 2010-11-16 22:34 ` Rob Herring @ 2010-11-17 5:32 ` Grant Likely 0 siblings, 0 replies; 4+ messages in thread From: Grant Likely @ 2010-11-17 5:32 UTC (permalink / raw) To: Rob Herring; +Cc: Wolfram Sang, devicetree-discuss, linux-mmc, Rob Herring On Tue, Nov 16, 2010 at 04:34:56PM -0600, Rob Herring wrote: > On 11/16/2010 03:44 PM, Wolfram Sang wrote: > >On Tue, Nov 16, 2010 at 02:33:52PM -0600, Rob Herring wrote: > >>From: Rob Herring<rob.herring@calxeda.com> > >> > >>Explicitly include err.h, of_address.h and of_irq.h. > >>Make use of machine_is() conditional on PPC. > >> > >>Signed-off-by: Rob Herring<rob.herring@calxeda.com> > > > >Hmm, sins of the past :/ I wonder if we can get away with less #ifdeffery, will > >think about it... > > > > I don't want to start a long debate, but is updating a kernel > without updating the dtb really something to worry about? Yes, once a .dtb is merged we try very hard not to break it. It may need to be updated to enable more features, but the goal is to not regress. One of the reason being that firmware may provide a default, but old, dtb and it is important to still be able to boot on those systems, even if the dtb is immediately going to be updated. That's one of the reasons why it is so important to document and review bindings up front and make sure they make sense before we commit to them. That being said, there are other ways to deal with old dtbs, like fixing up the data at platform setup time. > Isn't a year enough of a transition period. No. g. ^ permalink raw reply [flat|nested] 4+ messages in thread
end of thread, other threads:[~2010-11-17 5:32 UTC | newest]
Thread overview: 4+ messages (download: mbox.gz follow: Atom feed
-- links below jump to the message on this page --
[not found] <1289939635-30742-1-git-send-email-robherring2@gmail.com>
[not found] ` <1289939635-30742-6-git-send-email-robherring2@gmail.com>
2010-11-16 21:32 ` [PATCH 5/7] mmc: sdhci-of: support generic OF controllers Wolfram Sang
[not found] ` <1289939635-30742-5-git-send-email-robherring2@gmail.com>
2010-11-16 21:44 ` [PATCH 4/7] mmc: sdhci-of: fix build on non-powerpc platforms Wolfram Sang
2010-11-16 22:34 ` Rob Herring
2010-11-17 5:32 ` Grant Likely
This is a public inbox, see mirroring instructions for how to clone and mirror all data and code used for this inbox