Re: [PATCH] mmc: sdhci-pxa: Add device tree support

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

 



Hi Haojian,

On Wed, Jun 13 2012, haojian.zhuang@xxxxxxxxx wrote:
> Compatible node could be "mrvl,pxav3-mmc" or "mrvl,sdhci-pxav3". It's 
> depend on your choice. I prefer to use vendor name in compatible
> property.
> And we could avoid naming conflict.

Okay, I'll just stick with mrvl,pxav3-mmc.

> + if (of_find_property(np, "non-removable", NULL))
> + pdata->flags |= PXA_FLAG_CARD_PERMANENT;
> +
> + of_property_read_u32(np, "bus-width", &bus_width);
> + if (bus_width == 8)
> + pdata->flags |= PXA_FLAG_SD_8_BIT_CAPABLE_SLOT;
> +
> + of_property_read_u32(np, "clk-delay-cycles", &clk_delay_cycles);
> + if (clk_delay_cycles > 0)
> + pdata->clk_delay_cycles = clk_delay_cycles;
> +
>
> These properties should be "mrvl,non-removable", "mrvl,bus-width", 
> "mrvl,clk-delay-cycles". Since it's mentioned in 
> Documentations/devicetree/booting-without-of.txt. "It is recommended 
> that if you add any "custom" property whose name may clash with 
> standard defined ones, you prefix them with your vendor name and 
> a comma."

Not quite -- the "non-removable" and "bus-width" properties are part of
the MMC core DT bindings as defined in:

  Documentation/devicetree/bindings/mmc/mmc.txt

They are not custom Marvell properties.  So, these two need to stay as
they are (one day we'll interpret them in the sdhci driver instead of in
sdhci-pxa) but I've changed clk_delay_cycles to mrvl,clk_delay_cycles in
v2 of the patch, which I'm about to send out.  I also applied all of
Zhangfei's changes.

Thanks for the review!

- Chris.
-- 
Chris Ball   <cjb@xxxxxxxxxx>   <http://printf.net/>
One Laptop Per Child
--
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