Re: [PATCH v3 5/5] OMAP: adapt hsmmc to hwmod framework

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 



On 2/24/2011 2:55 PM, Kadiyala, Kishore wrote:
On Thu, Feb 24, 2011 at 5:19 PM, Cousson, Benoit<b-cousson@xxxxxx>  wrote:
On 2/23/2011 6:47 PM, Kadiyala, Kishore wrote:

[...]

  static struct hsmmc_controller {
         char                            name[HSMMC_NAME_LEN + 1];
-} hsmmc[OMAP34XX_NR_MMC];
+} hsmmc[OMAP44XX_NR_MMC];

I do not know the details of that driver, so this comment might be
completely irrelevant, but in theory, such static table should not be
necessary since the driver core already maintain a list of all instances
bound to it.

I agree, but this is used in slot data for the controller and is used
in the driver
to create a /sys entry.

I guess the sysfs should be able to use only the device instance.

I will try to avoid the "OMAP44XX_NR_MMC" dependency.

[...]

+static struct omap_mmc_platform_data *hsmmc_data[OMAP44XX_NR_MMC]
__initdata;

Same concern than for hsmmc, why do you need static table here?

Agree, will remove static declaration.

And why do you need another structure? The omap2_hsmmc_info should already
be in a pdata kind of structure.
The board file should just populate a table of pdata that you will use
during init.

No, omap2_hsmmc_info is intermediate structure used by the boards
files to update
some basic info of the controller, based on which the pdata is
populated in hsmmc.c.

This is the point, I guess you can potentially directly fill partially a pdata with controller information in the board file to avoid that intermediate structure.

[...]

+       name = "mmci-omap-hs";

Could you please rename the device to have something in the form: omap_XXXX?
In that case "omap_mmc" should be good. The "hs" is just a indication of one
of the mmc instance capability and does not have to be in the device name
since there is no none-hs instance.

I understood your concern but omap1,omap2420 uses mmc driver while
omap2430, omap3 , omap4 has hsmmc driver.

OK, it makes sense then.

omap1, omap2420 boards have device name as "mmci-omap" currently, but
if they undergo
the similar change as proposed above then it looks like "omap_mmc"

Therefore for hsmmc driver, I will be happy to have something like "omap_hsmmc"

please let me know if this is fine.

Excellent, that's fine for me.

Thanks,
Benoit
--
To unsubscribe from this list: send the line "unsubscribe linux-mmc" in
the body of a message to majordomo@xxxxxxxxxxxxxxx
More majordomo info at  http://vger.kernel.org/majordomo-info.html


[Index of Archives]     [Linux USB Devel]     [Linux Media]     [Video for Linux]     [Linux Audio Users]     [Yosemite News]     [Linux Kernel]     [Linux SCSI]

  Powered by Linux