On Thu, Jan 7, 2021 at 3:17 PM Saravana Kannan <[email protected]> wrote: > > On Thu, Jan 7, 2021 at 12:59 AM Greg Kroah-Hartman > <[email protected]> wrote: > > > > On Wed, Jan 06, 2021 at 03:26:41PM -0800, Saravana Kannan wrote: > > > The device link device's name was of the form: > > > <supplier-dev-name>--<consumer-dev-name> > > > > > > This can cause name collision as reported here [1] as device names are > > > not globally unique. Since device names have to be unique within the > > > bus/class, add the bus/class name as a prefix to the device names used to > > > construct the device link device name. > > > > > > So the devuce link device's name will be of the form: > > > <supplier-bus-name>:<supplier-dev-name>--<consumer-bus-name><consumer-dev-name> > > > > Minor nit, you forgot a ':' in the consumer side of the link here. The > > code is correct. > > Greg, > > Do you want me to send a v2 to fix this and add "fixes"? Or will you > just fix it up when picking it up? >
Actually, let me send a v2. There's another collision in the symlink's name. -Saravana
