Re: [PATCH v2] mmc: sdhci-of-esdhc: add workaround for clock glitch issue

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

 



>> >> If you have the possibility to enumerate the device ID (reading
>> >> device information directly from the HW), you should do that when
>> >> adding the device to its subsystem level bus, not while probing.
>> >> That's because you need to be able to match for a compatible driver.
>> > Thanks. Could you provide a reference in present code if you know? I'd
>> like to try.
>>
>> I guess the are several subsystem to look into, but maybe AMBA is one
>> that you could have a look at.
>>
>> drivers/amba/bus.c
>>
> Thank you, Uffe.
> I have seen the code of drivers/amba/ and drivers/of/.
> Do you suggest to get device information from HW when adding device in drivers/of ?
> I am afraid it still faces the same machine specific problem and is complicated.

No matter what you need to move things out of the machine specific
directory, if you want other drivers/buses to be able to make use of
it.

Complicated? Maybe, but I don't find that as good argument for not doing it. :-)

>
> I think it is not so unreasonable to get QorIQ silicon rev in sdhci-of-esdhc.c, because this file is only used for freescale QorIQ platform.

Sorry, I will not accept patches that include machine specific
headers. We can do better than that.

Kind regards
Uffe
--
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