On Mon, Oct 24, 2016 at 2:24 PM, Jagan Teki <ja...@openedev.com> wrote:
> Hi Stefano,
>
> On Mon, Oct 24, 2016 at 2:06 PM, Stefano Babic <sba...@denx.de> wrote:
>> On 24/10/2016 09:53, Jagan Teki wrote:
>>> Hi Stefano,
>>>
>>> On Mon, Oct 17, 2016 at 11:26 AM, Stefano Babic <sba...@denx.de> wrote:
>>>> Hi Jagan,
>>>>
>>>> On 17/10/2016 05:40, Jagan Teki wrote:
>>>>> Hi Stefano,
>>>>>
>>>>> On Tue, Oct 11, 2016 at 9:00 AM, Jagan Teki <jt...@openedev.com> wrote:
>>>>>> Hi Stefano,
>>>>>>
>>>>>> On Sun, Oct 9, 2016 at 8:15 PM, Jagan Teki <jagannadh.t...@gmail.com>
>>>>>> wrote:
>>>>>>> Hi Stefano,
>>>>>>>
>>>>>>> On Sat, Oct 8, 2016 at 6:00 PM, Jagan Teki <jt...@openedev.com> wrote:
>>>>>>>> From: Jagan Teki <ja...@amarulasolutions.com>
>>>>>>>>
>>>>>>>> This series supports Engicam i.CoreM6 QDL modules on top of
>>>>>>>> u-boot-imx/master
>>>>>>>> and test on the respective starter kits as well.
>>>>>>>>
>>>>>>>> Tested both MMC and NAND boot.
>>>>>>>>
>>>>>>>> Changes for v7:
>>>>>>>> - Add patch to CONFIG_DEFAULT_FDT_FILE to defconfigs
>>>>>>>> - Rebase on u-boot-imx/master
>>>>>>>>
>>>>>>>> Changes for v6:
>>>>>>>> - Rebase to u-boot-imx/master
>>>>>>>> - Move few SPL configs to defconfigs
>>>>>>>>
>>>>>>>> Changes for v5:
>>>>>>>> - Add NAND support
>>>>>>>>
>>>>>>>> Changes for v4:
>>>>>>>> - Add 'net: Kconfig: Add FEC_MXC entry' patch
>>>>>>>> - Updated ENV configs along with comments
>>>>>>>> - Restructured configs for more readability
>>>>>>>> - Add CONFIG_ENV_OVERWRITE
>>>>>>>> - Rename icorem6qdl_defconfig with icorem6qdl_mmc_defconfig
>>>>>>>>
>>>>>>>> Changes for v3:
>>>>>>>> - Remove "v2 01/17 imx: iomux-v3: Fix build error with snvs
>>>>>>>> base" patch
>>>>>>>> - Remove 'default n' on Kconfig DEFAULT_FDT_FILE patch
>>>>>>>> - Add minimal devicetree support for Engicam i.CoreM6 QDL
>>>>>>>> - Add is_mx6dl()
>>>>>>>> - Add is_mx6solo()
>>>>>>>> - Use is_mx6dq()
>>>>>>>> - Add last commit sha1 and header in pull devicetree files
>>>>>>>> from Linux
>>>>>>>>
>>>>>>>> Changes for v2:
>>>>>>>> - Make static to local iomux structure in board file
>>>>>>>> - Corrected rowaddr in mx6_ddr3_cfg
>>>>>>>> - Used imx_ddr_size
>>>>>>>> - Add FEC support and tested the same
>>>>>>>> - Add DM_GPIO, DM_MMC support
>>>>>>>> - Add pinctrl support
>>>>>>>> - Add devicetree support
>>>>>>>>
>>>>>>>> Jagan Teki (21):
>>>>>>>> serial: Kconfig: Add MXC_UART entry
>>>>>>>> thermal: Kconfig: Add IMX_THERMAL entry
>>>>>>>> config: Move CONFIG_DEFAULT_FDT_FILE to defconfigs
>>>>>>>> arm: imx: Add Engicam i.CoreM6 QDL Starter Kit initial support
>>>>>>>> net: Kconfig: Add FEC_MXC entry
>>>>>>>> imx6: icorem6: Add ENET support
>>>>>>>> imx: s/docs\/README.imximage/doc\/README.imximage/g
>>>>>>>> arm: dts: Add devicetree for i.MX6DL
>>>>>>>> arm: dts: Add devicetree for i.MX6DQL
>>>>>>>> arm: dts: imx6dl: Add pinctrl defines
>>>>>>>> dt-bindings: clock: imx6qdl: Add clock defines
>>>>>>>> arm: imx6q: Add devicetree support for Engicam i.CoreM6 DualLite/Solo
>>>>>>>> imx6q: icorem6: Enable pinctrl driver
>>>>>>>> engicam: icorem6: Add DM_GPIO, DM_MMC support
>>>>>>>> arm: dts: Add devicetree for i.MX6Q
>>>>>>>> arm: dts: imx6q: Add pinctrl defines
>>>>>>>> arm: imx6q: Add devicetree support for Engicam i.CoreM6 Quad/Dual
>>>>>>>> mtd: nand: Kconfig: Add NAND_MXS entry
>>>>>>>> imx6: icorem6: Add NAND support
>>>>>>>> imx6: icorem6: Enable MTD device support
>>>>>>>> imx6: icorem6: Add default mtd nand partition table
>>>>>>>
>>>>>>> All set now, please do apply.
>>>>>>
>>>>>> Let me know if you still find any issues, otherwise apply this.
>>>>>>
>>>>>
>>>>> Any help?
>>>>
>>>> Sorry, I was last week at ELCE - I will go on with your patches (and the
>>>> other ones left pending) this week.
>>>
>>> Please apply this, I need to rebase on top of tom/master and send few
>>> more series.
>>>
>>
>>
>> Tried again, it does not compile clean.
>> drivers/mmc/fsl_esdhc.c:715:21: error: variable 'esdhc_ops' has
>> initializer but incomplete type
>> static const struct mmc_ops esdhc_ops = {
>> ^
>> drivers/mmc/fsl_esdhc.c:716:2: error: unknown field 'send_cmd' specified
>> in initializer
>> .send_cmd = esdhc_send_cmd,
>> ^
>> drivers/mmc/fsl_esdhc.c:716:14: warning: excess elements in struct
>> initializer
>> .send_cmd = esdhc_send_cmd,
>> ^
>> drivers/mmc/fsl_esdhc.c:716:14: note: (near initialization for 'esdhc_ops')
>> drivers/mmc/fsl_esdhc.c:717:2: error: unknown field 'set_ios' specified
>> in initializer
>> .set_ios = esdhc_set_ios,
>> ^
>> drivers/mmc/fsl_esdhc.c:717:13: warning: excess elements in struct
>> initializer
>> .set_ios = esdhc_set_ios,
>> ^
>> drivers/mmc/fsl_esdhc.c:717:13: note: (near initialization for 'esdhc_ops')
>> drivers/mmc/fsl_esdhc.c:718:2: error: unknown field 'init' specified in
>> initializer
>> .init = esdhc_init,
>> ^
>> drivers/mmc/fsl_esdhc.c:718:11: warning: excess elements in struct
>> initializer
>> .init = esdhc_init,
>> ^
>> drivers/mmc/fsl_esdhc.c:718:11: note: (near initialization for 'esdhc_ops')
>> drivers/mmc/fsl_esdhc.c:719:2: error: unknown field 'getcd' specified in
>> initializer
>> .getcd = esdhc_getcd,
>> ^
>> drivers/mmc/fsl_esdhc.c:719:12: warning: excess elements in struct
>> initializer
>> .getcd = esdhc_getcd,
>> ^
>> drivers/mmc/fsl_esdhc.c:719:12: note: (near initialization for 'esdhc_ops')
>>
>> Can you check it ?
>
> Did you take the v7 patches? I re-based on top of u-boot-imx/master
> and it's compiled for me.
> Please see this branch [1]
>
> [1] https://mail.google.com/mail/u/1/#label/u-boot%2Fdenx/157a44636e8dc15f
Sorry wrong link, here is updated one
http://git.denx.de/?p=u-boot-spi.git;a=shortlog;h=refs/heads/engicam-working
thanks!
--
Jagan Teki
Free Software Engineer | www.openedev.com
U-Boot, Linux | Upstream Maintainer
Hyderabad, India.
_______________________________________________
U-Boot mailing list
U-Boot@lists.denx.de
http://lists.denx.de/mailman/listinfo/u-boot
