At Sun, 15 Feb 2015 14:25:05 +0700,
Johan Hovold wrote:
>
> On Thu, Feb 05, 2015 at 02:36:23PM +0100, Takashi Iwai wrote:
>
> > static int usb_serial_device_probe(struct device *dev)
> > {
> > struct usb_serial_driver *driver;
> > @@ -72,15 +79,9 @@ static int usb_serial_device_probe(struct device *dev)
> > goto exit_with_autopm;
> > }
> >
> > - retval = device_create_file(dev, &dev_attr_port_number);
> > - if (retval) {
> > - if (driver->port_remove)
> > - retval = driver->port_remove(port);
> > - goto exit_with_autopm;
> > - }
> > -
> > minor = port->minor;
> > - tty_register_device(usb_serial_tty_driver, minor, dev);
> > + tty_register_device_attr(usb_serial_tty_driver, minor, dev, NULL,
> > + usb_serial_dev_groups);
>
> This is just wrong. You're moving a usb-serial-port device attribute to
> the tty class device (i.e. ABI breakage) and you're not updating the
> attribute operation either.
>
> I assume you haven't tested this, as the kernel would likely crash or,
> if you're lucky, just return a bogus port number when you try to read
> the attribute file.
Hm, right, drop this one. Sorry for the noise.
Takashi
--
To unsubscribe from this list: send the line "unsubscribe linux-usb" in
the body of a message to [email protected]
More majordomo info at http://vger.kernel.org/majordomo-info.html
