Re: [PATCH] ARM: dts: stm32: Rework LAN8710Ai PHY reset on DHCOM SoM

[Greg KH <gregkh@linuxfoundation.org>]

On Mon, Jun 28, 2021 at 03:10:36PM +0200, Marek Vasut wrote:
> On 6/28/21 3:02 PM, Greg KH wrote:
> > On Mon, Jun 28, 2021 at 02:32:50PM +0200, Marek Vasut wrote:
> > > On 6/28/21 2:29 PM, Greg KH wrote:
> > > > On Mon, Jun 28, 2021 at 12:44:37PM +0200, Marek Vasut wrote:
> > > > > On 4/9/21 1:00 AM, Marek Vasut wrote:
> > > > > > The Microchip LAN8710Ai PHY requires XTAL1/CLKIN external clock to be
> > > > > > enabled when the nRST is toggled according to datasheet Microchip
> > > > > > LAN8710A/LAN8710Ai DS00002164B page 35 section 3.8.5.1 Hardware Reset:
> > > > > 
> > > > > [...]
> > > > > 
> > > > > > Fixes: 34e0c7847dcf ("ARM: dts: stm32: Add DH Electronics DHCOM STM32MP1 SoM and PDK2 board")
> > > > > 
> > > > > Adding stable to CC.
> > > > > 
> > > > > Patch is now part of Linux 5.13 as commit
> > > > > 
> > > > > 1cebcf9932ab ("ARM: dts: stm32: Rework LAN8710Ai PHY reset on DHCOM SoM")
> > > > 
> > > > $ git show 1cebcf9932ab
> > > > fatal: ambiguous argument '1cebcf9932ab': unknown revision or path not in the working tree.
> > > > Use '--' to separate paths from revisions, like this:
> > > > 'git <command> [<revision>...] -- [<file>...]'
> > > > 
> > > > Are you sure?
> > > 
> > > This would seem to indicate so:
> > > 
> > > https://git.kernel.org/pub/scm/linux/kernel/git/torvalds/linux.git/commit/?id=1cebcf9932ab76102e8cfc555879574693ba8956
> > > 
> > > linux-2.6$ git describe 1cebcf9932ab76102e8cfc555879574693ba8956
> > > v5.13-rc1-1-g1cebcf9932ab
> > > 
> > > Did the commit get abbreviated too much ?
> > 
> > Something is really odd, as that commit _is_ in linux-next, but it is
> > not in my local copy of Linus's tree.
> > 
> > So how it is showing up in that link above is beyond me.  Can you see it
> > locally on your machine?
> 
> Yes, that's where the git describe came from. And I used a different repo
> than the one from which I submitted the patch originally, so the commit
> must've come from fetching origin (i.e. linus tree).
> 
> Could it be this "ambiguous argument '1cebcf9932ab'" , which would indicate
> the commit hash got abbreviated too much ?

The web site "lies" it has a shared backend.  Trust your local copy of
the tree, that shows that this commit is NOT in Linus's tree just yet.
Please let stable@vger know when it does hit Linus's tree and we will be
glad to take it.

thanks,

greg k-h