Skip to content

Conversation

reey
Copy link

@reey reey commented Sep 15, 2025

The device is a battery powered WiFi/LTE hotspot.

The device is a battery powered WiFi/LTE hotspot.
Copy link

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Minor coding style check

Comment on lines +64 to +65
pinctrl-names = "default";
pinctrl-0 = <&gpio_leds_default>;

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Suggested change
pinctrl-names = "default";
pinctrl-0 = <&gpio_leds_default>;
pinctrl-0 = <&gpio_leds_default>;
pinctrl-names = "default";

Comment on lines +49 to +50
pinctrl-names = "default";
pinctrl-0 = <&gpio_keys_default>;

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Suggested change
pinctrl-names = "default";
pinctrl-0 = <&gpio_keys_default>;
pinctrl-0 = <&gpio_keys_default>;
pinctrl-names = "default";

Comment on lines +133 to +134
pinctrl-names = "default";
pinctrl-0 = <&usb_id_default>;

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Suggested change
pinctrl-names = "default";
pinctrl-0 = <&usb_id_default>;
pinctrl-0 = <&usb_id_default>;
pinctrl-names = "default";

Comment on lines +173 to +176
status = "okay";

monitored-battery = <&battery>;
power-supplies = <&pm8916_charger>;

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Suggested change
status = "okay";
monitored-battery = <&battery>;
power-supplies = <&pm8916_charger>;
monitored-battery = <&battery>;
power-supplies = <&pm8916_charger>;
status = "okay";

Comment on lines +180 to +185
status = "okay";

monitored-battery = <&battery>;

qcom,fast-charge-safe-current = <900000>;
qcom,fast-charge-safe-voltage = <4300000>;

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Suggested change
status = "okay";
monitored-battery = <&battery>;
qcom,fast-charge-safe-current = <900000>;
qcom,fast-charge-safe-voltage = <4300000>;
monitored-battery = <&battery>;
qcom,fast-charge-safe-current = <900000>;
qcom,fast-charge-safe-voltage = <4300000>;
status = "okay";

Comment on lines +189 to +191
status = "okay";
pinctrl-names = "default";
pinctrl-0 = <&pwm_out>;

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Suggested change
status = "okay";
pinctrl-names = "default";
pinctrl-0 = <&pwm_out>;
pinctrl-0 = <&pwm_out>;
pinctrl-names = "default";
status = "okay";

Comment on lines +206 to +207
status = "okay";
extcon = <&pm8916_charger>, <&usb_otg_hack>;

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Suggested change
status = "okay";
extcon = <&pm8916_charger>, <&usb_otg_hack>;
extcon = <&pm8916_charger>, <&usb_otg_hack>;
status = "okay";

/*
* NOTE: This allows detection of custom-made OTG adapters
*
* Since L8150 doesn't support OTG it's impossible to use normal OTG
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

this is not L8150. Is this true? Did you test this?

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Development

Successfully merging this pull request may close these issues.

3 participants