Re: [PATCH 1/2] coresight: Set affinity to invalid for missing CPU phandle

2019-06-20 Thread Sai Prakash Ranjan
Hi Mathieu, Thanks for the review comments. On 6/20/2019 11:09 PM, Mathieu Poirier wrote: Hi Sai, On Thu, Jun 20, 2019 at 07:15:46PM +0530, Sai Prakash Ranjan wrote: Affinity defaults to CPU0 in case of missing CPU phandle and this leads to crashes in some cases because of such wrong assumpti

Re: [PATCH 1/2] coresight: Set affinity to invalid for missing CPU phandle

2019-06-20 Thread Mathieu Poirier
Hi Sai, On Thu, Jun 20, 2019 at 07:15:46PM +0530, Sai Prakash Ranjan wrote: > Affinity defaults to CPU0 in case of missing CPU phandle > and this leads to crashes in some cases because of such > wrong assumption. Fix this by returning -ENODEV in > coresight platform for such cases and then handle

Re: [PATCH 1/2] coresight: Set affinity to invalid for missing CPU phandle

2019-06-20 Thread Sai Prakash Ranjan
Hi Suzuki, Thanks for the review. On 6/20/2019 7:25 PM, Suzuki K Poulose wrote: Sai, Thanks for the patch. Please could you change the subject to : "coresight: Do not default to CPU0 for missing CPU phandle" Sure. On 20/06/2019 14:45, Sai Prakash Ranjan wrote: Affinity defaults to CPU0

Re: [PATCH 1/2] coresight: Set affinity to invalid for missing CPU phandle

2019-06-20 Thread Suzuki K Poulose
Sai, Thanks for the patch. Please could you change the subject to : "coresight: Do not default to CPU0 for missing CPU phandle" On 20/06/2019 14:45, Sai Prakash Ranjan wrote: Affinity defaults to CPU0 in case of missing CPU phandle and this leads to crashes in some cases because of such wron

[PATCH 1/2] coresight: Set affinity to invalid for missing CPU phandle

2019-06-20 Thread Sai Prakash Ranjan
Affinity defaults to CPU0 in case of missing CPU phandle and this leads to crashes in some cases because of such wrong assumption. Fix this by returning -ENODEV in coresight platform for such cases and then handle it in the coresight drivers. Signed-off-by: Sai Prakash Ranjan --- drivers/hwtraci