13:32 <RolfNoot> Hi, I've updated some files in contrib/loaders/flash/psoc6/ and want to run checkpatch

13:32 <RolfNoot> I know there are some issues:

13:32 <RolfNoot> https://build.openocd.org/job/openocd-gerrit-build/18206/TARGET=mingw64/console

13:33 <RolfNoot> problem is that when I run checkpatch, it reports no issues

14:27 <karlp> are you sure you're running the right checkpatch?

16:37 <RolfNoot> The one in /tools/checkpatch.sh

17:26 <PaulFertser> RolfNoot: how do you run it, can you pastebin full transcript?

19:02 <RolfNoot> https://pastebin.com/Rtp8FBqw

19:02 <RolfNoot> Seems it looks at the wrong file

19:04 <PaulFertser> RolfNoot: it checks the last commit by default. BTW, you shouldn't push "fix-up" commits anyway, so you might need to squash the last one with the previous one anyway.

19:08 <PaulFertser> RolfNoot: you can run tools/checkpatch.sh HEAD~3"

19:08 <PaulFertser> to check more than the last commit.

21:26 <RolfNoot> I have 3 commits which address different cases. Currently I commit new changed and use fixup (git rebase -i origin/master) to squash the new commit into the original one.

21:26 <RolfNoot> I hope this is the right way

21:26 <PaulFertser> OK

21:26 <PaulFertser> Different things go to different commits, right

21:27 <PaulFertser> It's just that your last paste had "contrib/loaders/flash/psoc6: remove empty lines at the end of the .inc" commit message and that doesn't seem to be good if it's for a file that you add in previous commit.

21:28 <RolfNoot> I noted that as well. I started off a bit messy, sorry for that. It's more and more clear now

21:29 <PaulFertser> No problem at all :)

21:43 <RolfNoot> Thank you. I'm now using tools/checkpatch.sh HEAD~3 which is great to check the different commits.

22:00 <RolfNoot> Okay, looks like all patches are done & okay now from my side (7983, 7984 & 7986). This means a working implementation for PSoC6 (tested from my side).

22:02 <RolfNoot> I still want to apply the implementation for Infineon KitProg3 debugger (also based on Infineon's OpenOCD fork) and also implement a progress bar for programming (if there's interest)

22:03 <RolfNoot> [100%] [################################] [ Programming ]

22:25 <pedro_a> hi paul , you there

22:26 <pedro_a> i am trying to access an AT91SamS64 cpu but i think my config for openocd is not correct

22:27 <pedro_a> i am using this config

22:27 <pedro_a> interface ftdi

22:27 <pedro_a> ftdi_vid_pid 0x0403 0x8a98 0x0403 0x6010

22:27 <pedro_a> ftdi_layout_init 0x0038 0x087b

22:27 <pedro_a> ftdi_layout_signal nTRST -data 0x0020

22:27 <pedro_a> ftdi_layout_signal nSRST -data 0x0010

22:27 <pedro_a> source [find target/at91sam7sx.cfg]

22:27 <pedro_a> adapter_khz 1000

22:27 <pedro_a> init

22:35 <pedro_a> ok , i think i fixed it

22:35 <pedro_a> adapter driver ftdi

22:35 <pedro_a> ftdi vid_pid 0x0403 0x8a98 0x0403 0x6010

22:35 <pedro_a> ftdi layout_init 0x0038 0x087b

22:35 <pedro_a> transport select jtag

22:35 <pedro_a> source [find target/at91sam7sx.cfg]

22:35 <pedro_a> adapter speed 1000

22:35 <pedro_a> init

22:36 <pedro_a> Info : clock speed 1000 kHz

22:36 <pedro_a> Info : JTAG tap: at91sam7s.cpu tap/device found: 0x3f0f0f0f (mfg: 0x787 (<unknown>), part: 0xf0f0, ver: 0x3)

22:36 <pedro_a> Info : Embedded ICE version 1

22:36 <pedro_a> Info : at91sam7s.cpu: hardware has 2 breakpoint/watchpoint units

22:36 <pedro_a> posting just in case you show up

23:09 <karlp> you haven't really shared anything anyone coudl do anything about yet though.

23:09 <karlp> use a paste bin of some description for configs, and logs, and a description of what actually isn't working...

23:17 <pedro_a> i am using this config with my tiao to access an at91samS64 https://pastebin.com/sayGpCBw

23:18 <pedro_a> this is the output https://pastebin.com/jmQ7yPij

23:19 <pedro_a> after enter in telnet and try to flash image i get Open On-Chip Debugger

23:19 <pedro_a> > flash write_image /opt/remote/neuron.bin 0x0

23:19 <pedro_a> Target not halted

23:19 <pedro_a> auto_probe failed

23:27 <pedro_a> i think i fixed it

23:28 <pedro_a> i had to halt cpu for sometime to be able to flash

23:28 <pedro_a> > flash banks

23:28 <pedro_a> #0 : at91sam7s.flash (at91sam7) at 0x00100000, size 0x00010000, buswidth 0, chipwidth 0

23:28 <pedro_a> > flash write_image /opt/remote/neuron.bin 0x00100000

23:28 <pedro_a> wrote 51792 bytes from file /opt/remote/neuron.bin in 7.022848s (7.202 KiB/s)

23:33 <pedro_a> yes , i checked the hardware , it is working perfectly now

23:41 <karlp> excellent.

23:41 <karlp> so, if you have a tiao, can't you use an interface config from upstream?

23:42 <karlp> like "openocd -f interface/ftdi/tumpa.cfg -c 'transport select jtag' -f target/atm91sam7sx.cfg"

23:42 <karlp> you can probably use quite a bit higher than 1000 for the speed too, should be faster.

23:55 <sdewan> Hello all, I'm wondering if anyone else has been unable to use CMSIS-DAP v2 probes on macOS since fd75e9e542700e40f11d79532d19e311cf437de1 went in? The CMSIS-DAP  USB bulk backend now relies on libusb_dev_mem_alloc(), but this function is not implemented on macOS (in fact as far as I can tell it is only implemented on Linux and will always return

23:55 <sdewan> NULL on all other platforms including Windows).