[U-Boot] [PATCH] dts: am335x-pdu001: Fix polarity of card detection input

[Tom Rini <trini@konsulko.com>]

On Thu, Nov 29, 2018 at 05:07:47PM +0100, Felix Brack wrote:
> On 29.11.2018 16:52, Tom Rini wrote:
> > On Thu, Nov 29, 2018 at 04:33:36PM +0100, Felix Brack wrote:
> >> On 29.11.2018 16:25, Tom Rini wrote:
> >>> On Thu, Nov 29, 2018 at 01:45:06PM +0100, Felix Brack wrote:
> >>>
> >>>> When a micro SD card is inserted in the PDU001 card cage, the card
> >>>> detection switch is opened and the corresponding GPIO input is driven
> >>>> by a pull-up. Hence change the active level of the card detection
> >>>> input from low to high.
> >>>>
> >>>> Signed-off-by: Felix Brack <fb@ltec.ch>
> >>>> ---
> >>>>
> >>>>  arch/arm/dts/am335x-pdu001.dts | 2 +-
> >>>>  1 file changed, 1 insertion(+), 1 deletion(-)
> >>>>
> >>>> diff --git a/arch/arm/dts/am335x-pdu001.dts b/arch/arm/dts/am335x-pdu001.dts
> >>>> index 121e2c6207..3a5e952663 100644
> >>>> --- a/arch/arm/dts/am335x-pdu001.dts
> >>>> +++ b/arch/arm/dts/am335x-pdu001.dts
> >>>> @@ -576,7 +576,7 @@
> >>>>  	bus-width = <4>;
> >>>>  	pinctrl-names = "default";
> >>>>  	pinctrl-0 = <&mmc2_pins>;
> >>>> -	cd-gpios = <&gpio2 2 GPIO_ACTIVE_LOW>;
> >>>> +	cd-gpios = <&gpio2 2 GPIO_ACTIVE_HIGH>;
> >>>>  };
> >>>>  
> >>>>  &sham {
> >>>
> >>> Is this in upstream Linux as well?  If so what tag/hash?  Thanks!
> >>>
> >> Not yet. I will send the Linux patch within a few days.
> >> The fix appears for U-Boot first as it is required for the upcoming
> >> patch that enables CONFIG_BLK and CONFIG_DM_MMC for this board.
> > 
> > Please reply to the thread with a ML link there so we can track and make
> > sure it doesn't get out of sync, thanks!
> > I'm not sure if I got that correctly. What is an 'ML link'?
> Was there something wrong with my last post or would you like me to post
> a message in this thread once the patch is in upstream Linux?

The general preference is that when we touch the dts files that aren't
U-Boot centric the commit message references the Linux githash/tag it's
taken from, for easier future re-syncs.  Since you're in progress on
fixing this in Linux too, you can just reply here so it's at least
tracked on the ML and archives that the change _is_ going upstream so we
aren't likely to overwrite it by accident later (and since you'd be the
one pushing a future re-sync, you'd also notice, so it's really not
likely to happen).  Thanks!

-- 
Tom
-------------- next part --------------
A non-text attachment was scrubbed...
Name: signature.asc
Type: application/pgp-signature
Size: 819 bytes
Desc: not available
URL: <http://lists.denx.de/pipermail/u-boot/attachments/20181129/cf106c3f/attachment.sig>