On Tuesday 10 May 2016 10:14 PM, David Lechner wrote:
> On 05/10/2016 06:26 AM, Sergei Shtylyov wrote:
>> On 5/10/2016 2:46 AM, David Lechner wrote:
>>
>>> The CFGCHIP registers are used by a number of devices, so using a syscon
>>> device to share them. The first consumer of this will by the
>>> phy-da8xx-usb
>>> driver.
>>>
>>> Signed-off-by: David Lechner <david@xxxxxxxxxxxxxx>
>> [...]
>>> diff --git a/arch/arm/mach-davinci/devices-da8xx.c
>>> b/arch/arm/mach-davinci/devices-da8xx.c
>>> index 725e693..69d11a1 100644
>>> --- a/arch/arm/mach-davinci/devices-da8xx.c
>>> +++ b/arch/arm/mach-davinci/devices-da8xx.c
>>> @@ -11,6 +11,7 @@
>>> * (at your option) any later version.
>>> */
>>> #include <linux/init.h>
>>> +#include <linux/platform_data/syscon.h>
>>> #include <linux/platform_device.h>
>>> #include <linux/dma-contiguous.h>
>>> #include <linux/serial_8250.h>
>>> @@ -1109,3 +1110,30 @@ int __init da850_register_sata(unsigned long
>>> refclkpn)
>>> return platform_device_register(&da850_sata_device);
>>> }
>>> #endif
>>> +
>>> +static struct syscon_platform_data da8xx_cfgchip_platform_data = {
>>> + .label = "cfgchip",
>>> +};
>>> +
>>> +static struct resource da8xx_cfgchip_resources[] = {
>>> + {
>>> + .start = DA8XX_SYSCFG0_BASE + DA8XX_CFGCHIP0_REG,
>>> + .end = DA8XX_SYSCFG0_BASE + DA8XX_CFGCHIP4_REG + 3,
>>> + .flags = IORESOURCE_MEM,
>>> + },
>>> +};
>>> +
>>> +static struct platform_device da8xx_cfgchip_device = {
>>> + .name = "syscon",
>>> + .id = 0,
>>
>> Again, -1.
>>
>> [...]
>>
>> MBR, Sergei
>>
>
> I wish you would have noticed this when I first submitted it. I remember
> going back and forth about this. But it has been too long and I can't
> remember the reason why I chose to go this way.
>
> It seems like changing it broke something with either this one or the
> phy device and I opted to keep it this way on both to be consistent. For
> example, the USB devices both use id = 0 as well even though there are
> only one of each type.
Agree with Sergei here. Can you confirm what broke exactly? I think the
USB needs to be fixed too.
Thanks,
Sekhar
--
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
