Hi,
On Tue, Nov 26, 2013 at 10:25:26AM +0100, Markus Pargmann wrote:
> > On Mon, Nov 18, 2013 at 04:54:37PM +0100, Markus Pargmann wrote:
> > > @@ -350,6 +373,30 @@ out:
> > > return ret;
> > > }
> > >
> > > +static int dsps_musb_dbg_init(struct musb *musb, struct dsps_glue *glue)
> > > +{
> > > + struct dentry *root;
> > > + struct dentry *file;
> > > + char buf[128];
> > > +
> > > + sprintf(buf, "%s.dsps", dev_name(musb->controller));
> >
> > why ? just use the glue's device
>
> I used this to have a more obvious relationship between musb-core
> debugfs and musb-dsps debugfs. Otherwise we would have '47401400.usb'
> for dsps and 'musb-hdrc.0.auto' for musb core.
>
> But I can change it if you want.
yeah, I guess it's best to use the device name on this one, even though
it'll look at bit weird. At least it'll be guaranteed to be unique.
--
balbi
Attachment:
signature.asc
Description: Digital signature
