public inbox for u-boot@lists.denx.de
 help / color / mirror / Atom feed
From: Lukasz Majewski <lukma@denx.de>
To: u-boot@lists.denx.de
Subject: [U-Boot] [PATCH 0/5] Enable usage of SDP for i.MX6 Sabre Auto Boards
Date: Mon, 3 Jun 2019 23:15:32 +0200	[thread overview]
Message-ID: <20190603231532.39e8540e@jawa> (raw)
In-Reply-To: <20190603190029.10373-1-sjoerd.simons@collabora.co.uk>

Hi Sjoerd,

> Being able to upload u-boot over USB is rather useful, so ideally this
> functionality should be enabled by the default config for these
> boards. Currently however no USB support is built into the SPL nor is
> SDP enabled for i.MX6 Sabre Auto boards.
> 
> The first two patches in this series adjust the defconfig in a similar
> way as Fabio has done in a recent patchset for Sabre SD. The first one
> drops SPL DM support as there is no way to fit that *and* USB support
> in at the moment, the second one adds missing pinctrl configuration
> for the main u-boot binary.
> 
> The third patch enabled USB & SPL SDP support.
> 
> Patch four adds support for loading FIT images over SDP as that is
> what is in use for the Sabre auto's main u-boot image. This is simply
> the patch Frieder Schrempf submitted to the mainlinglist earlier, but
> made to build on current git master. (Frieder could you be so kind to
> add your SoB?).
> 
> The final patch fixes the u-boot image failing to start due to
> DM_VIDEO.
> 
> Patchset tested by using uuu with the following configuration:
> ```
> uuu_version 1.2
> SDP: boot -f  SPL
> SDPU: write -f u-boot-dtb.img -addr 0x10000000
> SDPU: jump -addr 0x10000000
> SPDU: done
> ```
> 
> I suspect these patches are to late for 2019.07, especially since FIT
> support in SDP would be a new feature? If so I'm happy to split this
> set out of patches that should definatly land for 2019.07 (Making
> sabre auto work) and submit the others for 2019.10

I think that Fabio and Stefano would have a final opinion, but IMHO,
it would be best if you would split those patches.

> 
> 
> Frieder Schrempf (1):
>   usb: gadget: f_sdp: Allow SPL to load and boot FIT via SDP
> 
> Sjoerd Simons (4):
>   mx6sabreauto: Remove CONFIG_SPL_DM to decrease the SPL size
>   mx6sabreauto: Select pinctrl driver
>   mx6sabreauto: Enable SPL SDP support
>   mx6sabreauto: set SYS_MALLOC_F for video
> 
>  common/spl/spl_sdp.c           | 11 +++++++--
>  configs/mx6sabreauto_defconfig |  8 +++++--
>  drivers/usb/gadget/f_sdp.c     | 41
> +++++++++++++++++++++++++++++----- include/sdp.h                  |
> 4 +++- 4 files changed, 53 insertions(+), 11 deletions(-)
> 




Best regards,

Lukasz Majewski

--

DENX Software Engineering GmbH,      Managing Director: Wolfgang Denk
HRB 165235 Munich, Office: Kirchenstr.5, D-82194 Groebenzell, Germany
Phone: (+49)-8142-66989-59 Fax: (+49)-8142-66989-80 Email: lukma at denx.de
-------------- next part --------------
A non-text attachment was scrubbed...
Name: not available
Type: application/pgp-signature
Size: 488 bytes
Desc: OpenPGP digital signature
URL: <http://lists.denx.de/pipermail/u-boot/attachments/20190603/699137ce/attachment.sig>

      parent reply	other threads:[~2019-06-03 21:15 UTC|newest]

Thread overview: 17+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2019-06-03 19:00 [U-Boot] [PATCH 0/5] Enable usage of SDP for i.MX6 Sabre Auto Boards Sjoerd Simons
2019-06-03 19:00 ` [U-Boot] [PATCH 1/5] mx6sabreauto: Remove CONFIG_SPL_DM to decrease the SPL size Sjoerd Simons
2019-06-04  2:48   ` Peng Fan
2019-06-03 19:00 ` [U-Boot] [PATCH 2/5] mx6sabreauto: Select pinctrl driver Sjoerd Simons
2019-06-04  2:49   ` Peng Fan
2019-06-04 14:15   ` Fabio Estevam
2019-06-04 20:09     ` Sjoerd Simons
2019-06-03 19:00 ` [U-Boot] [PATCH 3/5] mx6sabreauto: Enable SPL SDP support Sjoerd Simons
2019-06-03 19:00 ` [U-Boot] [PATCH 4/5] usb: gadget: f_sdp: Allow SPL to load and boot FIT via SDP Sjoerd Simons
2019-06-03 19:17   ` Fabio Estevam
2019-06-03 19:30     ` Sjoerd Simons
2019-06-03 21:18   ` Lukasz Majewski
2019-06-04  7:09   ` Schrempf Frieder
2019-06-04 13:50     ` Fabio Estevam
2019-06-03 19:00 ` [U-Boot] [PATCH 5/5] mx6sabreauto: set SYS_MALLOC_F for video Sjoerd Simons
2019-06-04  2:49   ` Peng Fan
2019-06-03 21:15 ` Lukasz Majewski [this message]

Reply instructions:

You may reply publicly to this message via plain-text email
using any one of the following methods:

* Save the following mbox file, import it into your mail client,
  and reply-to-all from there: mbox

  Avoid top-posting and favor interleaved quoting:
  https://en.wikipedia.org/wiki/Posting_style#Interleaved_style

* Reply using the --to, --cc, and --in-reply-to
  switches of git-send-email(1):

  git send-email \
    --in-reply-to=20190603231532.39e8540e@jawa \
    --to=lukma@denx.de \
    --cc=u-boot@lists.denx.de \
    /path/to/YOUR_REPLY

  https://kernel.org/pub/software/scm/git/docs/git-send-email.html

* If your mail client supports setting the In-Reply-To header
  via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line before the message body.
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox