On Mon, Jan 29, 2018 at 12:26:00PM +0200, Laurent Pinchart wrote: > Hi Kieran, > > Thank you for the patch. > > On Monday, 22 January 2018 14:50:00 EET Kieran Bingham wrote: > > The ADV7511 has four 256-byte maps that can be accessed via the main I²C > > ports. Each map has it own I²C address and acts as a standard slave > > device on the I²C bus. > > > > Allow a device tree node to override the default addresses so that > > address conflicts with other devices on the same bus may be resolved at > > the board description level. > > > > Signed-off-by: Kieran Bingham <kieran.bingham@xxxxxxxxxxxxxxxx> > > --- > > .../bindings/display/bridge/adi,adv7511.txt | 10 +++++- > > I don't mind personally, but device tree maintainers usually ask for DT > bindings changes to be split to a separate patch. Or perfect bindings, then I won't ask to split it just for that (usually). > > drivers/gpu/drm/bridge/adv7511/adv7511.h | 4 +++ > > drivers/gpu/drm/bridge/adv7511/adv7511_drv.c | 36 ++++++++++++------- > > 3 files changed, 37 insertions(+), 13 deletions(-)
