[PATCH] usb: gadget/fsl_mxc_udc: Detect the CPU type in run-time

[fabio.estevam@freescale.com (Fabio Estevam)]

On 1/19/2011 1:36 PM, Eric B?nard wrote:
> Hi Fabio,
> 
> On 19/01/2011 16:32, Fabio Estevam wrote:
>> Instead of checking the CPU type in build-time, do it in run-time.
>>
>> Signed-off-by: Fabio Estevam<fabio.estevam@freescale.com>
>> ---
>>   drivers/usb/gadget/fsl_mxc_udc.c |   23 ++++++++++++-----------
>>   1 files changed, 12 insertions(+), 11 deletions(-)
>>
>> diff --git a/drivers/usb/gadget/fsl_mxc_udc.c b/drivers/usb/gadget/fsl_mxc_udc.c
>> index 77b1eb5..167abe0 100644
>> --- a/drivers/usb/gadget/fsl_mxc_udc.c
>> +++ b/drivers/usb/gadget/fsl_mxc_udc.c
>> @@ -88,17 +88,18 @@ eenahb:
>>   void fsl_udc_clk_finalize(struct platform_device *pdev)
>>   {
>>       struct fsl_usb2_platform_data *pdata = pdev->dev.platform_data;
>> -#if defined(CONFIG_ARCH_MX35)
>> -    unsigned int v;
>> -
>> -    /* workaround ENGcm09152 for i.MX35 */
>> -    if (pdata->workaround&  FLS_USB2_WORKAROUND_ENGCM09152) {
>> -        v = readl(MX35_IO_ADDRESS(MX35_USB_BASE_ADDR +
>> -                USBPHYCTRL_OTGBASE_OFFSET));
>> -        writel(v | USBPHYCTRL_EVDO, MX35_IO_ADDRESS(MX35_USB_BASE_ADDR +
>> -                USBPHYCTRL_OTGBASE_OFFSET));
>> -    }
>> -#endif
>> +    if (cpu_is_mx35()) {
>> +        unsigned int v;
>> +
>> +        /* workaround ENGcm09152 for i.MX35 */
>> +        if (pdata->workaround&  FLS_USB2_WORKAROUND_ENGCM09152) {
>> +            v = readl(MX35_IO_ADDRESS(MX35_USB_BASE_ADDR +
>> +                    USBPHYCTRL_OTGBASE_OFFSET));
>> +            writel(v | USBPHYCTRL_EVDO,
>> +                MX35_IO_ADDRESS(MX35_USB_BASE_ADDR +
>> +                    USBPHYCTRL_OTGBASE_OFFSET));
>> +        }
>> +    }
>>
> are you sure this compiles when MX35 is not selected ?

Including #include <mach/mx35.h> fixes the build for a non- mx35 kernel.

Would this be acceptable? If so, I can send v2.

Regards,

Fabio Estevam