Re: staging: media: lirc: lirc_zilog.c: replace custom print macros with dev_* and pr_*

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 



Hi Sean,

Em Thu, 06 Nov 2014 12:46:29 +0000
Sean Young <sean@xxxxxxxx> escreveu:

> On Fri, Oct 31, 2014 at 05:35:41PM +0300, Dan Carpenter wrote:
> > On Fri, Oct 31, 2014 at 04:26:45PM +0200, Aya Mahfouz wrote:
> > > On Fri, Oct 31, 2014 at 04:06:00PM +0300, Dan Carpenter wrote:
> > > > drivers/staging/media/lirc/lirc_zilog.c
> > > >   1333  /* Close the IR device */
> > > >   1334  static int close(struct inode *node, struct file *filep)
> > > >   1335  {
> > > >   1336          /* find our IR struct */
> > > >   1337          struct IR *ir = filep->private_data;
> > > >   1338  
> > > >   1339          if (ir == NULL) {
> > > >                     ^^^^^^^^^^
> > > >   1340                  dev_err(ir->l.dev, "close: no private_data attached to the file!\n");
> > > >                                 ^^^^^^^^^
> > > > 
> > > > I suggest you just delete the error message.  Can "ir" actually be NULL
> > > > here anyway?
> > > >
> > > 
> > > Since I'm a newbie and this is not my code, I prefer to use pr_err().
> > 
> > This driver doesn't belong to anyone.  Go ahead and take ownership.  The
> > message is fairly worthless and no one will miss it.
> 
> Speaking of ownership, what this driver really needs is to be ported to 
> rc-core. In order to do this it'll need to be able to send raw IR rather
> key codes; I've been peering at the firmware but it neither looks like
> zilog z8 opcodes nor space/pulse information.

Actually, I think that all features provided by this driver were already
migrated into the ir-kbd-i2c (drivers/media/i2c/ir-kbd-i2c.c) driver.

Andy and Jarod worked on this conversion, but we decided, on that time,
to keep lirc_zilog for a while (can't remember why).

Andy/Jarod,

What's the status of the ir-kbd-i2c with regards to Zilog z8 support?

> Does anyone have any contacts at Hauppauge who could help with this?

Probably, it won't be easy to get someone there that worked on it,
as this device is too old.

Anyway, if are there anything still pending, I may be able to get
some contacts at the vendor.

Regards,
Mauro
--
To unsubscribe from this list: send the line "unsubscribe linux-media" in
the body of a message to majordomo@xxxxxxxxxxxxxxx
More majordomo info at  http://vger.kernel.org/majordomo-info.html




[Index of Archives]     [Linux Input]     [Video for Linux]     [Gstreamer Embedded]     [Mplayer Users]     [Linux USB Devel]     [Linux Audio Users]     [Linux Kernel]     [Linux SCSI]     [Yosemite Backpacking]
  Powered by Linux