* Russell King - ARM Linux [090316 03:17]: > On Mon, Mar 16, 2009 at 12:05:07PM +0200, Adrian Hunter wrote: > >> From 9b04f19a7e88f8877176c25845c38676ee3127c1 Mon Sep 17 00:00:00 2001 > > From: Adrian Hunter > > Date: Tue, 10 Mar 2009 22:13:52 +0100 > > Subject: [PATCH] ARM: OMAP3: mmc-twl4030 allow arbitrary slot names > > > > Signed-off-by: Adrian Hunter > > --- > > arch/arm/mach-omap2/mmc-twl4030.c | 6 +++++- > > arch/arm/mach-omap2/mmc-twl4030.h | 1 + > > 2 files changed, 6 insertions(+), 1 deletions(-) > > > > diff --git a/arch/arm/mach-omap2/mmc-twl4030.c b/arch/arm/mach-omap2/mmc-twl4030.c > > index a612a02..88b4bed 100644 > > --- a/arch/arm/mach-omap2/mmc-twl4030.c > > +++ b/arch/arm/mach-omap2/mmc-twl4030.c > > @@ -328,7 +328,11 @@ void __init twl4030_mmc_init(struct twl4030_hsmmc_info *controllers) > > return; > > } > > > > - sprintf(twl->name, "mmc%islot%i", c->mmc, 1); > > + if (c->name) > > + strlcpy(twl->name, c->name, ARRAY_SIZE(twl->name)); > > + else > > + snprintf(twl->name, ARRAY_SIZE(twl->name), > > + "mmc%islot%i", c->mmc, 1); > > Good, except now we have a confusion over which patch does what... Here's this one updated to apply with snprintf moved to the earlier patch in the series. Tony