On Wed, Feb 23, 2011 at 12:35 PM, Stephen Warren <swarren@xxxxxxxxxx> wrote: > Colin Cross wrote at Wednesday, February 23, 2011 12:09 PM: >> On Wed, Feb 23, 2011 at 10:58 AM, Stephen Warren <swarren@xxxxxxxxxx> wrote: >> > This ensures they're kept in sync between platform_data definitions and >> > the GPIO table initialization. >> >> Can you name these for what they are used for (if they have a single use)? >> i.e. TEGRA_GPIO_EMMC_CD instead of TEGRA_GPIO_SD2_CD > > Are you looking for correlation with the schematics, or something else; > my interpretation is that the names in the patch already appear to describe > what the signals are used for. > > In particular, in your example, you wrote "EMMC" rather than "SD2". I'm not > sure where the name "EMMC" comes from, nor why there's no number in the name > "EMMC". > > For reference, the schematics for these two boards use the following signal > names: > > Harmony: > SDIO2_CD, SDIO2_WP, EN_VDDIO_SD > HSMMC_CD, HSMMC_WP, EN_VDDIO_SDMMC > > Seaboard: > SDIO32_CD, SDIO3_WP, EN_VDDIO_SD > > Would those be OK? I meant use the name of the device they are connected to (if there is one). By EMMC I meant the embedded MMC storage device. The gpio is not notable because it is used by the SDIO driver for port 2, it is notable because it is connected to the EMMC chip or the wifi chip or a card slot. -- To unsubscribe from this list: send the line "unsubscribe linux-tegra" in the body of a message to majordomo@xxxxxxxxxxxxxxx More majordomo info at http://vger.kernel.org/majordomo-info.html