Re: [PATCH] sdhci: add quirk to support shared bus controller

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

 



Hi,

On Mon, Apr 25 2011, Philip Rakity wrote:
> A quirk does not seem to the right way to go since there is now 2 points of failure
>
> a) The quirk can be defined and host->ops->get_shared_pins may not be.
>
> how about
>
> 	if (host->ops->get_shared_pins)
> 		host->ops->get_shared_pins(host)

Yep, I agree with Philip -- even if you use a quirk, you should test a
function pointer before calling it.  I also agree that you don't need
a quirk here because this isn't a modification to SDHCI spec behavior,
it's just a hook into platform code.

Thanks,

- Chris.
-- 
Chris Ball   <cjb@xxxxxxxxxx>   <http://printf.net/>
One Laptop Per Child
--
To unsubscribe from this list: send the line "unsubscribe linux-mmc" in
the body of a message to majordomo@xxxxxxxxxxxxxxx
More majordomo info at  http://vger.kernel.org/majordomo-info.html


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

  Powered by Linux