[patch #9315] Add PRUSS programmer type for xmega pdi

classic Classic list List threaded Threaded
4 messages Options
Reply | Threaded
Open this post in threaded view
|  
Report Content as Inappropriate

[patch #9315] Add PRUSS programmer type for xmega pdi

Joerg Wunsch-6
URL:
  <http://savannah.nongnu.org/patch/?9315>

                 Summary:  Add PRUSS programmer type for xmega pdi
                 Project: AVR Downloader/UploaDEr
            Submitted by: eballetbo
            Submitted on: Tue 25 Apr 2017 07:42:28 AM UTC
                Category: None
                Priority: 5 - Normal
                  Status: None
                 Privacy: Public
             Assigned to: None
        Originator Email:
             Open/Closed: Open
         Discussion Lock: Any

    _______________________________________________________

Details:

This patch adds XMEGA PDI programming support using the Programmable Realtime
Unit (PRUSS) available on AM335x based boards like Beaglebone Black. You can
program an XMEGA in-circuit without requiring any external hardware, with only
a few extra components in your circuit. Note that the programmer needs a
custom firmware that needs to be loaded to the PRU (also attached here).
Avrdude loads the firmware and communicates with it to be able to program the
firmware on XMEGA. E.g.

  avrdude -p atxmega16d4 -c pruss -e -U flash:w:my-firmware.hex




    _______________________________________________________

File Attachments:


-------------------------------------------------------
Date: Tue 25 Apr 2017 07:42:28 AM UTC  Name:
add-support-for-am335x-pruss-programmer.patch  Size: 15kB   By: eballetbo
The .bin files are the firmware required for the programmer
<http://savannah.nongnu.org/patch/download.php?file_id=40494>
-------------------------------------------------------
Date: Tue 25 Apr 2017 07:42:28 AM UTC  Name: uio_pdi-text-1.bin  Size: 4kB  
By: eballetbo
The .bin files are the firmware required for the programmer
<http://savannah.nongnu.org/patch/download.php?file_id=40495>
-------------------------------------------------------
Date: Tue 25 Apr 2017 07:42:28 AM UTC  Name: uio_pdi-data-1.bin  Size: 4kB  
By: eballetbo
The .bin files are the firmware required for the programmer
<http://savannah.nongnu.org/patch/download.php?file_id=40496>

    _______________________________________________________

Reply to this item at:

  <http://savannah.nongnu.org/patch/?9315>

_______________________________________________
  Message sent via/by Savannah
  http://savannah.nongnu.org/


_______________________________________________
avrdude-dev mailing list
[hidden email]
https://lists.nongnu.org/mailman/listinfo/avrdude-dev
Reply | Threaded
Open this post in threaded view
|  
Report Content as Inappropriate

Re: [patch #9315] Add PRUSS programmer type for xmega pdi

Enric Balletbo Serra
Any feedback on this patch?

2017-04-25 9:42 GMT+02:00 Enric Balletbo i Serra <[hidden email]>:

> URL:
>   <http://savannah.nongnu.org/patch/?9315>
>
>                  Summary:  Add PRUSS programmer type for xmega pdi
>                  Project: AVR Downloader/UploaDEr
>             Submitted by: eballetbo
>             Submitted on: Tue 25 Apr 2017 07:42:28 AM UTC
>                 Category: None
>                 Priority: 5 - Normal
>                   Status: None
>                  Privacy: Public
>              Assigned to: None
>         Originator Email:
>              Open/Closed: Open
>          Discussion Lock: Any
>
>     _______________________________________________________
>
> Details:
>
> This patch adds XMEGA PDI programming support using the Programmable Realtime
> Unit (PRUSS) available on AM335x based boards like Beaglebone Black. You can
> program an XMEGA in-circuit without requiring any external hardware, with only
> a few extra components in your circuit. Note that the programmer needs a
> custom firmware that needs to be loaded to the PRU (also attached here).
> Avrdude loads the firmware and communicates with it to be able to program the
> firmware on XMEGA. E.g.
>
>   avrdude -p atxmega16d4 -c pruss -e -U flash:w:my-firmware.hex
>
>
>
>
>     _______________________________________________________
>
> File Attachments:
>
>
> -------------------------------------------------------
> Date: Tue 25 Apr 2017 07:42:28 AM UTC  Name:
> add-support-for-am335x-pruss-programmer.patch  Size: 15kB   By: eballetbo
> The .bin files are the firmware required for the programmer
> <http://savannah.nongnu.org/patch/download.php?file_id=40494>
> -------------------------------------------------------
> Date: Tue 25 Apr 2017 07:42:28 AM UTC  Name: uio_pdi-text-1.bin  Size: 4kB
> By: eballetbo
> The .bin files are the firmware required for the programmer
> <http://savannah.nongnu.org/patch/download.php?file_id=40495>
> -------------------------------------------------------
> Date: Tue 25 Apr 2017 07:42:28 AM UTC  Name: uio_pdi-data-1.bin  Size: 4kB
> By: eballetbo
> The .bin files are the firmware required for the programmer
> <http://savannah.nongnu.org/patch/download.php?file_id=40496>
>
>     _______________________________________________________
>
> Reply to this item at:
>
>   <http://savannah.nongnu.org/patch/?9315>
>
> _______________________________________________
>   Message sent via/by Savannah
>   http://savannah.nongnu.org/
>

_______________________________________________
avrdude-dev mailing list
[hidden email]
https://lists.nongnu.org/mailman/listinfo/avrdude-dev
Reply | Threaded
Open this post in threaded view
|  
Report Content as Inappropriate

[patch #9315] Add PRUSS programmer type for xmega pdi

Joerg Wunsch-6
In reply to this post by Joerg Wunsch-6
Follow-up Comment #1, patch #9315 (project avrdude):

Basically, a good idea.

Documentation patches are missing (either to the .texi or the
man page file, preferrably for both).

I see a couple of problems with the firmware files though:

First, we are guests on the FSF's servers, and they only accept
opensource projects.  Thus, we are not allowed to host a
closed-source firmware for the programmer.

Second, hardcoding a path name in the application is completely
a "don't ever do this".  This should preferrably be handled
using autoconf parameters, depending from --prefix.  However,
on Windows, this is usually completely different.  See
winconf.c for how this is implemented wrt. avrdude.conf.

    _______________________________________________________

Reply to this item at:

  <http://savannah.nongnu.org/patch/?9315>

_______________________________________________
  Message sent via/by Savannah
  http://savannah.nongnu.org/


_______________________________________________
avrdude-dev mailing list
[hidden email]
https://lists.nongnu.org/mailman/listinfo/avrdude-dev
Reply | Threaded
Open this post in threaded view
|  
Report Content as Inappropriate

[patch #9315] Add PRUSS programmer type for xmega pdi

Joerg Wunsch-6
Follow-up Comment #2, patch #9315 (project avrdude):

Joerg, thanks for you answer.

Some comments below.

About the documentation missing I'll prepare a second version including the
.texi and man page files. About hardcoding the path name I have another
version of this patch that uses and extra parameter to specify the location of
the firmware files, but seeing your answer might be not enough, I'll try to
investigate how to handle this part with autoconf parameter and see how it
works on Windows

Finally, the firmware is open [1] so that shouldn't be a problem, I guess

[1] https://github.com/eballetbo/pdi-pruss


    _______________________________________________________

Reply to this item at:

  <http://savannah.nongnu.org/patch/?9315>

_______________________________________________
  Message sent via/by Savannah
  http://savannah.nongnu.org/


_______________________________________________
avrdude-dev mailing list
[hidden email]
https://lists.nongnu.org/mailman/listinfo/avrdude-dev
Loading...