mbox series

[0/2] RTQ6056: Add compatible for the same chip family

Message ID cover.1703734994.git.cy_huang@richtek.com
Headers show
Series RTQ6056: Add compatible for the same chip family | expand

Message

ChiYuan Huang Dec. 28, 2023, 3:51 a.m. UTC
From: ChiYuan Huang <cy_huang@richtek.com>

*** Resend for the '--thread' config in 'git format-patch'.

RTQ6053 and RTQ6059 are the same RTQ6056 family.
The differences are listed below
- RTQ6053
  Only change chip package type
- RTQ6059
  1. Enlarge the shunt voltage sensing range
  2. Shrink the pinout for VBUS sense pin
  3. Due to 1, the scale value is also changed

ChiYuan Huang (2):
  dt-bindings: iio: adc: rtq6056: add support for the whole RTQ6056
    family
  iio: adc: rtq6056: Add support for the whole RTQ6056 family

 .../bindings/iio/adc/richtek,rtq6056.yaml     |   5 +-
 drivers/iio/adc/rtq6056.c                     | 265 +++++++++++++++++-
 2 files changed, 255 insertions(+), 15 deletions(-)

Comments

Krzysztof Kozlowski Dec. 28, 2023, 7:16 a.m. UTC | #1
On 28/12/2023 04:51, cy_huang@richtek.com wrote:
> From: ChiYuan Huang <cy_huang@richtek.com>
> 
> *** Resend for the '--thread' config in 'git format-patch'.

Resending the same after you receive feedback means you entirely bypass
that feedback. Don't do this. Ever.

Best regards,
Krzysztof
ChiYuan Huang Dec. 28, 2023, 7:27 a.m. UTC | #2
On Thu, Dec 28, 2023 at 08:16:40AM +0100, Krzysztof Kozlowski wrote:
> On 28/12/2023 04:51, cy_huang@richtek.com wrote:
> > From: ChiYuan Huang <cy_huang@richtek.com>
> > 
> > *** Resend for the '--thread' config in 'git format-patch'.
> 
> Resending the same after you receive feedback means you entirely bypass
> that feedback. Don't do this. Ever.
> 
Got it. I'm just trying to figure out why Jonathan said the thread cannot be found in
lore.kernel.org. Eventually I found that's the 'thread' option.

When sending v2 patch series, I think I won't forget it in the future.

Not inteneded to bypass the whole suggestion or feedback.

Sorry for that.

> Best regards,
> Krzysztof
>