On Thu, Jan 26, 2023 at 08:39:59PM +0200, Laurent Pinchart wrote: > Hi Adam, > > On Thu, Jan 26, 2023 at 12:31:16PM -0600, Adam Ford wrote: > > On Thu, Jan 26, 2023 at 11:06 AM Laurent Pinchart wrote: > > > > > > Signed-off-by: Laurent Pinchart <laurent.pinchart@xxxxxxxxxxxxxxxx> > > > --- > > > .../bindings/media/nxp,imx8-isi.yaml | 39 +++++++++++-------- > > > 1 file changed, 22 insertions(+), 17 deletions(-) > > > > Is there a reason not to squash the two bindings into just one patch? > > As indicated in the cover letter, I've kept this separate to clearly > show the impact on the bindings. If using a 'port' node is the preferred > option, I can squash this patch, if using a 'ports' unconditionally is > favoured, then I'll just drop it. Just always use 'ports'. Rob
