linux-kernel - RE: [PATCH 1/1] DaVinci: MMC: MMC/SD controller driver for DaVinci/DM6446.

lists.openwall.net		lists / announce owl-users owl-dev john-users john-dev passwdqc-users yescrypt popa3d-users / oss-security kernel-hardening musl sabotage tlsify passwords / crypt-dev xvendor / Bugtraq Full-Disclosure linux-kernel linux-netdev linux-ext4 linux-hardening linux-cve-announce PHC
Open Source and information security mailing list archives

Hash Suite: Windows password security audit tool. GUI, reports in PDF.

[<prev] [next>] [<thread-prev] [thread-next>] [day] [month] [year] [list]

Message-ID: <B85A65D85D7EB246BE421B3FB0FBB59301CC30E0D1@dbde02.ent.ti.com>
Date:	Tue, 17 Mar 2009 12:55:14 +0530
From:	"Kumar, Purushotam" <purushotam@...com>
To:	David Brownell <david-b@...bell.net>
CC:	"davinci-linux-open-source@...ux.davincidsp.com" 
	<davinci-linux-open-source@...ux.davincidsp.com>,
	"linux-kernel@...r.kernel.org" <linux-kernel@...r.kernel.org>,
	"drzeus-mmc@...eus.cx" <drzeus-mmc@...eus.cx>
Subject: RE: [PATCH 1/1] DaVinci: MMC: MMC/SD controller driver for
 DaVinci/DM6446.

 
> And there are other DaVinci chips that support MMC too ...
> some are compatible with those two chips, some not, all
> should use a version of this driver.  (And OMAP-L137 too,
> same tech family but with a different marketing label.)
> 
> So I'd remove the implication that DaVinci == DM6446, or
> that this driver is specific to that SoC.

I will update the description as suggested by you when I will re-submit.


> > +config MMC_DAVINCI
> > +        tristate "TI DAVINCI Multimedia Card Interface support"
> > +        depends on MMC
> 
> Everything in that menu depends on MMC.  This should
> depend on ARCH_DAVINCI instead.  :)

I will add "depends on ARCH_DAVINCI"


> > +       host->clk = clk_get(&pdev->dev, NULL);
> 
> The NULL won't work with the clock code currently
> included in mainline ... "MMCSDCLK" will though.

I will take care of this by changing to "MMCSDCLK" from NULL.



> > +       if (pdata->max_freq)
> > +               mmc->f_max = pdata->max_freq;
> > +       if (pdata->caps)
> > +               mmc->caps |= pdata->caps;
> 
> A build-time issue is that "struct davinci_mmc_config"
> is missing.  You could update this patch to include
> "arch/arm/mach-davinci/include/mach/mmc.h" too.

I will add in the patch as well.
- Purushotam

--
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majordomo@...r.kernel.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html
Please read the FAQ at  http://www.tux.org/lkml/