26 Jul
2019
26 Jul
'19
8:26 a.m.
Dear Chuanhua Han,
In message AM6PR04MB43577AAE78F9BC0B12E5D07897C10@AM6PR04MB4357.eurprd04.prod.outlook.com you wrote:
Is this bisectable? You first enable the feature in the code, and only later add the needed property to the DT? Should it not be reversed?
This should not matter because they are in the same patch set
You completely miss the point what bisecting means!!
Yes, of course it _does_ matter, as bisecting may apply only parts of your patch series, and omit the rest. If it selects the commit before the last, the code will break.
Please fix this!
Best regards,
Wolfgang Denk
--
DENX Software Engineering GmbH, Managing Director: Wolfgang Denk
HRB 165235 Munich, Office: Kirchenstr.5, D-82194 Groebenzell, Germany
Phone: (+49)-8142-66989-10 Fax: (+49)-8142-66989-80 Email: wd@denx.de
If there was anything that depressed him more than his own cynicism,
it was that quite often it still wasn't as cynical as real life.
- Terry Pratchett, _Guards! Guards!_