Re: [PATCH 2/2 v2] ARM: shmobile: Lager:add SPI FLASH support on QSPI

[カオ ミン ヒェップ <cm-hiep@jinso.co.jp>]

Hello Sergei

Thanks for your comment.


>> +void __init r8a7790_add_qspi_device(struct rspi_plat_data *pdata)
>
>    You apparently forgot to change the name prefix to 'lager_'. Also, 
> there's no dire need in this function, you can register the QSPI 
> device right in lager_add_standard_devices().
>
>> +{
>> +    platform_device_register_resndata(
>
>    You're breaking the line to early, I think.
>
>> +        &platform_bus, "qspi", 0,
>> +        qspi_resources, ARRAY_SIZE(qspi_resources),
>> +        pdata, sizeof(*pdata));
>> +}
>> +
>>   static const struct pinctrl_map lager_pinctrl_map[] = {
>>       /* DU (CN10: ARGB0, CN13: LVDS) */
>>       PIN_MAP_MUX_GROUP_DEFAULT("rcar-du-r8a7790", "pfc-r8a7790",
>
I'll update this patch as you pointed out to me!

Best Regards,

Hiep.