Re: [PATCH] ARM: OMAP: pdata-quirks: Remove pbias reference from Torpedo

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

 



* Adam Ford <aford173@xxxxxxxxx> [181227 15:53]:
> On Sun, Dec 23, 2018 at 10:17 AM Tony Lindgren <tony@xxxxxxxxxxx> wrote:
> >
> > * Adam Ford <aford173@xxxxxxxxx> [181222 20:17]:
> > > This fixes an issue where the MMC errors our during boot which
> > > requires the user to eject and re-insert the MMC.
> > >
> > > The bootloader has been updated for a while to correct set the
> > > pbias for the GPIO pins.  Kernels 4.20-RC1+ are getting hung up
> > > on intializing the MMC.  Removing the duplicative PBIAS setting
> > > appears to eliminate the issue.
> >
> > Hmm I wonder if the root cause might be some PBIAS regulator
> > regression between v4.19 and v4.20-rc series?
> 
> Possibly, but  I haven't bisected it, but do you think this is even
> necessary if the bootloader correctly addresses this now?  If
> anything, pdata quirks seems to me to be the wrong place, and we
> should add the IO disable/enable features into the pbias driver.  I'd
> like to get this applied for 4.20.1 if possible, but I understand if
> you want more investigation.  I am just not sure when I can get to it.
> As of right now, 4.20 doesn't properly boot the Torpedo board without
> ejecting and reinserting the card.

Yeah so there seems to be a MMC card detect GPIO regression
caused by commit e63201f19438 ("mmc: omap_hsmmc: Delete platform
data GPIO CD and WP") that I just sent an email about.

Anyways, yeah if we can do the IO disable/enable into the pbias
driver and get rid of more pdata-quirks.c handling that would
be great.

Care to check after reverting e63201f19438 and removing the
pdata-quirks.c pieces if more changes are still needed?

Regards,

Tony



[Index of Archives]     [Linux Arm (vger)]     [ARM Kernel]     [ARM MSM]     [Linux Tegra]     [Linux WPAN Networking]     [Linux Wireless Networking]     [Maemo Users]     [Linux USB Devel]     [Video for Linux]     [Linux Audio Users]     [Yosemite Trails]     [Linux Kernel]     [Linux SCSI]

  Powered by Linux