From mboxrd@z Thu Jan 1 00:00:00 1970 From: Hans de Goede Subject: Re: [PATCH v4 1/2] phy-sun4i-usb: Add full support for usb0 phy / OTG Date: Mon, 01 Jun 2015 11:28:23 +0200 Message-ID: <556C25B7.80708@redhat.com> References: <1433088626-8858-1-git-send-email-hdegoede@redhat.com> <1433088626-8858-2-git-send-email-hdegoede@redhat.com> <20150601092249.GF17267@lukather> Reply-To: hdegoede-H+wXaHxf7aLQT0dZR+AlfA@public.gmane.org Mime-Version: 1.0 Content-Type: text/plain; charset=UTF-8; format=flowed Return-path: In-Reply-To: <20150601092249.GF17267@lukather> List-Post: , List-Help: , List-Archive: , List-Unsubscribe: , To: Maxime Ripard Cc: Felipe Balbi , Kishon Vijay Abraham I , Chen-Yu Tsai , Roman Byshko , linux-usb-u79uwXL29TY76Z2rM5mHXA@public.gmane.org, linux-arm-kernel-IAPFreCvJWM7uuMidbF8XUB+6BGkLq7r@public.gmane.org, devicetree , linux-sunxi-/JYPxA39Uh5TLH3MbocFFw@public.gmane.org List-Id: devicetree@vger.kernel.org Hi, On 01-06-15 11:22, Maxime Ripard wrote: > On Sun, May 31, 2015 at 06:10:25PM +0200, Hans de Goede wrote: >> + /* We either want both gpio pins or neither (when in host mode) */ >> + if (!data->id_det_gpio != !data->vbus_det_gpio) { >> + dev_err(dev, "failed to get id or vbus detect pin\n"); >> + return -ENODEV; >> + } > > The fact that the driver expects both to be set if one is should be in > the binding documentation. That would be behavior description, and I've always been told that the binding should only document the hardware description, not the behavior. Also this may change in the future as we add support for more boards, e.g. the cubieboard seems to be unable to turn vbus off ever, so vbus-det will always read as 1 / high, making it rather useless. This is something which I still need to figure out how to deal with. Regards, Hans