On 06/10/2016 05:47 AM, Sriram Dash wrote:
>> -----Original Message-----
>> From: Marek Vasut [mailto:ma...@denx.de]
>> Sent: Thursday, June 09, 2016 7:01 PM
>> To: Sriram Dash <sriram.d...@nxp.com>; u-boot@lists.denx.de
>> Cc: york sun <york....@nxp.com>; albert.u.b...@aribaud.net; Rajesh Bhagat
>> <rajesh.bha...@nxp.com>
>> Subject: Re: [PATCH v3 2/5] usb: xhci: fsl: code cleanup for device tree
>> fixup for fsl
>> usb controllers
Would you please fix your mailer to omit this useless header ?
[...]
>>> diff --git a/include/fsl_usb.h b/include/fsl_usb.h index
>>> 187e384..882a5f5 100644
>>> --- a/include/fsl_usb.h
>>> +++ b/include/fsl_usb.h
>>> @@ -85,6 +85,12 @@ struct ccsr_usb_phy { #define
>>> CONFIG_SYS_FSL_USB_SQUELCH_PROG_MASK 0x07 #endif
>>>
>>> +/* USB Controllers */
>>> +#define FSL_USB2_MPH "fsl-usb2-mph"
>>> +#define FSL_USB2_DR "fsl-usb2-dr"
>>> +#define USB2_CI "usb2-ci"
What is this "usb2-ci" string ? Is this some DT property or DT
compatible property or just some random string?
>>> +#define SNPS_DWC3 "snps,dwc3"
>>
>> Is this needed as a global macro or can it be local to fsl-dt-fixup.c ?
>>
>
> Currently, the macro are only being used for dt fixup for fsl usb.
> However, in the near future, they might be needed for some other purpose,
> for example, board specific settings.
If they will be used for other purpose, then they can be moved. They are
local now, so keep them local.
> So, i think i will stick with the global macro.
>
>>> /* USB Erratum Checking code */
>>> #ifdef CONFIG_PPC
>>> static inline bool has_dual_phy(void)
>>>
>>
>>
>> --
>> Best regards,
>> Marek Vasut
--
Best regards,
Marek Vasut
_______________________________________________
U-Boot mailing list
U-Boot@lists.denx.de
http://lists.denx.de/mailman/listinfo/u-boot
