On 06/25/2013 03:13 PM, Sergei Shtylyov wrote:
> Hello.
Hello Sergei,
>> index c7c1d7a..4315d35 100644
>> --- a/drivers/usb/musb/omap2430.c
>> +++ b/drivers/usb/musb/omap2430.c
>> @@ -481,7 +481,7 @@ static u64 omap2430_dmamask = DMA_BIT_MASK(32);
>>
>> static int omap2430_probe(struct platform_device *pdev)
>> {
>> - struct resource musb_resouces[2];
>> + struct resource musb_resources[2];
>> struct musb_hdrc_platform_data *pdata = pdev->dev.platform_data;
>> struct omap_musb_board_data *data;
>> struct platform_device *musb;
>> @@ -568,7 +568,7 @@ static int omap2430_probe(struct platform_device
>> *pdev)
>>
>> INIT_WORK(&glue->omap_musb_mailbox_work, omap_musb_mailbox_work);
>>
>> - memset(musb_resouces, 0x00, sizeof(*musb_resources) *
>> + memset(musb_resources, 0x00, sizeof(*musb_resources) *
>> ARRAY_SIZE(musb_resources));
>
> Why not just sizeof(musb_resources)?
It is not wrong, is it? I tried the least possible change to make it
compile again.
I agree that your way would required only one line :) This
"complicated" sizeof is used by all six platforms so each one of them
could be changed.
>
> WBR, Sergei
>
Sebastian
--
To unsubscribe from this list: send the line "unsubscribe linux-usb" in
the body of a message to majordomo@xxxxxxxxxxxxxxx
More majordomo info at http://vger.kernel.org/majordomo-info.html
