Hi Bernd, Thank you very much!
Yes, it appears correct! Let's wait the CI to complete, but so far so good! BR, Alan On 10/29/20, Bernd Walter <ti...@cicely7.cicely.de> wrote: > On Thu, Oct 29, 2020 at 08:48:56AM -0300, Alan Carvalho de Assis wrote: >> Hi Bernd, >> >> Nice finding! Could you please submit a Pull Request? > > https://github.com/apache/incubator-nuttx/pull/2162 > Hope it is correct - this is my first pull request. > >> >> This is very easy, just fork the >> https://github.com/apache/incubator-nuttx to github user, apply your >> patch and click on Pulll requests -> New pull request >> >> Thank you very much! >> >> BR, >> >> Alan >> >> On 10/29/20, Bernd Walter <ti...@cicely7.cicely.de> wrote: >> > When using an I2C master, the file won't compile without the change, >> > because BUSY is undefined >> > >> > diff --git a/contrib/nuttx/arch/arm/src/samd2l2/sam_i2c_master.c >> > b/contrib/nuttx/arch/arm/src/samd2l2/sam_i2c_master.c >> > index 0874a66716..588b355336 100644 >> > --- a/contrib/nuttx/arch/arm/src/samd2l2/sam_i2c_master.c >> > +++ b/contrib/nuttx/arch/arm/src/samd2l2/sam_i2c_master.c >> > @@ -998,7 +998,7 @@ static int sam_i2c_transfer(FAR struct i2c_master_s >> > *dev, >> > >> > /* Initiate the message transfer */ >> > >> > - ret = -BUSY; >> > + ret = -EBUSY; >> > >> > /* Initiate the transfer. The rest will be handled from interrupt >> > logic. >> > * Interrupts must be disabled to prevent re-entrance from the >> > interrupt >> > >> > -- >> > B.Walter <be...@bwct.de> http://www.bwct.de >> > Modbus/TCP Ethernet I/O Baugruppen, ARM basierte FreeBSD Rechner uvm. >> > > > -- > B.Walter <be...@bwct.de> http://www.bwct.de > Modbus/TCP Ethernet I/O Baugruppen, ARM basierte FreeBSD Rechner uvm. >
