On 3/26/23 02:20, Tobias Dahms wrote: > Hello, > > the bisection gives following result: > -------------------------------------------------------------------- > 18c7deca2b812537aa4d928900e208710f1300aa is the first bad commit > commit 18c7deca2b812537aa4d928900e208710f1300aa > Author: AngeloGioacchino Del Regno <angelogioacchino.delregno@xxxxxxxxxxxxx> > Date: Tue May 17 12:47:08 2022 +0200 > > soc: mediatek: pwrap: Use readx_poll_timeout() instead of custom > function > > Function pwrap_wait_for_state() is a function that polls an address > through a helper function, but this is the very same operation that > the readx_poll_timeout macro means to do. > Convert all instances of calling pwrap_wait_for_state() to instead > use the read_poll_timeout macro. > > Signed-off-by: AngeloGioacchino Del Regno > <angelogioacchino.delregno@xxxxxxxxxxxxx> > Reviewed-by: Nícolas F. R. A. Prado <nfraprado@xxxxxxxxxxxxx> > Tested-by: Nícolas F. R. A. Prado <nfraprado@xxxxxxxxxxxxx> > Link: > https://lore.kernel.org/r/20220517104712.24579-2-angelogioacchino.delregno@xxxxxxxxxxxxx > Signed-off-by: Matthias Brugger <matthias.bgg@xxxxxxxxx> > > drivers/soc/mediatek/mtk-pmic-wrap.c | 60 > ++++++++++++++++++++---------------- > 1 file changed, 33 insertions(+), 27 deletions(-) > -------------------------------------------------------------------- > OK, I'm updating the regression status: #regzbot introduced: 18c7deca2b8125 And for replying, don't top-post, but rather reply inline with appropriate context instead; hence I cut the replied context. -- An old man doll... just what I always wanted! - Clara
