On Fri, 5 Jul 2024 06:02:53 -0700 Christoph Hellwig <hch@xxxxxxxxxxxxx> wrote: > If you just did a: > > if (npem_has_dsm(dev)) > return; > > we'd save a level of identation for the !dsm case an make the > code a bit easier to read. Also I think "OS" above should be "The OS". Thanks for review! I did it way because in next patch I'm adding DSM. With that I have less changes in second patch. Mariusz