On 23.08.2013 11:28, Sekhar Nori wrote:
> On Friday 23 August 2013 02:13 PM, Daniel Mack wrote:
>> This third memory region just denotes one single register in the CONTROL
>> module block. The driver uses that in order to set the correct physical
>> ethernet interface modes.
>>
>> Also update the compatible string to make use of the am335x specific
>> features of the cpsw driver.
>>
>> Signed-off-by: Daniel Mack <zonque@xxxxxxxxx>
>> ---
>> arch/arm/boot/dts/am33xx.dtsi | 5 +++--
>> 1 file changed, 3 insertions(+), 2 deletions(-)
>>
>> diff --git a/arch/arm/boot/dts/am33xx.dtsi b/arch/arm/boot/dts/am33xx.dtsi
>> index a785b95..1a1e9dd 100644
>> --- a/arch/arm/boot/dts/am33xx.dtsi
>> +++ b/arch/arm/boot/dts/am33xx.dtsi
>> @@ -446,7 +446,7 @@
>> };
>>
>> mac: ethernet@4a100000 {
>> - compatible = "ti,cpsw";
>> + compatible = "ti,am3352-cpsw";
>
> Please keep this as
>
> compatible = "ti,am3352-cpsw", "ti,cpsw";
>
> so users can upgrade the DTB or kernel independently. Apart from this
> the series looks good to me.
Ok, thanks. Given that this last patch will be merged through another
tree, can I just resend it separately, while David takes the rest as it is?
Thanks,
Daniel
--
To unsubscribe from this list: send the line "unsubscribe devicetree" in
the body of a message to majordomo@xxxxxxxxxxxxxxx
More majordomo info at http://vger.kernel.org/majordomo-info.html
