RE: [PATCH] OMAP3: MMC: Add mux for pins

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

 




>-----Original Message-----
>From: Kevin Hilman [mailto:khilman@xxxxxxxxxxxxxxxxxxx]
>
>>>If some pins are always needed, and don't have alternative pinouts, then
>>>the common pins could be muxed in devices.c.
>>
>> This is the algo we can use for MMC pin muxing in that case:
>>
>> MMC1: No pin has mux clash
>> 	Mux all 10 pins in devices.c
>
>Is this common across 34xx and 35xx?

Yes this is common.
Both are same OMAP3 marketed differently.

>
>> MMC2: MUX CLK,CMD,D0-D3 in devices.c: D4-D7 have mux clash
>> 	In case board needs 8 bit support,
>> 	then in devices.c print KERN_WARNING "Configure MMC2:D4-D7 mux in board file"
>
>I don't think you need a KERN_WARNING, what if the board code does this
>later?  Probably a comment in the code would suffice.

Given this split of common MMC mux in devices.c
and non-common MMC mux in board file, 
for someone starting new, a warning message helps a lot...

The idea is to warn only MMC2-8bit and MMC3 user to setup the mux.

If already done in uboot/board file, purpose is solved... 

>
>> MMC3: All pins have mux clash: No mux done in devices.c
>> 	In case board specifies MMC3 usage,
>> 	then in devices.c print KERN_WARNING "Configure MMC3 mux in board file"
>>
>> Let me know if this is final and I can submit a patch.
>
>Kevin
>
>

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

[Index of Archives]     [Linux Arm (vger)]     [ARM Kernel]     [ARM MSM]     [Linux Tegra]     [Linux WPAN Networking]     [Linux Wireless Networking]     [Maemo Users]     [Linux USB Devel]     [Video for Linux]     [Linux Audio Users]     [Yosemite Trails]     [Linux Kernel]     [Linux SCSI]

  Powered by Linux