On 02/14/2019 06:28 PM, Heikki Krogerus wrote:

>>>>> +static int tps6598x_check_mode(struct tps6598x *tps)
>>>>> +{
>>>>> + char mode[5] = { };
>>>>> + int ret;
>>>>> +
>>>>> + ret = tps6598x_read32(tps, TPS_REG_MODE, (void *)mode);
>>>>
>>>>    Casting pointers to 'void *' happens automagically, doesn't i?
>>>
>>> The third parameter in tps6598x_read32() is 'u32 *'.
>>
>>    Then why cast to 'void *'?
> 
> Because if we don't cast:
> 
> drivers/usb/typec/tps6598x.c:408:43: error: passing argument 3 of 
> ‘tps6598x_read32’ from incompatible pointer type

   Why not cast to 'u32 *', I meant?

MBR, Sergei

Reply via email to