rpmsg: remove the address check in rpmsg_send/rpmsg_trysend

xiaoxiang781216 · arnopo · commit ff798a9103a0 · 2020-05-13T09:52:55.000+02:00
since rpmsg_send_offchannel_raw already do the same validation

Signed-off-by: Xiang Xiao &lt;xiaoxiang@xiaomi.com&gt;
diff --git a/lib/include/openamp/rpmsg.h b/lib/include/openamp/rpmsg.h
@@ -146,8 +146,6 @@ int rpmsg_send_offchannel_raw(struct rpmsg_endpoint *ept, uint32_t src,
 static inline int rpmsg_send(struct rpmsg_endpoint *ept, const void *data,
 			     int len)
 {
-	if (ept->dest_addr == RPMSG_ADDR_ANY)
-		return RPMSG_ERR_ADDR;
 	return rpmsg_send_offchannel_raw(ept, ept->addr, ept->dest_addr, data,
 					 len, true);
 }
@@ -216,8 +214,6 @@ static inline int rpmsg_send_offchannel(struct rpmsg_endpoint *ept,
 static inline int rpmsg_trysend(struct rpmsg_endpoint *ept, const void *data,
 				int len)
 {
-	if (ept->dest_addr == RPMSG_ADDR_ANY)
-		return RPMSG_ERR_ADDR;
 	return rpmsg_send_offchannel_raw(ept, ept->addr, ept->dest_addr, data,
 					 len, false);
 }

Original file line number	Diff line number	Diff line change
`@@ -146,8 +146,6 @@ int rpmsg_send_offchannel_raw(struct rpmsg_endpoint *ept, uint32_t src,`
`146`	`146`	`static inline int rpmsg_send(struct rpmsg_endpoint ept, const void data,`
`147`	`147`	`int len)`
`148`	`148`	`{`
`149`		`- if (ept->dest_addr == RPMSG_ADDR_ANY)`
`150`		`- return RPMSG_ERR_ADDR;`
`151`	`149`	`return rpmsg_send_offchannel_raw(ept, ept->addr, ept->dest_addr, data,`
`152`	`150`	`len, true);`
`153`	`151`	`}`
`@@ -216,8 +214,6 @@ static inline int rpmsg_send_offchannel(struct rpmsg_endpoint *ept,`
`216`	`214`	`static inline int rpmsg_trysend(struct rpmsg_endpoint ept, const void data,`
`217`	`215`	`int len)`
`218`	`216`	`{`
`219`		`- if (ept->dest_addr == RPMSG_ADDR_ANY)`
`220`		`- return RPMSG_ERR_ADDR;`
`221`	`217`	`return rpmsg_send_offchannel_raw(ept, ept->addr, ept->dest_addr, data,`
`222`	`218`	`len, false);`
`223`	`219`	`}`