On Mon, 19 Feb 2024 18:29:21 +0100, Ahmad Fatoum wrote: > The return values of the bus probe function called inside device_probe() > are classified into 4 categories and they are checked by if statement > distributed across device_probe(). > > For clarity and easier changes, collect all of them into a switch > statement and while at it, use helpers to make use of %pe, list_move and > list_del_init instead of opencoding them. > > [...] Applied, thanks! [1/5] driver: refactor probe return value handling into switch statement https://git.pengutronix.de/cgit/barebox/commit/?id=6e08b4ab5bec (link may not be stable) [2/5] deep-probe: treat any probe deferral as permanent https://git.pengutronix.de/cgit/barebox/commit/?id=8a42a6bf42b0 (link may not be stable) [3/5] deep-probe: use IS_ERR_OR_NULL() instead of opencoding https://git.pengutronix.de/cgit/barebox/commit/?id=05785e25af00 (link may not be stable) [4/5] deep-probe: return -EPROBE_DEFER when ensuring probe fails https://git.pengutronix.de/cgit/barebox/commit/?id=1905fa7cb425 (link may not be stable) [5/5] phy: freescale: imx8mq-usb: make i.MX8MP support more explicit https://git.pengutronix.de/cgit/barebox/commit/?id=e87ca6e235e4 (link may not be stable) Best regards, -- Sascha Hauer <s.hauer@xxxxxxxxxxxxxx>