Skip to content
This repository was archived by the owner on Mar 31, 2023. It is now read-only.

Conversation

@xpix
Copy link

@xpix xpix commented Jul 20, 2020

Hello,

here an try to support my board with Arduino in the version 1.8.12 compiled also through, unfortunately, at the end of a link error where I need your help:

#error clnk C:\Users\-----\AppData\Local\Arduino15\packages\STM8\hardware\stm8\0.1.0\variants\XMOTO_8L151G4\lkfscript.lkf:1 symbol _NonHandledInterrupt not defined (C:\Users\----\AppData\Local\Temp\arduino_build_474709\core\stm8l15x_it.c.o )

There is one error message for each entry in stm8l15x_it.c. Can you please help me with this, I'm doing this for the first time.

Thanks a lot
Frank

@fpistm fpistm added this to the 0.2.0 milestone Aug 4, 2020
@fpistm fpistm self-requested a review August 4, 2020 14:34
@fpistm
Copy link
Member

fpistm commented Jul 14, 2021

Hi @xpix
Sorry for the delay. 😕
Are you able to fix your issue with the link error? --> OK just saw your other PR #24

I've planned to made a new release soon and would integrate your PR.

@fpistm
Copy link
Member

fpistm commented Jul 14, 2021

Removing your last commit, I've no link issue.
My guess is you had a cache issue.

Copy link
Member

@fpistm fpistm left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

The variant is not correct and there is no link to the board documentation.
As stated in the PR it is a try and I guess you don't go further.
As the PR is old I will close it.
If you want really to add support please update the variant and we could reopen this PR.


const PinName digitalPin[] = {
//PX_n, //Dx
PD_1, //D7
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

This is not aligned with the enum defined in the variant.h and the NUM_DIGITAL_PINS

@fpistm fpistm closed this Jul 15, 2021
@fpistm fpistm added the invalid This doesn't seem right label Jul 15, 2021
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

invalid This doesn't seem right New variant

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants