Groups | Search | Server Info | Keyboard shortcuts | Login | Register [http] [https] [nntp] [nntps]


Groups > linux.kernel > #1650981

Re: [GIT PULL (RESEND)] ARM: uniphier: UniPhier fixes for v4.12

From Olof Johansson <olof@lixom.net>
Newsgroups linux.kernel
Subject Re: [GIT PULL (RESEND)] ARM: uniphier: UniPhier fixes for v4.12
Date 2017-05-26 02:10 +0200
Message-ID <tLaZj-6RO-5@gated-at.bofh.it> (permalink)
References <tJKXf-7xh-5@gated-at.bofh.it>
Organization linux.* mail to news gateway

Show all headers | View raw


On Mon, May 22, 2017 at 11:03:49AM +0900, Masahiro Yamada wrote:
> Hi Arnd, Olof,
> 
> Last week, I thought I found a problem in my pull request,
> but it turned out my misunderstanding.
> 
> So, my pull request I sent is OK.
> I am re-sending this request mail for clarification.
> 
> Here are some ARM UniPhier SoC fixes for v4.12.
> Due to the eMMC driver update, PHY parameters must be specified
> via DT to make the eMMC controller work again.

That's not good, that's a regression in behavior. The driver should
have a fallback when the properties are not present to avoid regressions
like these.

Please work with the driver maintainer to either get them to revert
their change that regresses you, or promptly fix it appropriately with
fallback values.

Also, the driver changes were commited back in March, with plenty of time
to land DT changes well before the merge window. Waiting until the merge
window is closed is not appropriate -- this should have come in earlier.


-Olof

Back to linux.kernel | Previous | Next — Next in thread | Find similar | Unroll thread


Thread

Re: [GIT PULL (RESEND)] ARM: uniphier: UniPhier fixes for v4.12 Olof Johansson <olof@lixom.net> - 2017-05-26 02:10 +0200
  Re: [GIT PULL (RESEND)] ARM: uniphier: UniPhier fixes for v4.12 Masahiro Yamada <yamada.masahiro@socionext.com> - 2017-05-26 03:50 +0200

csiph-web