public inbox for u-boot@lists.denx.de
 help / color / mirror / Atom feed
From: Frank Mori Hess <fmh6jj@gmail.com>
To: u-boot@lists.denx.de
Subject: [U-Boot] [PATCH] ARM SOCFPGA: add resetmgr command so reset can be deasserted in bootcmd (for example on peripheral dma interfaces after fpga has been programmed).
Date: Sat, 16 Dec 2017 12:03:57 -0500	[thread overview]
Message-ID: <2294422.QzpSMFZzzW@bear> (raw)
In-Reply-To: <4279952.rNqcPfkFEa@bear>

Marek Vasut wrote:
> Please always CC the list. Do NOT top-post.

You do realize I was replying to an email you sent to my personal address and 
you didn't even send to the list?
 
> What is your goal here ?

To put things in context, my larger goal is to update u-boot from the old 
version altera integrates into Quartus to a reasonably recent version of 
mainline u-boot being provided by the distro we are using.  My naive hope was 
the new version of u-boot would work at least as well as the old altera one.  
Experience so far:  infinite reboot loop due to broken cadence driver:

https://lists.denx.de/pipermail/u-boot/2017-December/313470.html

No response except from author responsible for breaking the driver insisting 
his changes be kept.

And now: DMA peripheral requests for FPGA are non-functional due to mainline 
u-boot ignoring the reset_config.h in the handoff files generated by Quartus.  
Apparently, the mainline uboot position is that it is inappropriate to provide 
any more support for initializing the resets than providing the ability to 
write to memory addresses with "mw".
 
> 
> Going back to my initial question -- what is your usecase and your aim
> here ? Usually you use FPGA manager in Linux to load the FPGA.
> >> 
> >> But you can really just do mw to the correct address or create a U-Boot
> >> script , so this command is not really needed, is it ?
> 

Ok, I am running Linux on the board.  I don't see how it would help to load 
the FPGA from Linux rather than u-boot.  The dma peripheral requests would 
still be just as disabled.  The only difference would be that I would be 
forced to deassert the resets from Linux rather than u-boot, which I guess 
would make it not your problem?  In principle, the fpga manager provides a 
write_complete hook that the socfpga fpga manager could use to deassert resets 
(perhaps based on device tree settings), but looking at my 4.1.33 fpga/
socfpga.c it doesn't seem to.  

  parent reply	other threads:[~2017-12-16 17:03 UTC|newest]

Thread overview: 6+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2017-12-14 21:49 [U-Boot] [PATCH] ARM SOCFPGA: add resetmgr command so reset can be deasserted in bootcmd (for example on peripheral dma interfaces after fpga has been programmed) Frank Mori Hess
2017-12-15 19:17 ` Dinh Nguyen
2017-12-16 17:03 ` Frank Mori Hess [this message]
2017-12-16 18:00   ` Marek Vasut
  -- strict thread matches above, loose matches on Subject: below --
2017-12-14 21:03 Frank Mori Hess
     [not found] ` <848dcc81-5b50-1b03-3c6f-c8ce6940a50e@denx.de>
     [not found]   ` <CAJz5OpcM_nHQqecnP1xPkHZS=40rpPPwbGfvj8rq4rCsH24PYg@mail.gmail.com>
2017-12-15 17:21     ` Marek Vasut

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=2294422.QzpSMFZzzW@bear \
    --to=fmh6jj@gmail.com \
    --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