Re: [PATCH] Vibrator Driver for Intel MID Platforms

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

 



On Tue, Jun 01, 2010 at 01:35:13PM +0100, Alan Cox wrote:
> > > + *	If the PMIC hasn't been discovered or one is not found
> > > then
> > > + *	the calls will error for us.
> > 
> > Can't you do this at probe time?
> 
> We have to check for errors anyway. We could also check at probe time
> but that would add ordering issues or mean we needed the scu_ipc driver
> to create the platform device for the vibrator - which given it's not
> current discoverable is a bit ugly.

The ipc driver is presumably intended to probe first (and module 
dependencies will ensure that in the modular case). Once that probe has 
completed, isn't it expected that scu_ipc_ioread will error if there's 
no device?

-- 
Matthew Garrett | mjg59@xxxxxxxxxxxxx
--
To unsubscribe from this list: send the line "unsubscribe platform-driver-x86" in
the body of a message to majordomo@xxxxxxxxxxxxxxx
More majordomo info at  http://vger.kernel.org/majordomo-info.html


[Index of Archives]     [Linux Kernel Development]     [Linux USB Devel]     [Video for Linux]     [Linux Audio Users]     [Yosemite News]     [Linux Kernel]     [Linux SCSI]

  Powered by Linux