[PATCH v2 0/6] mx6cuboxi: enable OF_PLATDATA with MMC support

[Walter Lozano]

Hi Peng

On 22/7/20 22:34, Peng Fan wrote:
> Hi Walter,
>
>> Subject: [PATCH v2 0/6] mx6cuboxi: enable OF_PLATDATA with MMC support
>>
>> The SPL in iMX6 boards is restricted to 68 KB as this is the free available space
>> in OCRAM for most revisions. In this context, adding OF_CONTROL and DM
>> increases the SPL size which could make it difficult to add specific features
>> required for custom scenarios.
>>
>> These patches aim to take advantage of OF_PLATADATA in order to reduce the
>> SPL size in this scenario, by parsing DT data to generate platdata structures,
>> and thus removing the overhead caused by DT and related libraries.
>>
>> This series is focused in MMC driver, which is used for boot in boards such as
>> Cubox-i. Also, in order to support CD, the OF_PLATDATA support is also
>> implemented on GPIO driver.
> Do you have some data that how much is decreased using OF_PLATDATA?
>
Yes, I updated the commit message on Patch 6/6, but should have updated 
the cover letter. With OF_PLATDATA enabled the reduction on 
mx6cuboxi_defconfig is 5 KB, which is half of the overhead introduced by 
DM on this config. Additionally, I've in my backlog some additional 
patches for further improvements.

Regards,

Walter

>> Changes in v2:
>> - Improve commit message with footprint reduction
>>
>> Walter Lozano (6):
>>    mmc: fsl_esdhc_imx: rename driver name to match ll_entry
>>    mmc: fsl_esdhc_imx: add OF_PLATDATA support
>>    gpio: mxc_gpio: add OF_PLATDATA support
>>    mmc: fsl_esdhc_imx: add CD support when OF_PLATDATA is enabled
>>    drivers: rename more drivers to match compatible string
>>    mx6cuboxi: enable OF_PLATDATA
>>
>>   configs/mx6cuboxi_defconfig        |  1 +
>>   drivers/gpio/mxc_gpio.c            | 18 +++++-
>>   drivers/mmc/fsl_esdhc_imx.c        | 94
>> +++++++++++++++++++++++++-----
>>   drivers/pinctrl/nxp/pinctrl-imx6.c |  6 +-
>>   drivers/video/imx/mxc_ipuv3_fb.c   |  4 +-
>>   5 files changed, 103 insertions(+), 20 deletions(-)
>>
>> --
>> 2.20.1