Merge pull request #25 from TimerOverflow/20210318

20210318

Author: TimerOverflow

AvrModbus.c

@@ -9,7 +9,7 @@
 #include "AvrModbus.h"
 #include "crc16.h"
 /*********************************************************************************/
-#if(AVR_MODBUS_REVISION_DATE != 20201020)
+#if(AVR_MODBUS_REVISION_DATE != 20210318)
 #error wrong include file. (AvrModbus.h)
 #endif
 /*********************************************************************************/
@@ -723,6 +723,7 @@ static void MasterReceive(tag_AvrModbusMasterCtrl *Master)
 /*********************************************************************************/
 tU8 AvrModbusMasterGeneralInit(tag_AvrModbusMasterCtrl *Master, tag_AvrUartCtrl *Uart, tU8 MaxSlave, tU32 MasterProcTick_us)
 {
+	tU8 i;
 	/*
 		1) 인수
 			- Master : tag_AvrModbusMasterCtrl 인스턴스의 주소.
@@ -738,21 +739,39 @@ tU8 AvrModbusMasterGeneralInit(tag_AvrModbusMasterCtrl *Master, tag_AvrUartCtrl
 			- Master와 관련된 필수 정보들을 초기화 함.
 			- 본 함수를 호출하기전 Uart는 선행적으로 초기화가 완료 되어 있어야 함.
 	*/
-
-	if(Uart->Bit.InitComplete == true)
+	
+	if((Uart == null) || (Uart->Bit.InitComplete != true))
 	{
-		Master->Uart = Uart;
+		return false;
 	}
+	//Uart 초기화 여부 검사
 
 	Master->SlaveArray = (tag_AvrModbusMasterSlaveInfo *) calloc(MaxSlave, sizeof(tag_AvrModbusMasterSlaveInfo));
+	if(Master->SlaveArray == null)
+	{
+		return false;
+	}
+	//tag_AvrModbusMasterSlaveInfo 동적할당
+	
+	for(i = 0; i < MaxSlave; i++)
+	{
+		Master->SlaveArray[i].PollData = (tag_AvrModbusMasterSlavePollData *) calloc(1, sizeof(tag_AvrModbusMasterSlavePollData));
+		if(Master->SlaveArray[i].PollData == null)
+		{
+			return false;
+		}
+	}
+	//tag_AvrModbusMasterSlavePollData 동적 할당
+	
+	Master->Uart = Uart;
 	Master->MaxSlave = MaxSlave;
 	Master->Status = AVR_MODBUS_ReadHolding;
 	Master->Tick_us = MasterProcTick_us;
 	Master->Uart->ReceivingDelay = AVR_MODBUS_RECEIVING_DELAY_US / Master->Tick_us;
 	Master->PollDelay = AVR_MODBUS_DEFAULT_POLLING_DELAY_US / Master->Tick_us;
 	if(Master->Uart->ReceivingDelay < 2) Master->Uart->ReceivingDelay = 2;
 
-	Master->Bit.InitGeneral = ((Master->Uart == null) || (Master->SlaveArray == null)) ? false : true;
+	Master->Bit.InitGeneral = true;
 	Master->Bit.InitComplete = CheckAllOfMasterInit(Master);
 
 	return Master->Bit.InitGeneral;
@@ -821,26 +840,17 @@ tU8 AvrModbusMasterAddSlave(tag_AvrModbusMasterCtrl *Master, tU8 Id, enum_AvrMod
 	{
 		if(Master->SlaveArray[i].Id == 0)
 		{
-			Master->SlaveArray[i].PollData = (tag_AvrModbusMasterSlavePollData *) calloc(1, sizeof(tag_AvrModbusMasterSlavePollData));
-			if(Master->SlaveArray[i].PollData == null)
-			{
-				Master->Bit.PollDataAllocFail = true;
-				break;
-			}
-			else
-			{
-				Master->SlaveArray[i].Id = Id;
-				Master->SlaveArray[i].NoResponseCnt = 0;
-				Master->SlaveArray[i].NoResponseLimit = AVR_MODBUS_DEFAULT_SLAVE_NO_RESPONSE;
-				Master->SlaveArray[i].PollDataMax = 1;
-				Master->SlaveArray[i].PollData[0].PollFunction = PollFunction;
-				Master->SlaveArray[i].PollData[0].StartAddr = StartAddr;
-				Master->SlaveArray[i].PollData[0].NumberOfRegister = NumberOfRegister;
-				Master->SlaveArray[i].PollData[0].BaseAddr = BaseAddr;
-				Master->SlavePoll = &Master->SlaveArray[i];
-				Master->AddedSlave++;
-				return true;
-			}
+			Master->SlaveArray[i].Id = Id;
+			Master->SlaveArray[i].NoResponseCnt = 0;
+			Master->SlaveArray[i].NoResponseLimit = AVR_MODBUS_DEFAULT_SLAVE_NO_RESPONSE;
+			Master->SlaveArray[i].PollDataMax = 1;
+			Master->SlaveArray[i].PollData[0].PollFunction = PollFunction;
+			Master->SlaveArray[i].PollData[0].StartAddr = StartAddr;
+			Master->SlaveArray[i].PollData[0].NumberOfRegister = NumberOfRegister;
+			Master->SlaveArray[i].PollData[0].BaseAddr = BaseAddr;
+			Master->SlavePoll = &Master->SlaveArray[i];
+			Master->AddedSlave++;
+			return true;
 		}
 	}
 

AvrModbus.h

@@ -9,10 +9,15 @@
 /*********************************************************************************/
 #include "AvrUart.h"
 /*********************************************************************************/
-#define AVR_MODBUS_REVISION_DATE				20201020
+#define AVR_MODBUS_REVISION_DATE				20210318
 /*********************************************************************************/
 /** REVISION HISTORY **/
 /*
+	2021. 03. 18.					- Master파트 tag_AvrModbusMasterSlavePollData는 AvrModbusMasterAddSlave(), AvrModbusMasterAddSlavePollData() 호출할 때
+	Jeong Hyun Gu						할당 하게 되어 있는데, Master Init 시점에서 메모리 동적할당 성공여부를 확인할 수 없음.
+													AvrModbusMasterAddSlavePollData()는 보통 잘 사용하지 않으므로 AvrModbusMasterGeneralInit() 시점에서 미리 1개의 tag_AvrModbusMasterSlavePollData를
+													할당하여 Init시점에서 메모리 할당 성공여부 확인할 수있도록 변경.
+
 	2020. 10. 20.					- 2020. 07. 03에서 수정한 내용 AvrModbusMasterAddSlave()에서 사용자가 설정한 PollFunction이
 	Jeong Hyun Gu						반영되지 않고 AVR_MODBUS_ReadHolding로 설정되는 문제 수정.
 
@@ -125,7 +130,7 @@
 #define	false		0
 #define	null		0
 
-#define	AVR_MODBUS_MASTER			true
+#define	AVR_MODBUS_MASTER			false
 #define	AVR_MODBUS_SLAVE			true
 
 #define AVR_MODBUS_RECEIVING_DELAY_US							20000