Re: [PATCH 0/6] mmc: omap_hsmmc: simplify cover/card detect logic

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

 



2015-03-04 6:31 GMT+01:00 NeilBrown <neilb@xxxxxxx>:
> On Tue,  3 Mar 2015 13:28:12 +0100 Andreas Fenkart <afenkart@xxxxxxxxx> wrote:
>> While cover detect is only used by one platform (rx51), it
>> complicates the card detect logic. By separating the code
>> paths they both become easier to understand and maintain
>>
>> Patches have been tested by reverting: 95bebb5696ab
>> 'mmc: omap_hsmmc: use mmc_of_parse to parse common mmc configuration'
>> otherwise gpio detection is handled by mmc_of_parse
>
> Wouldn't it make more sense to put this core  in mmc/core rather than in just
> one host controller?
Yes, I would very welcome that, since it would free the omap_hsmmc of the
'protection' mechanism that makes the driver clumsy. But as an initial step
separating cover from card detect already seems enough for this series.
Another thought was, that since only one platform is using it, the prospect of
getting it merged into the core was minimal.

> That way it would be available to all hosts, and you
> wouldn't need to revert that patch.
Oh sorry, seems I was unclear. I definitely do not want to revert that patch
permanently, the opposite is true, I very much welcome it. Thanks for your work
by the way.
I only reverted it temporarily so I could test that the card detect
logic still works
with legacy board files, non-device-tree platforms. But since mmc_of_parse is
setting the card detect pin, my changes could not be tested with my
device-tree platform (bbone). So I reverted the above patch only for testing,
afterwards I rebased the patches on top of it, before submitting


Andreas
--
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