NotificationsYou must be signed in to change notification settings
Fork435
Star4.7k

Commitb08dbd5

authored

Disable DTR clearing on 1200-bps touch (only on Windows) (#2234)

The reason why it was originally introduced:arduino/Arduino@a6909bdWhy we are removing it now?* Windows does preserve the state of the RTS/DTR bits on successive opening of the serial port.* The serial library used in the Arduino IDE 1.8.x has a bug when trying to set DTR=false, on successive opening of the port the DTR line is set back high by the USB serial driver. This works differently from the serial library we use in the Arduino CLI, that sets DTR=false for good and this change is preserved on the successive opening of the port.* Having the serial port left in a state with DTR=false may cause problems to tools uploading later.It may probably completely removed, but for now, to reduce the testingsurface, it will be disabled only for Windows.

1 parent2661f5d commitb08dbd5Copy full SHA for b08dbd5

File tree

1 file changed

+10

-4

lines changed

arduino/serialutils
- serialutils.go

1 file changed

+10

-4

lines changed

`‎arduino/serialutils/serialutils.go‎`

Lines changed: 10 additions & 4 deletions

Original file line number	Diff line number	Diff line change
`@@ -17,6 +17,7 @@ package serialutils`
`17`	`17`
`18`	`18`	`import (`
`19`	`19`	`"fmt"`
	`20`	`+"runtime"`
`20`	`21`	`"strings"`
`21`	`22`	`"time"`
`22`	`23`
`@@ -37,10 +38,15 @@ func TouchSerialPortAt1200bps(port string) error {`
`37`	`38`	`returnerrors.WithMessage(err,tr("opening port at 1200bps"))`
`38`	`39`	`}`
`39`	`40`
`40`		`-// Set DTR to false`
`41`		`-iferr=p.SetDTR(false);err!=nil {`
`42`		`-p.Close()`
`43`		`-returnerrors.WithMessage(err,tr("setting DTR to OFF"))`
	`41`	`+ifruntime.GOOS!="windows" {`
	`42`	`+// This is not required on Windows`
	`43`	`+// TODO: Investigate if it can be removed for other OS too`
	`44`	`+`
	`45`	`+// Set DTR to false`
	`46`	`+iferr=p.SetDTR(false);err!=nil {`
	`47`	`+p.Close()`
	`48`	`+returnerrors.WithMessage(err,tr("setting DTR to OFF"))`
	`49`	`+}`
`44`	`50`	`}`
`45`	`51`
`46`	`52`	`// Close serial port`

0 commit comments

Comments

(0)

Movatterモバイル変換

Navigation Menu

Search code, repositories, users, issues, pull requests...

Provide feedback

Saved searches

Use saved searches to filter your results more quickly

Uh oh!

Uh oh!

Commitb08dbd5

File tree

1 file changed

1 file changed

`‎arduino/serialutils/serialutils.go‎`

0 commit comments