[U-Boot] [PATCH 1/3] mx7: Do not call lcdif_power_down() in the SPL case

From: Fabio Estevam festevam@gmail.com
Like it was done on imx6 in commit 9236269de57d ("imx: mx6: Fix implementantion reset_misc")
Do not call lcdif_power_down() in the SPL case to fix the following build error:
LD spl/u-boot-spl MKIMAGE u-boot.img arch/arm/mach-imx/built-in.o: In function `reset_misc': /home/fabio/ossystems/u-boot/arch/arm/mach-imx/mx7/soc.c:372: undefined reference to `lcdif_power_down' scripts/Makefile.spl:375: recipe for target 'spl/u-boot-spl' failed
Signed-off-by: Fabio Estevam festevam@gmail.com Signed-off-by: Otavio Salvador otavio@ossystems.com.br ---
arch/arm/mach-imx/mx7/soc.c | 2 ++ 1 file changed, 2 insertions(+)
diff --git a/arch/arm/mach-imx/mx7/soc.c b/arch/arm/mach-imx/mx7/soc.c index 3f74f8a3ed..7cfdff0981 100644 --- a/arch/arm/mach-imx/mx7/soc.c +++ b/arch/arm/mach-imx/mx7/soc.c @@ -368,8 +368,10 @@ void s_init(void)
void reset_misc(void) { +#ifndef CONFIG_SPL_BUILD #ifdef CONFIG_VIDEO_MXS lcdif_power_down(); #endif +#endif }

From: Fabio Estevam festevam@gmail.com
Add support for the VXT VL050-8048NT-C01 panel connected through the 24 bit parallel LCDIF interface.
Signed-off-by: Fabio Estevam festevam@gmail.com Signed-off-by: Otavio Salvador otavio@ossystems.com.br ---
board/technexion/pico-imx7d/pico-imx7d.c | 55 ++++++++++++++++++++++++ configs/pico-hobbit-imx7d_defconfig | 1 + configs/pico-imx7d_defconfig | 1 + configs/pico-pi-imx7d_defconfig | 1 + include/configs/pico-imx7d.h | 12 ++++++ 5 files changed, 70 insertions(+)
diff --git a/board/technexion/pico-imx7d/pico-imx7d.c b/board/technexion/pico-imx7d/pico-imx7d.c index 53e14693a5..767d13dfe5 100644 --- a/board/technexion/pico-imx7d/pico-imx7d.c +++ b/board/technexion/pico-imx7d/pico-imx7d.c @@ -39,8 +39,16 @@ DECLARE_GLOBAL_DATA_PTR; #define I2C_PAD_CTRL (PAD_CTL_DSE_3P3V_32OHM | PAD_CTL_SRE_SLOW | \ PAD_CTL_HYS | PAD_CTL_PUE | PAD_CTL_PUS_PU100KOHM)
+ +#define LCD_PAD_CTRL (PAD_CTL_HYS | PAD_CTL_PUS_PU100KOHM | \ + PAD_CTL_DSE_3P3V_49OHM) + +#define LCD_SYNC_PAD_CTRL (PAD_CTL_HYS | PAD_CTL_PUS_PU100KOHM | \ + PAD_CTL_DSE_3P3V_196OHM) + #ifdef CONFIG_SYS_I2C_MXC #define PC MUX_PAD_CTRL(I2C_PAD_CTRL) + /* I2C4 for PMIC */ static struct i2c_pads_info i2c_pad_info4 = { .scl = { @@ -246,11 +254,58 @@ int board_early_init_f(void) return 0; }
+#ifdef CONFIG_VIDEO_MXS +static iomux_v3_cfg_t const lcd_pads[] = { + MX7D_PAD_LCD_CLK__LCD_CLK | MUX_PAD_CTRL(LCD_PAD_CTRL), + MX7D_PAD_LCD_ENABLE__LCD_ENABLE | MUX_PAD_CTRL(LCD_SYNC_PAD_CTRL), + MX7D_PAD_LCD_HSYNC__LCD_HSYNC | MUX_PAD_CTRL(LCD_SYNC_PAD_CTRL), + MX7D_PAD_LCD_VSYNC__LCD_VSYNC | MUX_PAD_CTRL(LCD_SYNC_PAD_CTRL), + MX7D_PAD_LCD_DATA00__LCD_DATA0 | MUX_PAD_CTRL(LCD_PAD_CTRL), + MX7D_PAD_LCD_DATA01__LCD_DATA1 | MUX_PAD_CTRL(LCD_PAD_CTRL), + MX7D_PAD_LCD_DATA02__LCD_DATA2 | MUX_PAD_CTRL(LCD_PAD_CTRL), + MX7D_PAD_LCD_DATA03__LCD_DATA3 | MUX_PAD_CTRL(LCD_PAD_CTRL), + MX7D_PAD_LCD_DATA04__LCD_DATA4 | MUX_PAD_CTRL(LCD_PAD_CTRL), + MX7D_PAD_LCD_DATA05__LCD_DATA5 | MUX_PAD_CTRL(LCD_PAD_CTRL), + MX7D_PAD_LCD_DATA06__LCD_DATA6 | MUX_PAD_CTRL(LCD_PAD_CTRL), + MX7D_PAD_LCD_DATA07__LCD_DATA7 | MUX_PAD_CTRL(LCD_PAD_CTRL), + MX7D_PAD_LCD_DATA08__LCD_DATA8 | MUX_PAD_CTRL(LCD_PAD_CTRL), + MX7D_PAD_LCD_DATA09__LCD_DATA9 | MUX_PAD_CTRL(LCD_PAD_CTRL), + MX7D_PAD_LCD_DATA10__LCD_DATA10 | MUX_PAD_CTRL(LCD_PAD_CTRL), + MX7D_PAD_LCD_DATA11__LCD_DATA11 | MUX_PAD_CTRL(LCD_PAD_CTRL), + MX7D_PAD_LCD_DATA12__LCD_DATA12 | MUX_PAD_CTRL(LCD_PAD_CTRL), + MX7D_PAD_LCD_DATA13__LCD_DATA13 | MUX_PAD_CTRL(LCD_PAD_CTRL), + MX7D_PAD_LCD_DATA14__LCD_DATA14 | MUX_PAD_CTRL(LCD_PAD_CTRL), + MX7D_PAD_LCD_DATA15__LCD_DATA15 | MUX_PAD_CTRL(LCD_PAD_CTRL), + MX7D_PAD_LCD_DATA16__LCD_DATA16 | MUX_PAD_CTRL(LCD_PAD_CTRL), + MX7D_PAD_LCD_DATA17__LCD_DATA17 | MUX_PAD_CTRL(LCD_PAD_CTRL), + MX7D_PAD_LCD_DATA18__LCD_DATA18 | MUX_PAD_CTRL(LCD_PAD_CTRL), + MX7D_PAD_LCD_DATA19__LCD_DATA19 | MUX_PAD_CTRL(LCD_PAD_CTRL), + MX7D_PAD_LCD_DATA20__LCD_DATA20 | MUX_PAD_CTRL(LCD_PAD_CTRL), + MX7D_PAD_LCD_DATA21__LCD_DATA21 | MUX_PAD_CTRL(LCD_PAD_CTRL), + MX7D_PAD_LCD_DATA22__LCD_DATA22 | MUX_PAD_CTRL(LCD_PAD_CTRL), + MX7D_PAD_LCD_DATA23__LCD_DATA23 | MUX_PAD_CTRL(LCD_PAD_CTRL), + MX7D_PAD_GPIO1_IO06__GPIO1_IO6 | MUX_PAD_CTRL(LCD_PAD_CTRL), + MX7D_PAD_GPIO1_IO11__GPIO1_IO11 | MUX_PAD_CTRL(NO_PAD_CTRL), +}; + +void setup_lcd(void) +{ + imx_iomux_v3_setup_multiple_pads(lcd_pads, ARRAY_SIZE(lcd_pads)); + /* Set Brightness to high */ + gpio_direction_output(IMX_GPIO_NR(1, 11) , 1); + /* Set LCD enable to high */ + gpio_direction_output(IMX_GPIO_NR(1, 6) , 1); +} +#endif + int board_init(void) { /* address of boot parameters */ gd->bd->bi_boot_params = PHYS_SDRAM + 0x100;
+#ifdef CONFIG_VIDEO_MXS + setup_lcd(); +#endif #ifdef CONFIG_FEC_MXC setup_fec(); #endif diff --git a/configs/pico-hobbit-imx7d_defconfig b/configs/pico-hobbit-imx7d_defconfig index b02cae5237..0095050d43 100644 --- a/configs/pico-hobbit-imx7d_defconfig +++ b/configs/pico-hobbit-imx7d_defconfig @@ -57,4 +57,5 @@ CONFIG_USB_GADGET_MANUFACTURER="FSL" CONFIG_USB_GADGET_VENDOR_NUM=0x0525 CONFIG_USB_GADGET_PRODUCT_NUM=0xa4a5 CONFIG_CI_UDC=y +CONFIG_VIDEO=y CONFIG_OF_LIBFDT=y diff --git a/configs/pico-imx7d_defconfig b/configs/pico-imx7d_defconfig index f355f07be7..3c715f743b 100644 --- a/configs/pico-imx7d_defconfig +++ b/configs/pico-imx7d_defconfig @@ -57,4 +57,5 @@ CONFIG_USB_GADGET_MANUFACTURER="FSL" CONFIG_USB_GADGET_VENDOR_NUM=0x0525 CONFIG_USB_GADGET_PRODUCT_NUM=0xa4a5 CONFIG_CI_UDC=y +CONFIG_VIDEO=y CONFIG_OF_LIBFDT=y diff --git a/configs/pico-pi-imx7d_defconfig b/configs/pico-pi-imx7d_defconfig index 3a182dcb81..0097c816a4 100644 --- a/configs/pico-pi-imx7d_defconfig +++ b/configs/pico-pi-imx7d_defconfig @@ -57,4 +57,5 @@ CONFIG_USB_GADGET_MANUFACTURER="FSL" CONFIG_USB_GADGET_VENDOR_NUM=0x0525 CONFIG_USB_GADGET_PRODUCT_NUM=0xa4a5 CONFIG_CI_UDC=y +CONFIG_VIDEO=y CONFIG_OF_LIBFDT=y diff --git a/include/configs/pico-imx7d.h b/include/configs/pico-imx7d.h index 333c0b49c1..97503a9efb 100644 --- a/include/configs/pico-imx7d.h +++ b/include/configs/pico-imx7d.h @@ -64,10 +64,12 @@ #define CONFIG_EXTRA_ENV_SETTINGS \ "script=boot.scr\0" \ "image=zImage\0" \ + "splashpos=m,m\0" \ "console=ttymxc4\0" \ "fdt_high=0xffffffff\0" \ "initrd_high=0xffffffff\0" \ "fdtfile=" CONFIG_DEFAULT_FDT_FILE "\0" \ + "videomode=video=ctfb:x:800,y:480,depth:24,mode:0,pclk:30000,le:46,ri:210,up:22,lo:23,hs:20,vs:10,sync:0,vmode:0\0" \ BOOTMENU_ENV \ "fdt_addr=0x83000000\0" \ "fdt_addr_r=0x83000000\0" \ @@ -131,6 +133,16 @@ #define CONFIG_POWER_PFUZE3000 #define CONFIG_POWER_PFUZE3000_I2C_ADDR 0x08
+#ifdef CONFIG_VIDEO +#define CONFIG_VIDEO_MXS +#define CONFIG_VIDEO_LOGO +#define CONFIG_SPLASH_SCREEN +#define CONFIG_SPLASH_SCREEN_ALIGN +#define CONFIG_BMP_16BPP +#define CONFIG_VIDEO_BMP_RLE8 +#define CONFIG_VIDEO_BMP_LOGO +#endif + /* FLASH and environment organization */ #define CONFIG_ENV_SIZE SZ_8K

From: Fabio Estevam festevam@gmail.com Add support for the VXT VL050-8048NT-C01 panel connected through the 24 bit parallel LCDIF interface. Signed-off-by: Fabio Estevam festevam@gmail.com Signed-off-by: Otavio Salvador otavio@ossystems.com.br
Applied to u-boot-imx, master, thanks !
Best regards, Stefano Babic

From: Fabio Estevam festevam@gmail.com
Add the TechNexion's logo from their internal U-Boot tree.
Signed-off-by: Fabio Estevam festevam@gmail.com Signed-off-by: Otavio Salvador otavio@ossystems.com.br ---
tools/logos/technexion.bmp | Bin 0 -> 22390 bytes 1 file changed, 0 insertions(+), 0 deletions(-) create mode 100644 tools/logos/technexion.bmp
diff --git a/tools/logos/technexion.bmp b/tools/logos/technexion.bmp new file mode 100644 index 0000000000000000000000000000000000000000..bccde2de5922cb5d015ce302b74937b2812404ee GIT binary patch literal 22390 zcmeHOO=x4+6+TbDw(PMaTejrbV_UL4mL*HFq)Rt}HZ+U0gsf)KMV8rSIdqdnD9kd; zKwGj%7lG1d5lSFX8c5p|l0ufDr4Z6AvI~Sl7D+du_nx2cz9-A6-6;))_h9tiyXV|< z&OP`1-sk!LGe7<j@%5b&QQtrH6QUpZdqOKb{=|KvcQ@pp9}ED#sw*qBwzfuFTU)fZ zw?`)@Cv<k!r{Qo&lgTB`=W}}e@yF?V-+h9feDX<p>Zu>n(@#H5_wV1QXP<qRo_p>& zdj9$6>A`~s^uh}-(2FnrgdRS8NH4wg621KL%k<Nq{hVHT<rRAM)nCwSuf0aU{ME1N z_19mgH{SRSz4_*w^wwLyrQf~%Hof!C@9CXC{E^;$_fPcR`+uhQ-+P}v_}~Nj@WT)3 zFMs_Tee}^s^zp}^(BD7#gg*WBQ~K<)&*<~d|3P1T@dbVP<yZ9OSO281zy4YlmH*}% z$Tg5_AlE>yfm{Q*267GL8pt(}YarJ^u7O+wxdw6#<Qm8|kZa)E(?GIPZ#EhmRXv4W z6r{A)Xf_+Q!s7UU>%CZOG#d4qKP!IJs}LxyKLVlh-f@qwlg=WQq;YTV^t#wxT@<x; zFg56SuLfxIROp@F;i2Nb&_6z?g!f`w*>$%tgJ@Id9Jf<4lsaRhKRw+n3i`GX?{&{~ zfaq>l$BtV>rMvoaH?_4%^IR!i9@SIhb(Q6`&cq8HeLuXiRuAu)Fm3@_ItGLvF8ncn zyh?{z=s?s$RJHM*AcG1|Pqqvkz9#5Z>e}F?NF2tSmo~`v`M#*|%toiW;3VR$6BTw9 zK8ey+O-l@##tkYOEtO@@-ZxFM)O}}W&}K^2e&?eC4WSnuY*9@f<v?~g$}LQ`%<9@a zd(DWAqY180Y|vp5O`8VLWs?bF(PN>%8WDs14a0v-z(kd(NP$h(3^X)kS$V@S_hGDt zNwf;<?q#4&qmjCgu(r+As6C#uK_(r07uzVA5-!*vDt!gdb|6EFl`5OcFhir}@2u$p z7I(%3HnGp$u-cj}=TylTPEr))ac-<z25K;J(e_}x;93kdpmV9<Kn3Gxr1(BoYcAX3 zvrtxJ&BhKzQqUJQwheqFVDBr?taB6H37Cap^L;OvWEqk%gMDD|VvBe6edN6`Tz3FZ zJ?smtY(uJ4_<wYNK@7hg1nV>uVaB=9DI{|f;l5!EviEU_)g`muOX0C?INNO9(7?HT z;q;CHO?ph(*#flbru{xF#_b0ZtJtKDe>3~EZEhJmHJW9H>D(^MmSABb>ZbW{Cq@{7 z_}`iV-I2H{L!vJP?X5tXan-*jaT#uM*aZRC_pV!bOeLL4XfN9`hs=D{v@)}32?GrC z_SpGLr?%SD@E*OK*SBFiG2f&FZWws`Ys0j7z^h4&khj%?<0vTy!eqyw>UcJHoOUfH z>wgUBTuM*SgqrAqr#S8(wzq<w3P^Ty7Q6xeVH4}he7+|fuxaJ+o*=8po>CIE;3rNP zsbZ@ii~W{7gUdm0Z(DK=g8Xcw3hG-TeaZ6AIlFMlG~v3Qc@fMVZ{n&!1JGYE2Lxgn z=ilhzmfQWX8HF=vtjc!ri~<~aBy<W;lviWbwtFBzs9cg6DuXkwsHg~HEHo<ys~@NX z6w0YRB-%C`9X)2uoGlH}j_q~DG~t92(tAIcOv$u75m6!APKKQT*Ffq(2Es2EXL^bt zZ%^13n_;EM^vG{ou)T1NB>`9D%{kHA`h~vF#Mn!Sk38N95>^I^V|q$+56`>VNgjK^ zhUh^kyo8bxa#}{vkqvBmMo1x3Nl=hwyd*%8&2%n`)3iZf<X*XAwTLREKHP8Bt3urz zY2KEZaBMco1opGd;C169_i;cfSu(${)i26mdx6Oay(M#=K8FZHP|K&pOn(*_Z5UX? zi}|}-#y9fRT<QtHb1(Y~*TCB|*4AB=KfJG-FB}Ur>j{L{UNHSwZ>h*34Wg``4;Q%i z`?`X~JSN5&7$SHESAbez1MrmKm(jt)Orn<QZJBvs)$Xof``?01xO(^QQ$c*dgwzB2 zkfGFT(7E&nb+fNo@XITO8s~E}$*bIFwgNL7K}nVO=m1*5g`rQg5DO9Pi3scyiOmT6 zW`3z<QFOd1h7)AwtD+WT?44K<go)w7N+w-XY-QLl3s_<3L?D^*VF@>EBWyI@&y-bS zF^oZ0$a`RUq{t`1UJ*dh5`le?-N5rui}}muW{PeBQH*VKOB#_t6cA6|#BIN48iI_H z+)*c}*CTmKiv~YG^dZ71fjUdLv~LlVVTdc?%4nE_wv5N@9?Th7l12OoI0o6#TOJ4F zBSJ+5weqn1SjFQU;xeJ8j|;FT@0}=4#g$UD!ehm{`k@3oIQA(cLX9(D8C<nnb}Yzi zOmSBaNrg}bY55cw>{v1k+kzTq8p3+Gu(F|lK|?>0#J-T6c>;c7FtY5K3Z=7Fslpdp z8SfeJ>#0@MAQm1_H%Rk<J(a_d*e>{NjP>Ioh);LS0ZwaWVQ<bnwE%^DATwhi#|c3< zm>wED?Ke{_10y5rq$-Ya-^?~3=@nD;W$cvEPMFEW<@sq1TNVd_Kxe(ZVWE-`O63fr zc%j%@G9k9^ConaK!bU;^s$cQdSmv9}$Ec^or34D1n2wnedLZphi4glj{Hj}Wp0YtD zh+fMK0IRl=#}kM(n9hhKd^j#3$Y=nR_G8p-H~dDdAjWbUFIvswWKWys7_$P@;m|+J zA=J7pwx`-haM0Qo=50IXq^$L#!_s)^sFAb-OfVrBMH!CI**O}6#0Za(y5O?TjK>3O zCh0(oEGC$cwvgSORCq-zTc*JPtjTh2x>>H+S%-7%Q&hl?!bRLPuI`*AL-3b#2DFXt zh`|vZ%SP)06>?{f#@wB;%3GMgSqm}3&06J>orn5<QrKjEi|~YtkSz7uGJ<jfhbSZc za-=w@7pa6joQ6PwHp)ooeJsWS30g=kKpK22@SBxc1T6s739`{KXAnX>$`EFfK&m zX~$)IwVKKh!S4u%DG)^-5cdO;Ua=8Dlnf9!#ca1$X}}0(GRu<M0)0Mei>IVnsKyAS z<_O)OqOHy!mxL=E638`dR#&ktTIs-M+(E3OCz{iMEcv!>?XD5jJG03gf29!gScq}( z6MJD~g#d$@rRF==(`jgiGSD#>A~1#w%<1tyYT^hjq`GSj4am_E%RgO8VuSs<&DiF+ zp`1aW15n#TZ=9i|>xeNT1C_WA4ar0Kp5gR`lfswMKo<!2vB}{sijK=j!<-dASJZgw zwTpnuy|9-chRqZatrxBNap5}=jwqbAf3SUB#|CUOMM=UgwG?@f6-ti~lz`si$sUK{ z2uyjiaY!YCCEA#=)5k)zmr>?Os}h7hBN|+-VvB(c#mr>Jok^WgwUjBRixp%oZmwei zXE`vQsvAc%v*?9uNF4!ZDxoy0f}c<sIL-40Us?375ywUauf=Uu{!-QwA=ngA$$5+; zofmV0P78r0S_PcH(2DvHOcf7Zznd^g94EWGVJm7%)@kSDhZcAI6ze!w<gM7!_>p@| zckw6kv3kWYl19cJ_d%W!3Y(c2IZ|6r<$ya>MpnB3(FN^~*b1Arx_)F*>X?-UAScs6 zBy#Nu%?*r%uW#ru=El))!)}_Ad`@Jj>HN4vI5l>cQG`gspVPqRmc*}et2gpIrWf5U z@v8JpTG{RmJ#;?lcVZh&$PD2dNws}Gb+bXLw``e8<i5*Nt=U4f*%CI%olbu;5A^p? zau@2jtZiGhkw%bl&EC?>E!^4ddk2_}yY?NPITX{lm3BXfDrbmTXT8&wYX=VQdOJGS zNq>yTl$$=7B4(Wb7H7q@&UoUjIJjDL{>9n<^LMoFwyMm1kjZ_w6b_9|%E%%Kw0z8P zW~s;6chhDNXMJl=vAZz|Gxw!SyrHn@)AjpPc3S!qps&c(o$Ka!BRO8o&biH7Qftb^ zUU-|C%W-3OlTnbqXlu26>c1N`g43ml$VdT=PiQ?_+%`KrSvvP7gH2_h>g;ba?K+(6 zx*OqguI0{kv%hg12PM{ZWw0{^jTIAwhGQYEW~ScgrDg?1s<Xe*#N|lw1Vr_py)CV^ zmQ%ViiuQEr*)Tq-c!qi);VEF-;?NbkHyc2PA6Y`Y*}UtpmUbpkmL5G_H~AaK(Zoq5 zxQGiiIv0&6{!Ldro?M)G!}pdhBGcL5aPAVc>rOy~OZflu|LIk*EtoIeQT(Nn4^NQ= zoHbPkM(Ayx>gIST?r$^u3hs3$Ai5im5}+)UI{g!yEN}d!l1o>W*tYXWsYwDSe9Xvp z_E{lr@pjn1cjcR_4cr@`0=@-BLoE-K&hi~?4w&)1zW(VYPpfuS9Q=8dQM>iMbuJ4# zMwYlWiF`t?fm{Q*267GL8pt(}YarJ^u7O+wxdw6#<Qm8|kZT~<K(2vY1GxtNuQc#4 D8(^5`
literal 0 HcmV?d00001

Hi Stefano,
On Tue, Dec 11, 2018 at 4:41 PM Otavio Salvador otavio@ossystems.com.br wrote:
From: Fabio Estevam festevam@gmail.com
Add the TechNexion's logo from their internal U-Boot tree.
Signed-off-by: Fabio Estevam festevam@gmail.com Signed-off-by: Otavio Salvador otavio@ossystems.com.br
tools/logos/technexion.bmp | Bin 0 -> 22390 bytes 1 file changed, 0 insertions(+), 0 deletions(-) create mode 100644 tools/logos/technexion.bmp
I noticed this patch has not been applied.
Maybe a patchwork bug?
Could you please consider applying it?
Thanks

Hi Stefano,
On Mon, Feb 18, 2019 at 7:53 PM Fabio Estevam festevam@gmail.com wrote:
Hi Stefano,
On Tue, Dec 11, 2018 at 4:41 PM Otavio Salvador otavio@ossystems.com.br wrote:
From: Fabio Estevam festevam@gmail.com
Add the TechNexion's logo from their internal U-Boot tree.
Signed-off-by: Fabio Estevam festevam@gmail.com Signed-off-by: Otavio Salvador otavio@ossystems.com.br
tools/logos/technexion.bmp | Bin 0 -> 22390 bytes 1 file changed, 0 insertions(+), 0 deletions(-) create mode 100644 tools/logos/technexion.bmp
I noticed this patch has not been applied.
Maybe a patchwork bug?
Could you please consider applying it?
Would you like me to resend this one?

-----Original Message----- From: Otavio Salvador [mailto:otavio@ossystems.com.br] Sent: 2018年12月12日 2:41 To: U-Boot Mailing List u-boot@lists.denx.de Cc: Stefano Babic sbabic@denx.de; Fabio Estevam fabio.estevam@nxp.com; Marek Vasut marex@denx.de; Richard Hu richard.hu@technexion.com; John Weber john.weber@technexion.com; Fabio Estevam festevam@gmail.com; Otavio Salvador otavio@ossystems.com.br; Ye Li ye.li@nxp.com; Peng Fan peng.fan@nxp.com; Bryan O'Donoghue bryan.odonoghue@linaro.org; Albert Aribaud albert.u.boot@aribaud.net; Rui Miguel Silva rui.silva@linaro.org; Anson Huang anson.huang@nxp.com; dl-linux-imx linux-imx@nxp.com Subject: [PATCH 1/3] mx7: Do not call lcdif_power_down() in the SPL case
From: Fabio Estevam festevam@gmail.com
Like it was done on imx6 in commit 9236269de57d ("imx: mx6: Fix implementantion reset_misc")
Do not call lcdif_power_down() in the SPL case to fix the following build error:
LD spl/u-boot-spl MKIMAGE u-boot.img arch/arm/mach-imx/built-in.o: In function `reset_misc': /home/fabio/ossystems/u-boot/arch/arm/mach-imx/mx7/soc.c:372: undefined reference to `lcdif_power_down' scripts/Makefile.spl:375: recipe for target 'spl/u-boot-spl' failed
Signed-off-by: Fabio Estevam festevam@gmail.com Signed-off-by: Otavio Salvador otavio@ossystems.com.br
arch/arm/mach-imx/mx7/soc.c | 2 ++ 1 file changed, 2 insertions(+)
diff --git a/arch/arm/mach-imx/mx7/soc.c b/arch/arm/mach-imx/mx7/soc.c index 3f74f8a3ed..7cfdff0981 100644 --- a/arch/arm/mach-imx/mx7/soc.c +++ b/arch/arm/mach-imx/mx7/soc.c @@ -368,8 +368,10 @@ void s_init(void)
void reset_misc(void) { +#ifndef CONFIG_SPL_BUILD #ifdef CONFIG_VIDEO_MXS lcdif_power_down(); #endif +#endif }
Reviewed-by: Peng Fan peng.fan@nxp.com
-- 2.20.0

From: Fabio Estevam festevam@gmail.com Like it was done on imx6 in commit 9236269de57d ("imx: mx6: Fix implementantion reset_misc") Do not call lcdif_power_down() in the SPL case to fix the following build error: LD spl/u-boot-spl MKIMAGE u-boot.img arch/arm/mach-imx/built-in.o: In function `reset_misc': /home/fabio/ossystems/u-boot/arch/arm/mach-imx/mx7/soc.c:372: undefined reference to `lcdif_power_down' scripts/Makefile.spl:375: recipe for target 'spl/u-boot-spl' failed Signed-off-by: Fabio Estevam festevam@gmail.com Signed-off-by: Otavio Salvador otavio@ossystems.com.br Signed-off-by: Fabio Estevam festevam@gmail.com Signed-off-by: Otavio Salvador otavio@ossystems.com.br Reviewed-by: Peng Fan peng.fan@nxp.com
Applied to u-boot-imx, master, thanks !
Best regards, Stefano Babic
participants (4)
-
Fabio Estevam
-
Otavio Salvador
-
Peng Fan
-
sbabic@denx.de